Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'n' component needs to be a numpy array with retained dimensions #174

Merged
merged 3 commits into from
Feb 5, 2024

Conversation

davidhassell
Copy link
Collaborator

Description

'n' component needs to be a numpy array with retained dimensions

Closes #173

Before you get started

Checklist

  • This pull request has a descriptive title and labels
  • This pull request has a minimal description (most was discussed in the issue, but a two-liner description is still desirable)
  • Unit tests have been added (if codecov test fails)
  • Any changed dependencies have been added or removed correctly (if need be)
  • All tests pass (couldn't run them!)

@valeriupredoi
Copy link
Collaborator

many thanks @davidhassell - this was a bug that I meant to fix for ages now 🐛 🍻 Don't worry about the S3 tests, they conked up since last night, virtually hours after your PR, I suspect it's the new botocore package, am gonna fix that after I merge this. Ref test fail outside this PR https://github.com/valeriupredoi/PyActiveStorage/actions/runs/7778058297/job/21228174521

Copy link
Collaborator

@valeriupredoi valeriupredoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍻

@valeriupredoi valeriupredoi merged commit 2a258fb into main Feb 5, 2024
1 of 5 checks passed
@valeriupredoi valeriupredoi deleted the n-dimensions branch February 5, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'n' component needs to be a numpy array with retained dimensions
2 participants