Skip to content

[quant][fx] Move the remaining fixed qparam ops to backend_config_dict #75314

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.

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

Closed
wants to merge 2 commits into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Apr 6, 2022

Summary:
this is a refactor to use backend_config_dict for operators with fixed quantization parameters
api is not final yet, we'll update the api after we moved everything to backend_config_dict

Test Plan:
python test/test_quantization.py TestQuantizeFx
python test/test_quantization.py TestQuantizeFxOps
python test/test_quantization.py TestFXNumericSuiteCoreAPIs

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Apr 6, 2022

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 0eaef19 (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages

See GitHub Actions build pull / win-vs2019-cuda11.3-py3 / test (default, 1, 2, windows.8xlarge.nvidia.gpu) (1/1)

Step: "Setup Windows" (full log | diagnosis details | 🔁 rerun)

2022-04-07T02:53:40.0436472Z C:\actions-runner\...dfa867c7280.sh: line 2: python3: command not found
2022-04-07T02:53:39.9791738Z   BRANCH: 
2022-04-07T02:53:39.9792261Z   JOB_BASE_NAME: win-vs2019-cuda11.3-py3-test
2022-04-07T02:53:39.9792877Z   BUILD_ENVIRONMENT: win-vs2019-cuda11.3-py3
2022-04-07T02:53:39.9793382Z   PR_NUMBER: 75314
2022-04-07T02:53:39.9793881Z   SHA1: 0eaef1926c77b9111df33bde1a214837360bbb12
2022-04-07T02:53:39.9794362Z   TAG: 
2022-04-07T02:53:39.9794735Z   WORKFLOW_ID: 2105471346
2022-04-07T02:53:39.9795592Z   GITHUB_TOKEN: ***
2022-04-07T02:53:39.9796018Z ##[endgroup]
2022-04-07T02:53:40.0339725Z + python3 -m pip install -r requirements.txt
2022-04-07T02:53:40.0436472Z C:\actions-runner\_work\_temp\4944e4ba-41ec-4cb0-9180-cdfa867c7280.sh: line 2: python3: command not found
2022-04-07T02:53:40.0465363Z ##[error]Process completed with exit code 127.
2022-04-07T02:53:40.0580068Z Prepare all required actions
2022-04-07T02:53:40.0617199Z ##[group]Run ./.github/actions/teardown-win
2022-04-07T02:53:40.0617452Z with:
2022-04-07T02:53:40.0617620Z env:
2022-04-07T02:53:40.0617822Z   IN_CI: 1
2022-04-07T02:53:40.0618018Z   IS_GHA: 1
2022-04-07T02:53:40.0618220Z   GIT_DEFAULT_BRANCH: master
2022-04-07T02:53:40.0618446Z ##[endgroup]
2022-04-07T02:53:40.0723800Z ##[group]Run .github\scripts\wait_for_ssh_to_drain.ps1

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.

@jerryzh168
Copy link
Contributor Author

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

@jerryzh168 jerryzh168 added the topic: not user facing topic category label Apr 6, 2022
facebook-github-bot pushed a commit that referenced this pull request Apr 6, 2022
#75314)

Summary:
Pull Request resolved: #75314

this is a refactor to use backend_config_dict for operators with fixed quantization parameters
api is not final yet, we'll update the api after we moved everything to backend_config_dict

Test Plan:
python test/test_quantization.py TestQuantizeFx
python test/test_quantization.py TestQuantizeFxOps
python test/test_quantization.py TestFXNumericSuiteCoreAPIs

Imported from OSS

Reviewed By: vkuzo

Differential Revision: D35423790

fbshipit-source-id: a69ce19340e2e3c996f1435b887ba122de85f22f
…_config_dict"

Summary:
this is a refactor to use backend_config_dict for operators with fixed quantization parameters
api is not final yet, we'll update the api after we moved everything to backend_config_dict

Test Plan:
python test/test_quantization.py TestQuantizeFx
python test/test_quantization.py TestQuantizeFxOps
python test/test_quantization.py TestFXNumericSuiteCoreAPIs

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D35423790](https://our.internmc.facebook.com/intern/diff/D35423790)

[ghstack-poisoned]
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.

3 participants