Skip to content
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

[chore] update to torch v1.8.0 #508

Merged
merged 2 commits into from
Mar 12, 2021
Merged

[chore] update to torch v1.8.0 #508

merged 2 commits into from
Mar 12, 2021

Conversation

msbaines
Copy link
Contributor

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
  • Did you read the contributor guideline?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes # (issue).

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@msbaines msbaines requested a review from min-xu-ai March 10, 2021 22:25
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 10, 2021
Copy link
Contributor

@min-xu-ai min-xu-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!!!

I vaguely remember there is a reference of torch version in docs or setup.py or something like that. Do a quick check there too?

@msbaines msbaines force-pushed the torch18 branch 12 times, most recently from 3e6752c to abe2628 Compare March 11, 2021 19:01
@@ -146,12 +146,17 @@ def dist_init(rank: int, world_size: int, filename: str, filename_rpc: str = "")

torch.distributed.init_process_group(backend=backend, rank=rank, world_size=world_size, init_method=url)

tp_options = {"init_method": url_rpc}
# Workaround for bug in torch v1.8.0. Should be fixed in v1.8.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh wow, thanks for catching that

Copy link
Contributor

@blefaudeux blefaudeux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, many thanks @msbaines for tackling that

Copy link
Contributor

@anj-s anj-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this change @msbaines!

@msbaines msbaines force-pushed the torch18 branch 7 times, most recently from 9f9e800 to 3403be1 Compare March 12, 2021 02:11
@msbaines msbaines force-pushed the torch18 branch 6 times, most recently from 8c12480 to 301756c Compare March 12, 2021 03:58
@msbaines msbaines merged commit c79bbd0 into master Mar 12, 2021
@msbaines msbaines deleted the torch18 branch March 12, 2021 05:17
@blefaudeux blefaudeux linked an issue Mar 12, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update to pytorch 1.8
6 participants