Skip to content

Improve static typing in tracer. #3973

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

Closed
wants to merge 1 commit into from

Conversation

zhxchen17
Copy link
Contributor

Summary:
X-link: pytorch/pytorch#128552

as title.

Reviewed By: angelayi

Differential Revision: D58485487

Summary:
X-link: pytorch/pytorch#128552

as title.

Reviewed By: angelayi

Differential Revision: D58485487
Copy link

pytorch-bot bot commented Jun 13, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3973

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure

As of commit be45726 with merge base c67ce86 (image):

NEW FAILURE - The following job has failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Jun 13, 2024
@facebook-github-bot
Copy link
Contributor

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

zhxchen17 added a commit to zhxchen17/pytorch that referenced this pull request Jun 13, 2024
Summary:
X-link: pytorch/executorch#3973

Pull Request resolved: pytorch#128552

as title.

Test Plan: CI

Reviewed By: angelayi

Differential Revision: D58485487
Copy link
Contributor

@JacobSzwejbka JacobSzwejbka left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@digantdesai digantdesai left a comment

Choose a reason for hiding this comment

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

arm test failure doesn't seem related.

@digantdesai
Copy link
Contributor

cc @freddan80 - the arm CI failure here seems like flaky test. Can we look into it?

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 4b6331d.

kirklandsign added a commit that referenced this pull request Jul 25, 2024
This reverts commit 4b6331d.

PT pin not updated to this
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 Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants