-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[RELAY] [AST] Add virtual_device as a first class field in Relay #9641
Conversation
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.
Thanks Lily, this can't come soon enough as I spent yesterday tracking down more accidentally-lost on_device annotations!
fa8466d
to
0fe8291
Compare
@comaniac Could you take a look at this? It's my implementation of apache/tvm-rfcs#45. Thanks! |
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.
Overall LGTM. Just some docstring errors.
fa8466d
to
1637380
Compare
@comaniac Thanks, updated the comments and docstrings! I think this is good to go in once CI is green. |
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.
LGTM
THanks @electriclilies for the contribution. Thanks @comaniac @mbs-octoml for reviewing. This PR is now merged |
This PR adds virtual_device (SEScope) as a first class field in Relay. This is an implementation of apache/tvm-rfcs#45 please take a look the RFC for the motivation, implementation, future work, etc.
@mbs-octoml @jroesch @mikepapadim