New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document example of Proxy use #50583
Conversation
[ghstack-poisoned]
ghstack-source-id: 4bdbd9781c194f75e707be90cd3dbd2c94779719 Pull Request resolved: #50583
[ghstack-poisoned]
ghstack-source-id: 763f7dd09ebae474e84805be54dc3bd2280aa8db Pull Request resolved: #50583
There was a problem hiding this 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)
[ghstack-poisoned]
ghstack-source-id: 66a8c17645b9ae3e1be91c933691f8614d1b8738 Pull Request resolved: #50583
There was a problem hiding this 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
[ghstack-poisoned]
ghstack-source-id: c25e0d3300ca648892fa3964cc8c486a516df0c2 Pull Request resolved: #50583
Differential Revision: [D26010501](https://our.internmc.facebook.com/intern/diff/D26010501) [ghstack-poisoned]
ghstack-source-id: 278e7e57194e81464785334f08d7db8c44264f1e Pull Request resolved: #50583
Stack from ghstack:
Differential Revision: D26010501