Skip to content

Use GraphBuilder in unit tests for ops removal. #11010

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

Merged
merged 1 commit into from
May 22, 2025

Conversation

eigen-k
Copy link
Contributor

@eigen-k eigen-k commented May 20, 2025

Summary: Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207

@eigen-k eigen-k requested a review from tarun292 as a code owner May 20, 2025 20:02
Copy link

pytorch-bot bot commented May 20, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 065e7b4 with merge base df5e7df (image):
💚 Looks good so far! There are no failures yet. 💚

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

@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 May 20, 2025
@facebook-github-bot
Copy link
Contributor

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

@eigen-k eigen-k added release notes: none Do not include this in the release notes topic: not user facing labels May 20, 2025
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from ba641bf to d723ebf Compare May 20, 2025 20:07
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from d723ebf to e859e26 Compare May 20, 2025 20:12
@@ -247,6 +244,7 @@ def forward(self, x):
count_node(graph_module, exir_ops.edge.aten.view_copy.default), 2
)

# Already covered
Copy link
Contributor

Choose a reason for hiding this comment

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

are these commented needed?

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from e859e26 to cd9f9ca Compare May 20, 2025 21:40
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from cd9f9ca to c5f3d09 Compare May 20, 2025 21:45
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from c5f3d09 to 8520388 Compare May 20, 2025 23:26
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 8520388 to f05300a Compare May 20, 2025 23:30
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from f05300a to 952039b Compare May 20, 2025 23:33
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 952039b to 2c7a5a7 Compare May 20, 2025 23:34
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 2c7a5a7 to 285d2f7 Compare May 20, 2025 23:36
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 285d2f7 to 8717e33 Compare May 20, 2025 23:40
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 8717e33 to 6884171 Compare May 20, 2025 23:43
@facebook-github-bot
Copy link
Contributor

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

eigen-k added a commit to eigen-k/executorch that referenced this pull request May 20, 2025
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 6884171 to 99d3af5 Compare May 20, 2025 23:47
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 21, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 21, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 21, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 22, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@eigen-k eigen-k force-pushed the export-D75026207 branch from 99d3af5 to c9f81df Compare May 22, 2025 16:35
@facebook-github-bot
Copy link
Contributor

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

@eigen-k eigen-k force-pushed the export-D75026207 branch from c9f81df to a920c23 Compare May 22, 2025 21:40
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 22, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 22, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
eigen-k added a commit to eigen-k/executorch that referenced this pull request May 22, 2025
Summary:

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
Summary:
Pull Request resolved: pytorch#11010

Use GraphBuilder in unit tests for ops removal.

Reviewed By: zonglinpeng

Differential Revision: D75026207
@facebook-github-bot
Copy link
Contributor

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

@eigen-k eigen-k force-pushed the export-D75026207 branch from a920c23 to 065e7b4 Compare May 22, 2025 21:49
@facebook-github-bot facebook-github-bot merged commit 8620702 into pytorch:main May 22, 2025
92 of 93 checks passed
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 release notes: none Do not include this in the release notes topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants