-
Notifications
You must be signed in to change notification settings - Fork 31.1k
Cleanup workflow #42023
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
Cleanup workflow #42023
Conversation
|
This comment contains models: $(tail -n 1 output.txt) |
CI Results |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
CI Results✅ No failing test specific to this PR 🎉 ! |
|
This comment contains models: ["models/vit"] |
CI ResultsModel CI Report❌ Failed tests
|
435c01e to
bf7679f
Compare
|
This comment contains models: ["models/vit"] |
CI ResultsModel CI Report❌ Failed tests
|
|
[For maintainers] Suggested jobs to run (before merge) run-slow: vit |
|
This comment contains models: ["models/vit"] |
CI Results✅ No failing test specific to this PR 🎉 ! |
1 similar comment
CI Results✅ No failing test specific to this PR 🎉 ! |
6ea4afc to
6a78b31
Compare
paulinebm
left a comment
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.
LGTM
| CI_TEST_JOB: ${{ inputs.job }} | ||
| SETUP_STATUS: ${{ inputs.setup_status }} | ||
| REPORT_REPO_ID: ${{ inputs.report_repo_id }} | ||
| quantization_matrix: ${{ inputs.quantization_matrix }} |
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.
not in capital ?
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.
yeah, it's fine. I don't try to use capital for each new environment variables added in this PR, a bit too much work.
fix Co-authored-by: ydshieh <[email protected]>
* part 1 * part 2 * part 3 * part 4 * part 5 * fix 1 * check 1 * part 6 * part 7 * part 8 * part 9 * part 10: rename file * OK: new_model_pr_merged_notification.yml * part 11 * fix 2 * revert check * fix * fix * fix --------- Co-authored-by: ydshieh <[email protected]>
…42049) fix Co-authored-by: ydshieh <[email protected]>
What does this PR do?
Great!