-
Notifications
You must be signed in to change notification settings - Fork 25.7k
Add manual_seed in script #19510
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
Add manual_seed in script #19510
Conversation
| push(stack, a.cpu()); | ||
| return 0; | ||
| }), | ||
| Operator( |
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.
we should mark this op as having side effects. I'm actually kind of surprised the test didn't have it DCE'd out
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.
Is there a way of doing that in the schema ? I added it to hasSideEffects() in ir.cpp
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.
oh I missed that lol. carry on.
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.
@eellison has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Add manual_seed to torch script. Pull Request resolved: pytorch/pytorch#19510 Reviewed By: suo, driazati Differential Revision: D15018823 Pulled By: eellison fbshipit-source-id: d7734a8ad05ba254c0d88abf3fb58c4ce6a4e53b
Summary: Add manual_seed to torch script. Pull Request resolved: pytorch#19510 Reviewed By: suo, driazati Differential Revision: D15018823 Pulled By: eellison fbshipit-source-id: d7734a8ad05ba254c0d88abf3fb58c4ce6a4e53b
Add manual_seed to torch script.