Skip to content
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

Continue benchmark other models even if one fails to export #6186

Closed
wants to merge 1 commit into from

Conversation

huydhn
Copy link
Contributor

@huydhn huydhn commented Oct 13, 2024

Copy link

pytorch-bot bot commented Oct 13, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/6186

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 64054b2 with merge base 8957dc8 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 13, 2024
@huydhn huydhn marked this pull request as ready for review October 13, 2024 03:13
@huydhn huydhn requested a review from guangy10 October 13, 2024 03:13
@huydhn huydhn temporarily deployed to upload-benchmark-results October 13, 2024 03:13 — with GitHub Actions Inactive
@facebook-github-bot
Copy link
Contributor

@huydhn has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@huydhn
Copy link
Contributor Author

huydhn commented Oct 13, 2024

The failures on the PR is expected, as I use this to test that other models can continue as usual https://github.com/pytorch/executorch/actions/runs/11310814159

@facebook-github-bot facebook-github-bot force-pushed the handle-model-export-failures branch from 8392c93 to 64054b2 Compare October 14, 2024 18:54
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64295995

@facebook-github-bot
Copy link
Contributor

@huydhn merged this pull request in aa852cc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants