Skip to content

Conversation

seemethere
Copy link
Member

Summary:
They removed the specific function in Python 3.9 so we should just
remake the function here and use our own instead of relying on hidden
functions from the stdlib

Signed-off-by: Eli Uriegas eliuriegas@fb.com

Fixes #48617

Pull Request resolved: #48618

Reviewed By: samestep

Differential Revision: D25230281

Pulled By: seemethere

fbshipit-source-id: 57216af40a4ae4dc8bafcf40d2eb3ba793b9b6e2
(cherry picked from commit 780f2b9)
Signed-off-by: Eli Uriegas eliuriegas@fb.com

Fixes #{issue number}

Summary:
They removed the specific function in Python 3.9 so we should just
remake the function here and use our own instead of relying on hidden
functions from the stdlib

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Fixes pytorch#48617

Pull Request resolved: pytorch#48618

Reviewed By: samestep

Differential Revision: D25230281

Pulled By: seemethere

fbshipit-source-id: 57216af40a4ae4dc8bafcf40d2eb3ba793b9b6e2
(cherry picked from commit 780f2b9)
Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
@seemethere seemethere added this to the 1.7.1 milestone Dec 3, 2020
@seemethere seemethere requested a review from a team December 3, 2020 04:12
@seemethere seemethere added the module: cpp-extensions Related to torch.utils.cpp_extension label Dec 3, 2020
@dr-ci
Copy link

dr-ci bot commented Dec 3, 2020

💊 CI failures summary and remediations

As of commit 55d3ff8 (more details on the Dr. CI page):


  • 2/2 failures possibly* introduced in this PR
    • 1/2 non-CircleCI failure(s)

XLA failure

Job pytorch_xla_linux_bionic_py3_6_clang9_test is failing. Please create an issue with title prefixed by [PT_BREAK] in pytorch/xla and link to to this PR. If you have questions, please reach out to @ailzhang / @dlibenzi / @JackCaoG.


Extra GitHub checks: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 2 times.

@seemethere
Copy link
Member Author

Tests passed on upstream PR going to go ahead and merge

@seemethere seemethere merged commit 1d9b64d into pytorch:release/1.7 Dec 3, 2020
@seemethere seemethere deleted the stop_using_nt_17 branch December 3, 2020 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed module: cpp-extensions Related to torch.utils.cpp_extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants