-
Notifications
You must be signed in to change notification settings - Fork 106
[Nova] Passing Torchvision Smoke Test to build workflow #989
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
Conversation
|
@osalpekar is attempting to deploy a commit to the Meta Open Source Team on Vercel. A member of the Team first needs to authorize it. |
| ${CONDA_RUN} conda install -y 'ffmpeg<5' | ||
| if [[ "${{ inputs.package-name }}" = "torchaudio" ]]; then | ||
| ${CONDA_RUN} conda install -y 'ffmpeg<5' |
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.
Shouldn't this be a part of the repository specific verification scripts?
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.
Repo-specific validation scripts are currently just python scripts. I'm going to add shell scripts that install the necessary run-time deps for running the smoke tests soon, this is just a stop-gap.
Pass Smoke Tests to test workflows. Also fix mac conda smoke tests and remove the double-installation of pytorch.
Pass Smoke Tests to test workflows. Also fix mac conda smoke tests and remove the double-installation of pytorch.