Skip to content

Conversation

pmeier
Copy link
Collaborator

@pmeier pmeier commented Nov 26, 2021

The failure is currently not visible in CI, since we do not test with loaded weights. See this CI run from #4772 for the failure.

@pmeier pmeier added bug module: models.quantization Issues related to the quantizable/quantized models labels Nov 26, 2021
@pmeier pmeier requested a review from datumbox November 26, 2021 11:34
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Nov 26, 2021

💊 CI failures summary and remediations

As of commit 3abfbd2 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@datumbox datumbox merged commit 58016b0 into pytorch:main Nov 26, 2021
@pmeier pmeier deleted the fix-mobilenet-quantization branch November 26, 2021 13:02
facebook-github-bot pushed a commit that referenced this pull request Nov 30, 2021
Reviewed By: NicolasHug

Differential Revision: D32694307

fbshipit-source-id: 1f3f35072192259a05280e01721baa26ad7bb6c9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug ciflow/default cla signed module: models.quantization Issues related to the quantizable/quantized models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants