Skip to content
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

[quant][graphmode][fx] Support non_traceable_module/module_class #46298

Closed
wants to merge 5 commits into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Oct 14, 2020

Stack from ghstack:

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D24294210

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

💊 CI failures summary and remediations

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


Commit a881f90 was recently pushed. Waiting for builds...


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 1 times.

…_class"

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 14, 2020
Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 8b99f426c3bd78629d8eec7839c7361802e36aff
Pull Request resolved: #46298
@dr-ci
Copy link

dr-ci bot commented Oct 14, 2020

💊 CI failures summary and remediations

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


None of the CI failures appear to be your fault 💚



🚧 5 ongoing upstream failures:

These were probably caused by upstream breakages that are not fixed yet:


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 24 times.

…_class"

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
…_class"

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
Comment on lines +798 to +805
prepare_custom_config_dict = {
"non_traceable_module_name": [
"m1"
],
"non_traceable_module_class": [
NonTraceable2
]
}
Copy link
Contributor

Choose a reason for hiding this comment

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

nice!

…_class"

Summary:
Allow user to specify a list of qualified names for non traceable submodule
or type of the non traceable submodule
See quantize_fx.py for api

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

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

This pull request has been merged in a06b95b.

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in a06b95b.

@facebook-github-bot facebook-github-bot deleted the gh/jerryzh168/465/head branch October 23, 2020 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants