Skip to content

[WIP] Package torchvision with libjpeg-turbo instead of libjpeg #5951

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

Closed
wants to merge 24 commits into from

Conversation

NicolasHug
Copy link
Member

Follow up to #5941

@@ -157,7 +157,9 @@ setup_wheel_python() {
conda create ${CONDA_CHANNEL_FLAGS} -yn "env$PYTHON_VERSION" python="$PYTHON_VERSION"
conda activate "env$PYTHON_VERSION"
# Install libpng from Anaconda (defaults)
conda install ${CONDA_CHANNEL_FLAGS} libpng "jpeg<=9b" -y
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this script used? The jpeg<=9b is definitely stale as we no longer pin the library.

@NicolasHug
Copy link
Member Author

Copying @datumbox's #5941 (comment) here:

If we merge this, we should test that it has no big effects on the model accuracies. We don't need to test all the models, a sample of 2-3 cases will do.

@NicolasHug NicolasHug mentioned this pull request Jun 15, 2023
@NicolasHug
Copy link
Member Author

#7820

@NicolasHug NicolasHug closed this Aug 10, 2023
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.

4 participants