Skip to content

Conversation

mergennachin
Copy link
Contributor

@mergennachin mergennachin commented Mar 26, 2025

Copy link

pytorch-bot bot commented Mar 26, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure, 2 Pending

As of commit a4fa427 with merge base 12240cf (image):

NEW FAILURE - The following job has failed:

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 Mar 26, 2025
@mcr229
Copy link
Contributor

mcr229 commented Mar 26, 2025

looks like need to rebase for the phi4 ci?

Copy link
Contributor

@jackzhxng jackzhxng left a comment

Choose a reason for hiding this comment

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

If the pull for phi4 is also ooming we can just remove it, it didn't use to be tested on pull before the ci refactor anyway

Copy link
Contributor

@mcr229 mcr229 left a comment

Choose a reason for hiding this comment

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

looks good. Just curious, what is the cost of using the larger runner.

@mergennachin mergennachin merged commit f5bbad1 into main Mar 26, 2025
164 of 165 checks passed
@mergennachin mergennachin deleted the upgrade_phi4_mini branch March 26, 2025 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants