Enable torch tracing by changing assertions in d2go forwards to allow for torch.fx.proxy.Proxy type. #241
Conversation
This file contains 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
facebook-github-bot
added
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
May 11, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/d2go#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Differential Revision: D35518556 fbshipit-source-id: 96f4389e5c6c6efd4790ce9c3f179e0544f3ff05
simonhollis
force-pushed
the
export-D35518556
branch
from
328698d
to
f41476d
Compare
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
May 24, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Differential Revision: D35518556 fbshipit-source-id: 09e516e3c098000dea63ddaf3907648bcae5a1b1
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
force-pushed
the
export-D35518556
branch
from
f41476d
to
9d2a676
Compare
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Jul 19, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Differential Revision: D35518556 fbshipit-source-id: ed234bf6e6b1c306ef9670ffbcb5ae00376e2d03
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Jul 20, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Differential Revision: D35518556 fbshipit-source-id: 7feb77561f140825a66d46de93d5ae67b02288cb
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
force-pushed
the
export-D35518556
branch
from
9d2a676
to
d8c380b
Compare
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
Jul 25, 2022
… for torch.fx.proxy.Proxy type. Summary: Pull Request resolved: facebookresearch#4227 X-link: facebookresearch/d2go#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Differential Revision: D35518556 fbshipit-source-id: 7f1cc384a30d6f79772e68c28133e5420f966f06
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Jul 26, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This diff adds a helper function `is_fx_proxy()`, for checking for `torch.fx.proxy.Proxy` instances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: 539c8c0ccb39c3e2ba953ccda72103d847ad0667
simonhollis
force-pushed
the
export-D35518556
branch
from
d8c380b
to
e66fad7
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Aug 2, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: b5d65165f271722af24e3dd9d33b3e37e4cf0e34
simonhollis
force-pushed
the
export-D35518556
branch
from
e66fad7
to
2dc70f9
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Aug 2, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: 524a3dbcb42736789580459bed2c9323b91a5615
simonhollis
force-pushed
the
export-D35518556
branch
from
2dc70f9
to
bb1d40a
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
wat3rBro
pushed a commit
to wat3rBro/detectron2-1
that referenced
this pull request
Aug 2, 2022
… for torch.fx.proxy.Proxy type. Summary: Pull Request resolved: facebookresearch#4227 X-link: facebookresearch/d2go#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: 82e6f1af479828176baba19bfa19a3272aca25af
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Aug 15, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Differential Revision: D35518556 fbshipit-source-id: 8f5181fccf6cdd29ff2fea0d26385a65a08df76e
simonhollis
force-pushed
the
export-D35518556
branch
from
bb1d40a
to
1d4adac
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
simonhollis
pushed a commit
to simonhollis/d2go
that referenced
this pull request
Aug 16, 2022
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: c866f49a2be8d04bb6ed646ab07004cbdb582938
simonhollis
force-pushed
the
export-D35518556
branch
from
1d4adac
to
3a7e623
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
… for torch.fx.proxy.Proxy type. Summary: X-link: facebookresearch/detectron2#4227 Pull Request resolved: facebookresearch#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: e8dbeb2538c2ce9042e5f1c481b30f48b3c8988f
simonhollis
force-pushed
the
export-D35518556
branch
from
3a7e623
to
cc05a7b
Compare
|
This pull request was exported from Phabricator. Differential Revision: D35518556 |
facebook-github-bot
pushed a commit
to facebookresearch/detectron2
that referenced
this pull request
Aug 18, 2022
… for torch.fx.proxy.Proxy type. Summary: Pull Request resolved: #4227 X-link: facebookresearch/d2go#241 Torch FX tracing propagates a type of `torch.fx.proxy.Proxy` through the graph. Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail. This adds a check for FX tracing in progress and adds a helper function `assert_fx_safe()`, that can be used in place of a standard assertion. This function only applies the assertion if one is not tracing, allowing d2go assertion tests to be compatible with FX tracing. Reviewed By: wat3rBro Differential Revision: D35518556 fbshipit-source-id: a9b5d3d580518ca74948544973ae89f8b9de3282
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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:
Torch FX tracing propagates a type of
torch.fx.proxy.Proxythrough the graph.Existing type assertions in the d2go code base trigger during torch FX tracing, causing tracing to fail.
This diff adds a helper function
is_fx_proxy(), for checking fortorch.fx.proxy.Proxyinstances, then uses this to guard the existing assertions, thus enabling the tracing, as well as maintaining the originally intended functionality.Differential Revision: D35518556