-
Couldn't load subscription status.
- Fork 36
Fix CI regression and update Python version support #236
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #236 +/- ##
==========================================
- Coverage 82.21% 82.05% -0.16%
==========================================
Files 102 102
Lines 7555 7556 +1
==========================================
- Hits 6211 6200 -11
- Misses 1344 1356 +12 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me to
Testing on macos-latest now uses Apple M1 chips, so runs on ARM. The required package mkl_fft is not available on ARM. This PR makes mkl_fft an optional dependency for testing.
The distributed FFMpeg of conda now no longer supports the preset argument for some reason. This PR makes the preset argument optional.
This PR also removes support for Python 3.7 & Python 3.8, and adds support for Python 3.12.