Skip to content

Skip modded_nanogpt model in TorchInductor benchmark#172125

Closed
huydhn wants to merge 1 commit intopytorch:mainfrom
huydhn:skip-modded_nanogpt
Closed

Skip modded_nanogpt model in TorchInductor benchmark#172125
huydhn wants to merge 1 commit intopytorch:mainfrom
huydhn:skip-modded_nanogpt

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Jan 9, 2026

This has never been working correctly #172015 (comment), so let's skip it completely. For more context, this model switches from eager_fail_to_run to fail_accuracy status recently, root cause is not clear, but it's not related to 2.10 RC

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

@pytorch-bot
Copy link

pytorch-bot bot commented Jan 9, 2026

🔗 Helpful Links

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

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

⏳ No Failures, 28 Pending

As of commit 067e3d5 with merge base db4f3aa (image):
💚 Looks good so far! There are no failures yet. 💚

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

@zou3519
Copy link
Contributor

zou3519 commented Jan 9, 2026

can you file an issue saying "this has never worked correctly before" and assign it to whoever added the model originally?

Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm!

@atalman atalman added the topic: not user facing topic category label Jan 9, 2026
@huydhn
Copy link
Contributor Author

huydhn commented Jan 9, 2026

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

This has never been working correctly pytorch#172015 (comment)

Signed-off-by: Huy Do <huydhn@gmail.com>
@pytorchmergebot
Copy link
Collaborator

Successfully rebased skip-modded_nanogpt onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via git checkout skip-modded_nanogpt && git pull --rebase)

@huydhn
Copy link
Contributor Author

huydhn commented Jan 9, 2026

@pytorchbot drci

@huydhn
Copy link
Contributor Author

huydhn commented Jan 9, 2026

@pytorchbot merge -f 'Should be ok to force merge, lint jobs are green'

@huydhn
Copy link
Contributor Author

huydhn commented Jan 9, 2026

can you file an issue saying "this has never worked correctly before" and assign it to whoever added the model originally?

Yes, @atalman mentions the same thing, let me create a separate issue for that

@pytorchmergebot
Copy link
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

@huydhn
Copy link
Contributor Author

huydhn commented Jan 9, 2026

@pytorchbot cherry-pick --onto release/2.10 --fixes 'Skip modded_nanogpton TorchInductor benchmark' -c regression

@pytorchbot
Copy link
Collaborator

Cherry picking #172125

The cherry pick PR is at #172141 and it is linked with issue Skip modded_nanogpton TorchInductor benchmark. The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job

pbielak added a commit to intel/torch-xpu-ops that referenced this pull request Mar 11, 2026
- Re-enable `detectron2_maskrcnn` skip in skip.all.
- Re-enable all `timm_*` model skips in skip.all.
- Keep explicit upstream PR context comments for `modded_nanogpt`
  and `pytorch_CycleGAN_and_pix2pix`.
- Remove stale expected-accuracy rows for skipped models.

Relevant PRs:
[1] pytorch/pytorch#120299
[2] pytorch/pytorch#164816
[3] pytorch/pytorch#172125
[4] pytorch/pytorch#175066
[5] #2306
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