Skip to content

Conversation

@cyyever
Copy link
Collaborator

@cyyever cyyever commented Nov 4, 2025

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 4, 2025

🔗 Helpful Links

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

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

⏳ 46 Pending, 1 Unrelated Failure

As of commit fe9de10 with merge base 4271ffe (image):

UNSTABLE - The following jobs are marked as unstable, possibly due to flakiness on trunk:

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

@cyyever cyyever marked this pull request as draft November 4, 2025 05:32
@pytorch-bot pytorch-bot bot added ciflow/b200 ciflow/h100 ciflow/rocm Trigger "default" config CI on ROCm release notes: quantization release notes category labels Nov 4, 2025
@cyyever cyyever force-pushed the up035_f branch 8 times, most recently from 734cc0d to dd45780 Compare November 4, 2025 09:40
@cyyever cyyever marked this pull request as ready for review November 4, 2025 10:20
@cyyever cyyever requested a review from jerryzh168 as a code owner November 4, 2025 10:20
@cyyever cyyever changed the title Apply UP035 fixes [12/N] Apply ruff UP035 rule Nov 4, 2025
Copy link
Contributor

@Lucaskabela Lucaskabela left a comment

Choose a reason for hiding this comment

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

Thanks for the improvements to typing; LGTM

Signed-off-by: Yuanyuan Chen <cyyever@outlook.com>
@cyyever
Copy link
Collaborator Author

cyyever commented Nov 5, 2025

@pytorchbot merge -i

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 5, 2025
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 0 checks:

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

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: Limited CI on H100 / linux-jammy-cuda12_8-py3_10-gcc11-sm90-FA3-ABI-stable-test / test

Details for Dev Infra team Raised by workflow job

@cyyever
Copy link
Collaborator Author

cyyever commented Nov 5, 2025

@pytorchmergebot merge -f "Unrelated failures"

@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

@donigian
Copy link

donigian commented Nov 5, 2025

@pytorchbot revert -c ghfirst -m "Temporarily need to revert this to continue a revert for #165076. @cyyever Please re-merge after revert of #165076".

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

pytorchmergebot added a commit that referenced this pull request Nov 5, 2025
This reverts commit 5863ba1.

Reverted #166929 on behalf of https://github.com/donigian due to Temporarily need to revert this to continue a revert for #165076. @cyyever Please re-merge after revert of #165076. ([comment](#166929 (comment)))
@pytorchmergebot
Copy link
Collaborator

@cyyever your PR has been successfully reverted.

@donigian
Copy link

donigian commented Nov 5, 2025

@pytorchbot merge -f "Was reverted to unblock diff train can be safely merged again"

@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

@cyyever cyyever deleted the up035_f branch November 5, 2025 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td Do not run TD on this PR ciflow/b200 ciflow/h100 ciflow/inductor ciflow/rocm Trigger "default" config CI on ROCm ciflow/trunk Trigger trunk jobs on your pull request Merged module: dynamo module: inductor oncall: distributed Add this issue/PR to distributed oncall triage queue open source release notes: distributed (torchelastic) release notes: quantization release notes category Reverted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants