KEMBAR78
Add output_node util function to fx.Graph by SherlockNoMad · Pull Request #139770 · pytorch/pytorch · GitHub
Skip to content

Add output_node util function to fx.Graph #139770

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Add output_node util function to fx.Graph #139770

wants to merge 1 commit into from

Conversation

@SherlockNoMad
Copy link
Contributor

@SherlockNoMad SherlockNoMad commented Nov 5, 2024 •
edited by pytorch-bot bot
Loading

Summary: A util function for access output node for FX graph

Test Plan: OSS CI

Differential Revision: D65486457

cc @ezyang @EikanWang @jgong5 @wenzhe-nrv

@pytorch-bot pytorch-bot bot added the release notes: fx release notes category label Nov 5, 2024
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 5, 2024 •
edited
Loading

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit be07e49 with merge base c4a323e (image):

NEW FAILURE - The following job has failed:

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

@facebook-github-bot
Copy link
Contributor

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

SherlockNoMad added a commit to SherlockNoMad/pytorch that referenced this pull request Nov 5, 2024
@SherlockNoMad @facebook-github-bot
Add output_node util function to fx.Graph (pytorch#139770)
193379c 
Summary:

A util function for access output node for FX graph

Test Plan: OSS CI

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

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

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 5, 2024
@ezyang ezyang requested a review from Chillee November 5, 2024 23:56
SherlockNoMad added a commit to SherlockNoMad/pytorch that referenced this pull request Nov 6, 2024
@SherlockNoMad @facebook-github-bot
Add output_node util function to fx.Graph (pytorch#139770)
6032a74 
Summary:

A util function for access output node for FX graph

Test Plan: OSS CI

Reviewed By: ezyang

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

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

@facebook-github-bot
Copy link
Contributor

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

@SherlockNoMad @facebook-github-bot
Add output_node util function to fx.Graph (pytorch#139770)
be07e49 
Summary:

A util function for access output node for FX graph

Test Plan: OSS CI

Reviewed By: ezyang, Chillee

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

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

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

5 similar comments
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

13 similar comments
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge -i

(Initiating merge automatically since Phabricator Diff has merged, merging with -i because oss signals were bypassed internally)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 1 checks: pull / cuda12.1-py3.10-gcc9-sm75 / test (pr_time_benchmarks, 1, 1, linux.g4dn.metal.nvidia.gpu)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
@SherlockNoMad
Add output_node util function to fx.Graph (pytorch#139770)
0e5cbbc 
Summary: A util function for access output node for FX graph

Test Plan: OSS CI

Differential Revision: D65486457

Pull Request resolved: pytorch#139770
Approved by: https://github.com/ezyang, https://github.com/Chillee
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ezyang ezyang ezyang approved these changes

@Chillee Chillee Chillee approved these changes

@angelayi angelayi Awaiting requested review from angelayi

@zhxchen17 zhxchen17 Awaiting requested review from zhxchen17

Assignees

No one assigned

Labels

ciflow/trunk Trigger trunk jobs on your pull request fb-exported fx Merged release notes: fx release notes category

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@SherlockNoMad @facebook-github-bot @pytorchmergebot @ezyang @Chillee