Skip to content

Conversation

@jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Aug 19, 2020

Stack from ghstack:

Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D23221734

Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Aug 19, 2020

💊 CI failures summary and remediations

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


None of the CI failures appear to be your fault 💚



🚧 1 fixed upstream failure:

These were probably caused by upstream breakages that were already fixed.

Please rebase on the viable/strict branch (expand for instructions)

Since your merge base is older than viable/strict, run these commands:

git fetch https://github.com/pytorch/pytorch viable/strict
git rebase FETCH_HEAD

Check out the recency history of this "viable master" tracking branch.


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 37 times.

Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
Copy link
Contributor

@vkuzo vkuzo left a comment

Choose a reason for hiding this comment

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

lg. Optional suggeston for future PRs: would it make sense to have a clean split between:

  • eager mode
  • fx
  • shared between eager mode and fx

i.e. the shared code would be in a separate directory or file, to make it clear what's shared and what's not, with ideally the shared part of the codebase growing over time, and eager-only shrinking?

Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
@jerryzh168
Copy link
Contributor Author

lg. Optional suggeston for future PRs: would it make sense to have a clean split between:

  • eager mode
  • fx
  • shared between eager mode and fx

i.e. the shared code would be in a separate directory or file, to make it clear what's shared and what's not, with ideally the shared part of the codebase growing over time, and eager-only shrinking?

If you just talk about things under torch/quantization then there is not much that is eager mode only, probably just https://github.com/pytorch/pytorch/blob/master/torch/quantization/quantize.py and https://github.com/pytorch/pytorch/blob/master/torch/quantization/stubs.py I think. There is also https://github.com/pytorch/pytorch/blob/master/torch/nn/quantized/modules/functional_modules.py which is only used in eager mode.

we can certainly put them in the same folder, but not sure if it's worth it or not.

Summary:
We need to use this in graph mode quantization on fx

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

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

This pull request has been merged in 217ddea.

@facebook-github-bot facebook-github-bot deleted the gh/jerryzh168/395/head branch August 24, 2020 14:15
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.

6 participants