Skip to content

Try to fix Inductor CI periodic tests#160932

Closed
yf225 wants to merge 2 commits intomainfrom
fix_inductor_peridic_tests
Closed

Try to fix Inductor CI periodic tests#160932
yf225 wants to merge 2 commits intomainfrom
fix_inductor_peridic_tests

Conversation

@yf225
Copy link
Copy Markdown
Contributor

@yf225 yf225 commented Aug 19, 2025

  • hf_Reformer: this one starts failing due to increased graph breaks due to transformers pin bump (Bump transformers pin #159291). We can likely just bump the expected graph break count.
  • dla102: this one starts timing out on 8/13 Wed between commit 6e8865f and ee1b041. But based on the PT2 dashboard, this model actually doesn't have compile time or runtime regression. Will try to bump up the timeout and see if it can work.
  • hf_BigBird: this one has its accuracy status improved since today. Will update hf_BigBird accuracy status.

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang @amjames @Lucaskabela

@yf225 yf225 requested a review from zou3519 August 19, 2025 01:00
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Aug 19, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 3 Unrelated Failures

As of commit 2e6ac53 with merge base bbc7c03 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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

@yf225 yf225 force-pushed the fix_inductor_peridic_tests branch from ac9a65e to d842f90 Compare August 19, 2025 01:38
Copy link
Copy Markdown
Contributor

@huydhn huydhn left a comment

Choose a reason for hiding this comment

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

Thank you for the fix! I missed slapping ciflow/inductor-periodic on #159291 :(

@yf225 yf225 force-pushed the fix_inductor_peridic_tests branch from d842f90 to 7cdb19b Compare August 19, 2025 04:16
@yf225 yf225 changed the title Try to fix Inductor CI periodic tests for hf_Reformer / hf_BigBird / dla102 Try to fix Inductor CI periodic tests for hf_Reformer / hf_BigBird / dla102 / DistillGPT2 / basic_gnn_sage Aug 19, 2025
@yf225 yf225 changed the title Try to fix Inductor CI periodic tests for hf_Reformer / hf_BigBird / dla102 / DistillGPT2 / basic_gnn_sage Try to fix Inductor CI periodic tests for hf_Reformer / hf_BigBird / dla102 Aug 19, 2025
@yf225 yf225 force-pushed the fix_inductor_peridic_tests branch from f7d86fe to f3172ba Compare August 19, 2025 20:54
@yf225 yf225 changed the title Try to fix Inductor CI periodic tests for hf_Reformer / hf_BigBird / dla102 Try to fix Inductor CI periodic tests Aug 20, 2025
@yf225
Copy link
Copy Markdown
Contributor Author

yf225 commented Aug 20, 2025

@pytorchbot merge -f "there are still other known failures on Inductor CI periodic tests, we will fix in another PR"

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
- hf_Reformer: this one starts failing due to increased graph breaks due to transformers pin bump (pytorch#159291). We can likely just bump the expected graph break count.
- dla102: this one starts timing out on 8/13 Wed between commit 6e8865f and ee1b041. But based on the PT2 dashboard, this model actually doesn't have compile time or runtime regression. Will try to bump up the timeout and see if it can work.
- hf_BigBird: this one has its accuracy status improved since today. Will update hf_BigBird accuracy status.

Pull Request resolved: pytorch#160932
Approved by: https://github.com/zou3519, https://github.com/huydhn, https://github.com/malfet
@github-actions github-actions Bot deleted the fix_inductor_peridic_tests branch September 20, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants