-
Notifications
You must be signed in to change notification settings - Fork 332
Compatibility update for PyTorch API v1.0 #7
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
Conversation
…input 2. changed parameter name "async" to "non_blocking". Start from Python 3.5, "async" has become a keyword and torch has changed the parameter name.
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
…seem to be a list. Signed-off-by: Bill Tao <billtao@pku.edu.cn>
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
Repaired deprecated call to APIs, the program now exhibits no warning except malformed input data. Signed-off-by: Bill Tao <billtao@pku.edu.cn>
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 for the pull request and it looks great! Could you resolve the following comments before merging?
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
One more comment: could you update the README file to show that the code currently works for the latest version of the PyTorch? |
You mean to update the dependencies section, to set the pytorch version to the latest? |
Correct. Can you remove all the sentences that indicates the code only works up to v0.4? |
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
Updated. |
Excellent! Thanks for all the updates. |
No description provided.