[ao][fx] fixing public v private qconfig_mapping_utils.py

[HDCharles]

Stack from ghstack (oldest at bottom):

• -> [ao][fx] fixing public v private qconfig_mapping_utils.py #88399

Summary: made _check_is_valid_config_dict,
_compare_prepare_convert_qconfig_mappings,
_generate_node_name_to_qconfig,
_is_qconfig_supported_by_dtype_configs,
_maybe_adjust_qconfig_for_module_name_object_type_order,
_update_qconfig_for_fusion private

Test Plan: python test/test_public_bindings.py

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D41015544

cc @jerryzh168 @jianyuh @raghuramank100 @jamesr66a @vkuzo @jgong5 @Xia-Weiwen @leslie-fang-intel

[pytorch-bot]

🔗 Helpful Links

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

• 📄 Preview Python docs built from this PR
• 📄 Preview C++ docs built from this PR
• ❓ Need help or want to give feedback on the CI? Visit the bot commands wiki or our office hours

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

❌ 2 Failures

As of commit a1b9897:

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

• linux-focal-rocm5.3-py3.8 / test (default, 1, 2, linux.rocm.gpu)

BROKEN TRUNK - The following jobs failed but were present on the merge base 283cf71:

• linux-bionic-cuda11.6-py3.10-gcc7 / test (deploy, 1, 1, linux.4xlarge.nvidia.gpu)

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

[HDCharles]

@HDCharles has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[HDCharles]

@HDCharles has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[HDCharles]

@pytorchmergebot merge

[pytorchmergebot]

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

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]

Merge failed

Reason: Command git -C /home/runner/work/pytorch/pytorch cherry-pick -x 700b7dbdba65295968c6b4f9cac22e46686e54fa returned non-zero exit code 1

Auto-merging test/quantization/fx/test_quantize_fx.py
Auto-merging torch/ao/ns/_numeric_suite_fx.py
CONFLICT (content): Merge conflict in torch/ao/ns/_numeric_suite_fx.py
Auto-merging torch/ao/quantization/fx/convert.py
Auto-merging torch/ao/quantization/fx/prepare.py
error: could not apply 700b7dbdba... [ao][fx] fixing public v private qconfig_mapping_utils.py
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

Details for Dev Infra team

Raised by workflow job

[HDCharles]

@pytorchmergebot merge

[pytorchmergebot]

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

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]

Merge failed

Reason: The following mandatory check(s) failed (Rule superuser):

• pull

Dig deeper by viewing the failures on hud

Details for Dev Infra team

Raised by workflow job

[HDCharles]

@pytorchmergebot merge -f

[pytorch-bot]

❌ 🤖 pytorchbot command failed:

@pytorchbot merge: error: argument -f/--force: expected one argument

usage: @pytorchbot merge [-g | -f MESSAGE | -l] [-r [{viable/strict,master}]]

Try @pytorchbot --help for more info.

[HDCharles]

@pytorchmergebot merge -f the linux nvidia test is broken on master and this PR has no functional changes, just name changes

[pytorch-bot]

❌ 🤖 pytorchbot command failed:

@pytorchbot: error: unrecognized arguments: linux nvidia test is broken on master and this PR has no functional changes, just name changes

usage: @pytorchbot [-h] {merge,revert,rebase,label,drci} ...

Try @pytorchbot --help for more info.

[HDCharles]

@pytorchmergebot merge -f "the linux nvidia test is broken on master and this PR has no functional changes, just name changes"

[pytorchmergebot]

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

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]

Merge failed

Reason: Command git -C /home/runner/work/pytorch/pytorch cherry-pick -x 6b8bc3f7e86bb4fa8fd7ce2e1c57e67a1bca9260 returned non-zero exit code 1

Auto-merging test/quantization/fx/test_quantize_fx.py
Auto-merging torch/ao/ns/_numeric_suite_fx.py
CONFLICT (content): Merge conflict in torch/ao/ns/_numeric_suite_fx.py
Auto-merging torch/ao/quantization/fx/convert.py
Auto-merging torch/ao/quantization/fx/prepare.py
error: could not apply 6b8bc3f7e8... [ao][fx] fixing public v private qconfig_mapping_utils.py
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

Details for Dev Infra team

Raised by workflow job

[HDCharles]

@pytorchmergebot merge -f "Broken on trunk/flaky tests, this PR doesn't have any functional changes, just name changes"

[pytorchmergebot]

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

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