Skip to content

Add Per Tensor Quantization Support to FXIRImporter #55405

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 1 commit into from

Conversation

842974287
Copy link
Contributor

Summary:
Allows FXIRImport to import quantized model.

This diff doesn't include the supports for per-channel weights, linear and conv. Will address them in the next diff.

Test Plan: buck test glow/fb/fx/nnpi_importer:test_importer

Reviewed By: jackm321

Differential Revision: D27313543

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Apr 6, 2021

💊 CI failures summary and remediations

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


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-scanned 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 to the (internal) Dr. CI Users group.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D27313543

@codecov
Copy link

codecov bot commented Apr 6, 2021

Codecov Report

Merging #55405 (01bd538) into master (f1a0b81) will decrease coverage by 0.45%.
The diff coverage is 29.13%.

@@            Coverage Diff             @@
##           master   #55405      +/-   ##
==========================================
- Coverage   77.43%   76.98%   -0.46%     
==========================================
  Files        1892     1895       +3     
  Lines      187597   187719     +122     
==========================================
- Hits       145268   144508     -760     
- Misses      42329    43211     +882     

Summary:
Pull Request resolved: pytorch#55405

Pull Request resolved: pytorch/glow#5516

Allows FXIRImport to import quantized model.

This diff doesn't include the supports for per-channel weights, linear and conv. Will address them in the next diff.

Test Plan: buck test glow/fb/fx/nnpi_importer:test_importer

Reviewed By: jackm321, jfix71

Differential Revision: D27313543

fbshipit-source-id: b730263a46537609aba8007330dc44b0b98d2d42
842974287 added a commit to 842974287/glow that referenced this pull request Apr 8, 2021
Summary:
Pull Request resolved: pytorch/pytorch#55405

Pull Request resolved: pytorch#5516

Allows FXIRImport to import quantized model.

This diff doesn't include the supports for per-channel weights, linear and conv. Will address them in the next diff.

Reviewed By: jackm321, jfix71

Differential Revision: D27313543

fbshipit-source-id: 240776c689f9a3bb8c955ba1a735e5bc0c9873f5
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D27313543

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 02f8276.

facebook-github-bot pushed a commit that referenced this pull request Apr 9, 2021
Summary:
Pull Request resolved: #55405

Pull Request resolved: pytorch/glow#5516

Allows FXIRImport to import quantized model.

This diff doesn't include the supports for per-channel weights, linear and conv. Will address them in the next diff.

Test Plan: buck test glow/fb/fx/nnpi_importer:test_importer

Reviewed By: jackm321, jfix71

Differential Revision: D27313543

fbshipit-source-id: bf5c96ef5f2ff1835c09db981e0ceefaec56dd5b
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
Pull Request resolved: pytorch#55405

Pull Request resolved: pytorch/glow#5516

Allows FXIRImport to import quantized model.

This diff doesn't include the supports for per-channel weights, linear and conv. Will address them in the next diff.

Test Plan: buck test glow/fb/fx/nnpi_importer:test_importer

Reviewed By: jackm321, jfix71

Differential Revision: D27313543

fbshipit-source-id: bf5c96ef5f2ff1835c09db981e0ceefaec56dd5b
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.

2 participants