Skip to content

Enable verifier [2/n]#1155

Closed
zhxchen17 wants to merge 1 commit intopytorch:mainfrom
zhxchen17:export-D51014944
Closed

Enable verifier [2/n]#1155
zhxchen17 wants to merge 1 commit intopytorch:mainfrom
zhxchen17:export-D51014944

Conversation

@zhxchen17
Copy link
Copy Markdown
Contributor

Summary:
X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944

@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 Nov 6, 2023
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Nov 6, 2023

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Unrelated Failure

As of commit fc6e7f9 with merge base b182621 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/pytorch that referenced this pull request Nov 7, 2023
Summary:
X-link: pytorch/executorch#1155


Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Test Plan: CI

Differential Revision: D51014944
zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Reviewed By: angelayi

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/pytorch that referenced this pull request Nov 7, 2023
Summary:
X-link: pytorch/executorch#1155


Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Test Plan: CI

Reviewed By: angelayi

Differential Revision: D51014944
zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Reviewed By: angelayi

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Reviewed By: angelayi

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

zhxchen17 added a commit to zhxchen17/executorch-1 that referenced this pull request Nov 7, 2023
Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Reviewed By: angelayi

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

Summary:

X-link: pytorch/pytorch#113075

Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Reviewed By: angelayi

Differential Revision: D51014944
zhxchen17 added a commit to zhxchen17/pytorch that referenced this pull request Nov 8, 2023
Summary:
X-link: pytorch/executorch#1155


Turn on verifier check for exportec program ctor. Note that this effectively detect a large surface of spec violations, so we also spend some time fixing them one by one in this diff.

Test Plan: CI

Reviewed By: angelayi

Differential Revision: D51014944
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 6765b94.

@zhxchen17
Copy link
Copy Markdown
Contributor Author

@mergennachin @larryliu0820 this PR requires a version bump of pytorch. I've tested some model run on my side and it seems to work. Let me know if I can help with anything further.

mergennachin pushed a commit to mergennachin/executorch-1 that referenced this pull request Nov 9, 2023
Summary: Due to pytorch/pytorch#113075 and pytorch#1155

Differential Revision: D51159878
facebook-github-bot pushed a commit that referenced this pull request Nov 9, 2023
Summary:
Need to update due to #1155

Pull Request resolved: #1180

Reviewed By: shoumikhin, angelayi

Differential Revision: D51160638

Pulled By: mergennachin

fbshipit-source-id: 1f165e4555f79d296d7aa449ced3f78afd73c60a
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.

2 participants