[nnpi glow unit test] SLS tests shape sweep with hypothesis testing #36833
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This pull request was exported from Phabricator. Differential Revision: D21098346 |
amylittleyang
force-pushed
the
export-D21098346
branch
from
075d317 to
32178af
Compare
|
This pull request was exported from Phabricator. Differential Revision: D21098346 |
Summary: This will increase the test coverage. More tests to enable: ``` test_fc_nnpi_fp16.py test_op_nnpi_fp16.py test_batchmatmul_nnpi_fp16.py ``` Pull Request resolved: pytorch#36717 Differential Revision: D21061269 Pulled By: yinghai fbshipit-source-id: 2761cf3d0ab075b809f5759017a0560062fb2d9c
Differential Revision: D21084876 fbshipit-source-id: d6a27ad79bd1ac10dd454db339927787cbbb99b9
Differential Revision: D21085221 fbshipit-source-id: 7abb17de6836fec073122947bc6067eefeb046cd
…ytorch#36833) Summary: Pull Request resolved: pytorch#36833 Add hypothesis testing sweep for one test in each SLS test suite for different precisions. Sweep random seed, embedding shape, batch_size, weight with hypothesis testing. Refactor sls tests into proper file with precision labeled in filename. Test Plan: FB intern: buck test mode/dev //caffe2/caffe2/contrib/fakelowp/test:test_sls_8bit_nnpi_fp32nnpi Will test OSS after exporting PR. Differential Revision: D21098346 fbshipit-source-id: caaefd63a14348a9b0a7795d5ab86ebc4d63ed2c
|
This pull request was exported from Phabricator. Differential Revision: D21098346 |
amylittleyang
force-pushed
the
export-D21098346
branch
from
32178af to
584b4a8
Compare
💊 Build failures summary and remediationsAs of commit 584b4a8 (more details on the Dr. CI page):
ci.pytorch.org: 1 failedThis 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. This comment has been revised 1 time. |
|
This pull request has been merged in b0b9e70. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
Add hypothesis testing sweep for one test in each SLS test suite for different precisions.
Sweep random seed, embedding shape, batch_size, weight with hypothesis testing.
Refactor sls tests into proper file with precision labeled in filename.
Test Plan:
buck test mode/dev //caffe2/caffe2/contrib/fakelowp/test:test_sls_8bit_nnpi_fp32nnpi
Will test OSS after exporting PR.
Differential Revision: D21098346