Ban or change behavior of TensorVariable.size #120568
Labels
dynamo-must-fix
These bugs affect TorchDynamo reliability.
module: dynamic shapes
module: dynamo
oncall: pt2
small
We think this is a small issue to fix. Consider knocking off high priority small issues
triaged
This issue has been looked at a team member, and triaged and prioritized into an appropriate module
馃悰 Describe the bug
Recent bug #120482 was due to someone directly accessing TensorVariable.size; fixed in #120567. But this field is None when the tensor has dynamic shapes. This is a dynamic shapes footgun. Either ban this, or change this to query size off of the fake tensor (and be willing to return SymInts).
Versions
main
cc @msaroufim @bdhirsh @anijain2305 @zou3519 @chauhang @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @aakhundov
The text was updated successfully, but these errors were encountered: