Skip to content

[FIX] remove the duplicate key in DEFAULT_STATIC_QUANT_MODULE_MAPPINGS #149043

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[FIX] remove the duplicate key in DEFAULT_STATIC_QUANT_MODULE_MAPPINGS #149043

wants to merge 1 commit into from

Conversation

hackty
Copy link
Contributor

@hackty hackty commented Mar 12, 2025

nn.Dropout appeared at line 81

@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Mar 12, 2025 •
edited
Loading

🔗 Helpful Links

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

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

⏳ No Failures, 42 Pending

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

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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

@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Mar 12, 2025 •
edited
Loading

CLA Signed


The committers listed above are authorized under a signed CLA.

jingsh
jingsh approved these changes Mar 12, 2025
View reviewed changes
Copy link
Contributor

@jingsh jingsh left a comment

Choose a reason for hiding this comment

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

lgtm

@hackty
Copy link
Contributor Author

hackty commented Mar 13, 2025

@pytorchmergebot merge

@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Mar 13, 2025

Pull workflow has not been scheduled for the PR yet. It could be because author doesn't have permissions to run those or skip-checks keywords were added to PR/commits, aborting merge. Please get/give approval for the workflows and/or remove skip ci decorators before next merge attempt. If you think this is a mistake, please contact PyTorch Dev Infra.

@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Mar 13, 2025

You are not authorized to force merges to this repository. Please use the regular @pytorchmergebot merge command instead

@hackty hackty requested a review from jingsh March 13, 2025 03:22
@cyyever
Copy link
Collaborator

cyyever commented Mar 13, 2025

@pytorchmergebot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Mar 13, 2025
@cyyever
Copy link
Collaborator

cyyever commented Mar 13, 2025

@pytorchmergebot merge -f "Safe fix"

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jingsh jingsh Awaiting requested review from jingsh

Assignees

No one assigned

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged open source release notes: AO frontend release notes: quantization release notes category

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@hackty @cyyever @pytorchmergebot @jingsh @pytorchbot