Skip to content

Conversation

khabinov
Copy link
Contributor

@khabinov khabinov commented Apr 1, 2021

Summary: We need to make sure dim types are preserved after applying Transpose.

Test Plan:

$ buck build caffe2/caffe2/opt:bound_shape_inference_test && ./buck-out/gen/caffe2/caffe2/opt/bound_shape_inference_test --gtest_filter=*Transpose*

Reviewed By: yinghai

Differential Revision: D27514487

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Apr 1, 2021

💊 CI failures summary and remediations

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


💚 💚 Looks good so far! There are no failures yet. 💚 💚


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: D27514487

Summary:
Pull Request resolved: pytorch#55188

We need to make sure dim types are preserved after applying Transpose.

Test Plan:
```
$ buck build caffe2/caffe2/opt:bound_shape_inference_test && ./buck-out/gen/caffe2/caffe2/opt/bound_shape_inference_test --gtest_filter=*Transpose*
```

Reviewed By: yinghai

Differential Revision: D27514487

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

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

@codecov
Copy link

codecov bot commented Apr 2, 2021

Codecov Report

Merging #55188 (2984837) into master (8822c7e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #55188   +/-   ##
=======================================
  Coverage   77.39%   77.39%           
=======================================
  Files        1897     1897           
  Lines      187160   187160           
=======================================
+ Hits       144848   144850    +2     
+ Misses      42312    42310    -2     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 28531c9.

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