-
Notifications
You must be signed in to change notification settings - Fork 7.1k
Updated gpu decoder test to use parametrize #5328
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
Updated gpu decoder test to use parametrize #5328
Conversation
💊 CI failures summary and remediationsAs of commit 2fc7a71 (more details on the Dr. CI page):
🕵️ 1 new failure recognized by patternsThe following CI failures do not appear to be due to upstream breakages:
|
Job | Step | Action |
---|---|---|
curl -o conda.sh https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.sh | ||
sh conda.sh -b | ||
source $HOME/miniconda3/bin/activate | ||
conda install -yq conda-build cmake | ||
packaging/build_cmake.sh | ||
🔁 rerun |
This comment was automatically generated by Dr. CI (expand for details).
Please report bugs/suggestions to the (internal) Dr. CI Users group.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @prabhat00155 , LGTM.
Hey @prabhat00155! 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 |
Summary: * Updated gpu decoder test to use parametrize * Remove list comprehension Reviewed By: kazhang Differential Revision: D33927498 fbshipit-source-id: 731b998f24ac12f6f68e73109f27cf07ece388e2
Use parametrize instead of looping over the list. This would help point out the exact scenario in case of failures.