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

[reland][quant][eagermode] Move custom_module registration to prepare/convert_custom_config_dict (#46293) #46364

Closed
wants to merge 15 commits into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Oct 14, 2020

Stack from ghstack:

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

Differential Revision: D24322747

…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 14, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: ddcc9e2af4c80def4381d132743103a28fd4c55a
Pull Request resolved: #46364
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Oct 14, 2020

💊 CI failures summary and remediations

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


  • 1/1 failures introduced in this PR---

1 failure not recognized by patterns:

Job Step Action
CircleCI docker-pytorch-linux-xenial-py3-clang5-android-ndk-r19c Check if image should be built 🔁 rerun
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 3 times.

@dr-ci
Copy link

dr-ci bot commented Oct 15, 2020

💊 CI failures summary and remediations

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


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)---
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 67 times.

… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 15, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 9be87cbc6386da97c39355b09097530ece9ff9f4
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 15, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 60ab2608d0340b54fae28f3cf855f004e312fd20
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 15, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 3b00e66fcda9a2fc3e04c1e17318f27284b52f58
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 15, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 7691d8a77ff6555dee1e6cf2fb6315f28a2c00c5
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 16, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 025c580592f27237b747f6db0bdca4c809538b70
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 16, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 6c5fa5c2324b56aea849a42d217bb3c7a968b2ef
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 16, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 2c9a7c2bf860761b08d037038c2b99efe1848677
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 16, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: 679f2e1057b465de6c63e6ff5cb566c694a924fa
Pull Request resolved: #46364
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
… to prepare/convert_custom_config_dict (#46293)"

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

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

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Oct 19, 2020
…/convert_custom_config_dict (#46293)

Summary:

Test Plan: Imported from OSS

Reviewed By: raghuramank100

ghstack-source-id: c05d29d7e4ec494942ead88e6688b2c4135c1ed3
Pull Request resolved: #46364
@codecov
Copy link

codecov bot commented Oct 19, 2020

Codecov Report

Merging #46364 into gh/jerryzh168/472/base will decrease coverage by 0.01%.
The diff coverage is 96.29%.

@@                    Coverage Diff                     @@
##           gh/jerryzh168/472/base   #46364      +/-   ##
==========================================================
- Coverage                   68.39%   68.38%   -0.02%     
==========================================================
  Files                         411      410       -1     
  Lines                       53972    53956      -16     
==========================================================
- Hits                        36914    36896      -18     
- Misses                      17058    17060       +2     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 30d6875.

@facebook-github-bot facebook-github-bot deleted the gh/jerryzh168/472/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