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.

Sign up for GitHub

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

Jump to bottom

[quant] Use PlaceholderObserver as default dynamic quant observer #45343

Closed
wants to merge 16 commits into from
Closed

[quant] Use PlaceholderObserver as default dynamic quant observer #45343

wants to merge 16 commits into from

Commits on Sep 25, 2020

  1. [quant] Use PlaceholderObserver as default dynamic quant observer 

    Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    7acbe20 View commit details
    Browse the repository at this point in the history

  2. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    18de52a View commit details
    Browse the repository at this point in the history

  3. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    e1a0d5f View commit details
    Browse the repository at this point in the history

  4. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    5571ef0 View commit details
    Browse the repository at this point in the history

  5. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 25, 2020
    Configuration menu
    Copy the full SHA
    bb36d3b View commit details
    Browse the repository at this point in the history

Commits on Sep 26, 2020

  1. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 26, 2020
    Configuration menu
    Copy the full SHA
    c2d65de View commit details
    Browse the repository at this point in the history

Commits on Sep 28, 2020

  1. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 28, 2020
    Configuration menu
    Copy the full SHA
    bf3e163 View commit details
    Browse the repository at this point in the history

Commits on Sep 29, 2020

  1. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    016aa53 View commit details
    Browse the repository at this point in the history

  2. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    073a222 View commit details
    Browse the repository at this point in the history

  3. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    4a720b9 View commit details
    Browse the repository at this point in the history

  4. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    34b9a1a View commit details
    Browse the repository at this point in the history

  5. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    4153e46 View commit details
    Browse the repository at this point in the history

  6. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 29, 2020
    Configuration menu
    Copy the full SHA
    dc9ae2f View commit details
    Browse the repository at this point in the history

Commits on Sep 30, 2020

  1. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 30, 2020
    Configuration menu
    Copy the full SHA
    9df4fd9 View commit details
    Browse the repository at this point in the history

  2. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 30, 2020
    Configuration menu
    Copy the full SHA
    3981258 View commit details
    Browse the repository at this point in the history

  3. Update on "[quant] Use PlaceholderObserver as default dynamic quant o… 

    …bserver"

Summary:
Current default dynamic quant observer is not correct since we don't accumulate
min/max and we don't need to calculate qparams.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

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

[ghstack-poisoned]
    @jerryzh168
    jerryzh168 committed Sep 30, 2020
    Configuration menu
    Copy the full SHA
    6e9abe7 View commit details
    Browse the repository at this point in the history