Skip to content

Document example of Proxy use#50583

Closed
ansley wants to merge 5 commits intogh/ansleyadelaide/33/basefrom
gh/ansleyadelaide/33/head
Closed

Document example of Proxy use#50583
ansley wants to merge 5 commits intogh/ansleyadelaide/33/basefrom
gh/ansleyadelaide/33/head

Conversation

@ansley
Copy link
Copy Markdown

@ansley ansley commented Jan 15, 2021

Stack from ghstack:

Differential Revision: D26010501

[ghstack-poisoned]
ansley pushed a commit that referenced this pull request Jan 15, 2021
ghstack-source-id: 4bdbd97
Pull Request resolved: #50583
ansley pushed a commit that referenced this pull request Jan 15, 2021
ghstack-source-id: 763f7dd
Pull Request resolved: #50583
@ansley ansley requested a review from jamesr66a January 15, 2021 18:58
Copy link
Copy Markdown
Collaborator

@jamesr66a jamesr66a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, though I think this emphasizes a "boundary case" I guess I would call it, where you're tracing an entire function. It might be good to show also show using Proxy's to inline a given function into another function, or to show using Proxies within some metaprogramming code to emit operations (i.e. not just tracing through straightline code)

Comment thread torch/fx/examples/create_graph_using_proxy.py Outdated
ansley pushed a commit that referenced this pull request Jan 21, 2021
ghstack-source-id: 66a8c17
Pull Request resolved: #50583
@ansley ansley requested a review from jamesr66a January 22, 2021 00:08
Copy link
Copy Markdown
Collaborator

@jamesr66a jamesr66a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just some small stuff then we should be good to go

Comment thread torch/fx/examples/inline_function.py Outdated
Comment thread torch/fx/examples/inline_function.py Outdated
Comment thread torch/fx/examples/inline_function.py Outdated
Comment thread torch/fx/examples/wrap_output_dynamically.py Outdated
ansley pushed a commit that referenced this pull request Jan 22, 2021
ghstack-source-id: c25e0d3
Pull Request resolved: #50583
ansley pushed a commit that referenced this pull request Jan 22, 2021
ghstack-source-id: 278e7e5
Pull Request resolved: #50583
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@ansley merged this pull request in d5dc65a.

@facebook-github-bot facebook-github-bot deleted the gh/ansleyadelaide/33/head branch January 26, 2021 15:21
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#50583

Test Plan: Imported from OSS

Reviewed By: pbelevich

Differential Revision: D26010501

Pulled By: ansley

fbshipit-source-id: 947121af7e57c16c96f849fbbb3fa83e97d003b2
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.

3 participants