Skip to content

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

Merged
merged 8 commits into from
May 10, 2019

Conversation

TanyaAdams1
Copy link
Contributor

No description provided.

…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>
Copy link
Owner

@txie-93 txie-93 left a 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?

@txie-93 txie-93 mentioned this pull request May 4, 2019
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
Signed-off-by: Bill Tao <billtao@pku.edu.cn>
@txie-93
Copy link
Owner

txie-93 commented May 5, 2019

One more comment: could you update the README file to show that the code currently works for the latest version of the PyTorch?

@TanyaAdams1
Copy link
Contributor Author

You mean to update the dependencies section, to set the pytorch version to the latest?

@txie-93
Copy link
Owner

txie-93 commented May 5, 2019

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>
@TanyaAdams1
Copy link
Contributor Author

Updated.

@txie-93
Copy link
Owner

txie-93 commented May 10, 2019

Excellent! Thanks for all the updates.

@txie-93 txie-93 changed the title Update usage of PyTorch API Compatibility update for PyTorch API v1.0 May 10, 2019
@txie-93 txie-93 merged commit a7a6b81 into txie-93:master May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants