Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ehpor
Copy link
Owner

@ehpor ehpor commented Oct 3, 2024

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.

@ehpor ehpor added the bugfix label Oct 3, 2024
@ehpor ehpor self-assigned this Oct 3, 2024
@codecov
Copy link

codecov bot commented Oct 3, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 82.05%. Comparing base (3cdb57f) to head (df4497d).
Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
hcipy/plotting/animation.py 80.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

@ehpor ehpor changed the title Fix MacOS on ARM and FFMpeg preset bug Fix CI regression and update Python version support Oct 7, 2024
Copy link
Collaborator

@raphaelpclt raphaelpclt left a 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

@raphaelpclt raphaelpclt marked this pull request as ready for review October 7, 2024 22:32
Copy link
Collaborator

@ivalaginja ivalaginja left a 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

@ehpor ehpor merged commit 5d7835b into master Oct 15, 2024
16 checks passed
@ehpor ehpor deleted the bugfix/ffmpeg_and_macos_arm branch October 15, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants