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

Attempt to fix failing CI tests #221

Closed
wants to merge 23 commits into from

Conversation

GenevieveBuckley
Copy link
Contributor

@GenevieveBuckley GenevieveBuckley commented Jun 3, 2024

This PR:

  1. Drops python 3.8, and adds python 3.12 to the CI test matrix
  2. Pins macos-13, the latest Intel mac version, for the CI tests. PySide2 is unavailable for Apple Silicon mac, and pip install imageio[ffmpeg] also does not include the ffmpeg binary for Apple Silicon mac (but imageio can still work if you install ffmpeg separately)
  3. Skips PySide CI tests on Windows platforms, and on all platforms for python >3.10

napari is experiencing CI failures with PySide backends
on Windows platforms for all versions of python,
and on all platforms with python greater than 3.10.
Also, PySide2 is not currently available for macos arm-64 machines.

Python 3.11 PySide problems:

Windows PySide problems:

Previously:

…s for mac arm64 not available via imageio[ffmpeg] plugin
@GenevieveBuckley GenevieveBuckley changed the title Skip PySide test on windows-latest and with python 3.11 Fix failing CI tests Jun 4, 2024
include_package_data = True
install_requires =
imageio
imageio-ffmpeg
imageio[ffmpeg]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should be functionally equivalent, but fewer CI jobs failed when imageio & ffmpeg were installed as a single step.

@GenevieveBuckley
Copy link
Contributor Author

Turns out you can run the PySide tests on Windows (possibly the napari discussion was a bit confused by the fact PySide requires python<=3.10).
Also turns out, it's relatively easy to fix the ffmpeg install problem on Apple Silicon mac as well.

Fixes here: #222

@GenevieveBuckley GenevieveBuckley changed the title Fix failing CI tests Attempt to fix failing CI tests Jun 4, 2024
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.

1 participant