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

sync torchx .pyre_configuration.internal with external config and upg… #986

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

stroxler
Copy link
Contributor

@stroxler stroxler commented Dec 9, 2024

There are some issues with Pyre versions in github CI, and at the moment we aren't getting type errors.

Making a PR to
(a) upgrade to the 2024-11-25 release
(b) suppress errors

I'm making a fresh PR because the original one doesn't seem to trigger github CI (maybe because it was exported from a diff)

@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 Dec 9, 2024
@facebook-github-bot
Copy link
Contributor

@stroxler has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Contributor

@jesszzzz jesszzzz left a comment

Choose a reason for hiding this comment

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

Amazing, thank you!!

stroxler added a commit to stroxler/torchx that referenced this pull request Dec 10, 2024
pytorch#986)

Summary:
There are some issues with Pyre versions in github CI, and at the moment we aren't getting type errors.

Making a PR to
(a) bump to Ubuntu 24.04, because the root cause is a glibc issue
(b) upgrade to the  2024-11-25 release
(c) suppress errors

I'm making a fresh PR because the original one doesn't seem to trigger github CI (maybe because it was exported from a diff)


Test Plan: Ran github CI, the problem is fixed.

Reviewed By: jesszzzz

Differential Revision: D66994745

Pulled By: stroxler
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D66994745

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D66994745

stroxler added a commit to stroxler/torchx that referenced this pull request Dec 10, 2024
pytorch#986)

Summary:
There are some issues with Pyre versions in github CI, and at the moment we aren't getting type errors.

Making a PR to
(a) bump to Ubuntu 24.04, because the root cause is a glibc issue
(b) upgrade to the  2024-11-25 release
(c) suppress errors

I'm making a fresh PR because the original one doesn't seem to trigger github CI (maybe because it was exported from a diff)


Test Plan: Ran github CI, the problem is fixed.

Reviewed By: jesszzzz

Differential Revision: D66994745

Pulled By: stroxler
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D66994745

pytorch#986)

Summary:
There are some issues with Pyre versions in github CI, and at the moment we aren't getting type errors.

Making a PR to
(a) bump to Ubuntu 24.04, because the root cause is a glibc issue
(b) upgrade to the  2024-11-25 release
(c) suppress errors

I'm making a fresh PR because the original one doesn't seem to trigger github CI (maybe because it was exported from a diff)


Test Plan: Ran github CI, the problem is fixed.

Reviewed By: jesszzzz

Differential Revision: D66994745

Pulled By: stroxler
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D66994745

@stroxler stroxler merged commit c8a633a into pytorch:main Dec 10, 2024
17 of 24 checks passed
@stroxler stroxler deleted the try-fix-torchx branch December 10, 2024 18:47
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. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants