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

Drop Python 3.8, in accordance with NEP29 recommendation #315

Merged
merged 5 commits into from
May 10, 2023

Conversation

GenevieveBuckley
Copy link
Collaborator

Closes #314

This PR drops support for python 3.8

The NEP 29 recommendation is to drop support for python 3.8 on April 14th 2023. We're about a month past that date now, so we're dropping support for it.

The second reason for doing this now is that our python 3.8 CI test environments are starting to have some problems with dependency version mismatches, and it's not a good use of maintainer time to figure out those issues when we're about to drop support for python 3.8 anyway.

@GenevieveBuckley
Copy link
Collaborator Author

I don't understand why the GPU CI is returning build failures, when the github actions CI manages to create its environment just fine.

@jakirkham do you have any debugging tips? I'm not finding a lot of useful information in the Jenkins console output.

@jakirkham
Copy link
Member

cc @charlesbluca

Copy link
Member

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

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

@GenevieveBuckley sorry for the GPU CI confusions - looks like the python 3.11 build failed because there currently aren't python 3.11 images for dask-image's GPU CI being published (more info in my comment below); the 3.9 tests do look legitimate though, I'll pull locally and try to reproduce

continuous_integration/gpuci/axis.yaml Outdated Show resolved Hide resolved
Copy link
Member

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

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

@GenevieveBuckley sorry for the GPU CI confusions - looks like the python 3.11 build failed because there currently aren't python 3.11 images for dask-image's GPU CI being published (more info in my comment below); the 3.9 tests do look legitimate though, I'll pull locally and try to reproduce

@charlesbluca
Copy link
Member

Did some digging into the python 3.9 failures on GPU and looks like those are also related to the images - since we are bumping the conda environment files that the images are built against in this PR, they don't yet reflect some of the changes that were made to unblock tests, specifically the Dask version bump.

I will note that locally I was able to unblock these failures by modifying the dask-image install command in the GPU test script:

gpuci_logger "Install dask-image"
python setup.py install

As a pip install pulls the required Dask version.

Python 3.11 images are not available yet. So stick to Python 3.10 images
until that changes.

Co-authored-by: Charles Blackmon-Luca <[email protected]>
@jakirkham
Copy link
Member

After talking with Charles about this offline, it sounds like this CI issue should go away after these changes are in main (though not before). So going to go ahead and merge. Can create a trivial change PR afterwards to make sure things are working (or identify any remaining issues to fix)

@jakirkham jakirkham merged commit c62a503 into dask:main May 10, 2023
@jakirkham
Copy link
Member

Thanks Genevieve and Charles! 🙏

@jakirkham jakirkham mentioned this pull request May 10, 2023
@jakirkham
Copy link
Member

Submitted PR ( #317 ) to retest gpuCI now that this is merged

@jakirkham
Copy link
Member

Fixed one more Python version constrain in PR ( #336 )

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.

Drop python 3.8
3 participants