Skip to content

Conversation

csummersea
Copy link
Contributor

Summary: The model exported from online training workflow with int8 quantization contains FCs with 4 inputs. The extra input is the quant_param blob. This diff is to adjust the bound_shape_inferencer to get shape info for the quant_param input.

Test Plan:

buck test caffe2/caffe2/opt:bound_shape_inference_test

Differential Revision: D22543215

Summary: The model exported from online training workflow with int8 quantization contains FCs with 4 inputs. The extra input is the quant_param blob. This diff is to adjust the bound_shape_inferencer to get shape info for the quant_param input.

Test Plan:
```
buck test caffe2/caffe2/opt:bound_shape_inference_test
```

Differential Revision: D22543215

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

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

@dr-ci
Copy link

dr-ci bot commented Jul 15, 2020

💊 CI failures summary and remediations

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


  • 2/2 failures possibly* introduced in this PR
    • 1/2 non-CircleCI failure(s)

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

See CircleCI build pytorch_windows_vs2019_py36_cuda10.1_test1 (1/1)

Step: "Test" (full log | diagnosis details | 🔁 rerun)

[ FAILED ] BoundShapeInference.FC
[ RUN      ] BoundShapeInference.Tile 
[       OK ] BoundShapeInference.Tile (0 ms) 
[ RUN      ] BoundShapeInference.Combo0 
[       OK ] BoundShapeInference.Combo0 (0 ms) 
[----------] 16 tests from BoundShapeInference (319 ms total) 
 
[----------] Global test environment tear-down 
[==========] 16 tests from 1 test case ran. (319 ms total) 
[  PASSED  ] 15 tests. 
[  FAILED  ] 1 test, listed below: 
[  FAILED  ] BoundShapeInference.FC 
 
 1 FAILED TEST 
  YOU HAVE 1 DISABLED TEST 
 
"bound_shape_inference_test" failed with exit code 1 
+ cleanup
+ retcode=1
+ set +x

ci.pytorch.org: 1 failed


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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 86a2bdc.

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.

4 participants