Skip to content

Conversation

@pmeier
Copy link
Contributor

@pmeier pmeier commented Dec 5, 2022

This was reported offline by @ejguan. We previously ignored a warning there thinking that it was actually scipy.io.savemat that is causing it. Turns out, we were providing deprecated input data and starting with numpy=1.24 this fails. This is visible in torchdata CI, e.g. https://github.com/pytorch/data/actions/runs/3579763020.

I've temporarily expanded the prototype test matrix to cover all cases.

cc @bjuncek

@pmeier pmeier changed the title Imagenet mock data fix prototype dataset imagenet mock data Dec 5, 2022
Copy link
Contributor

@ejguan ejguan left a comment

Choose a reason for hiding this comment

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

Thank you for the quick fix

@pmeier
Copy link
Contributor Author

pmeier commented Dec 5, 2022

macOS CI seems stuck, but since everything else is green, I think we can move forward here.

@pmeier pmeier marked this pull request as ready for review December 5, 2022 22:36
@pmeier pmeier merged commit 842e178 into pytorch:main Dec 5, 2022
@pmeier pmeier deleted the imagenet-mock-data branch December 5, 2022 22:37
@github-actions
Copy link

github-actions bot commented Dec 5, 2022

Hey @pmeier!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

facebook-github-bot pushed a commit that referenced this pull request Dec 12, 2022
Summary:
* fix prototype dataset imagenet mock data

* [REVERT ME] temporarily add prototype datasets test to CI

* Revert "[REVERT ME] temporarily add prototype datasets test to CI"

This reverts commit 7cafb5f.

Reviewed By: datumbox

Differential Revision: D41836895

fbshipit-source-id: bc6ae8b90f0220ea321f33d3231bb0a150c23e30
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