Skip to content

expand prototype test matrix to different Python versions #6065

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented May 23, 2022

This expands the prototype test matrix to test all Python versions on Linux. This should give us some extra safety to catch stuff like #5801 and https://github.com/pytorch/data/runs/6500848588#step:9:1977, which are not visible on Python 3.7.

Since GitHub actions is free for OSS projects, this does not add any costs.

@pmeier
Copy link
Collaborator Author

pmeier commented May 23, 2022

After some offline discussion with @NicolasHug, we decided to only follow up on this after the release to avoid for the CI going red for something that does not need to be fixed before the release.

@pmeier
Copy link
Collaborator Author

pmeier commented May 24, 2022

@NicolasHug apart from the flaky open file handle warning, expanding the CI matrix also turned up warnings that #5219 tried to solve. I've added fixes for ImageNet and CUB200 here and CI is now green. Do you want me to extract these changes out of this PR and send a separate one, or keep them here?

@NicolasHug
Copy link
Member

Do you want me to extract these changes out of this PR and send a separate one

Yes please!

@pmeier
Copy link
Collaborator Author

pmeier commented May 24, 2022

See #6076.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants