Serializing from_node info in et serializer #12462

Gasoonjia · 2025-07-14T21:22:00Z

Stack from ghstack (oldest at bottom):

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info.

Differential Revision: D78293986

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

pytorch-bot · 2025-07-14T21:22:03Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

✅ You can merge normally! (2 Unrelated Failures)

As of commit 005d2f3 with merge base e44da93 ():

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

pull / test-setup-linux-gcc / linux-job (gh) (detected as infra flaky with no log or failing log classifier)

BROKEN TRUNK - The following job failed but was present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

pull / test-phi-3-mini-runner-linux / linux-job (gh) (trunk failure)
AttributeError: 'StaticCacheConfig' object has no attribute 'get'

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) ghstack-source-id: 296149500 Pull Request resolved: #12462

facebook-github-bot · 2025-07-14T21:22:39Z

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

github-actions · 2025-07-14T21:22:45Z

This PR needs a `release notes:` label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-15T23:38:43Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-15T23:49:21Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-16T17:32:51Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-16T18:06:00Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-17T19:55:25Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-18T21:15:40Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-18T22:45:39Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-18T22:49:47Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-24T18:48:09Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-24T22:47:52Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

Pull Request resolved: #12462 We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. ghstack-source-id: 298619334 Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/)

facebook-github-bot · 2025-07-25T17:33:22Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

Pull Request resolved: #12462 We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. ghstack-source-id: 298622707 Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/)

facebook-github-bot · 2025-07-25T19:39:57Z

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

We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/) [ghstack-poisoned]

facebook-github-bot · 2025-07-25T20:06:33Z

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

Pull Request resolved: #12462 We need to use the from_node informaton in deserialzied exported graph for operator tracing in et.inspector. this diff updates the serizalier to support serde from_node info. ghstack-source-id: 298666561 Differential Revision: [D78293986](https://our.internmc.facebook.com/intern/diff/D78293986/)

Gasoonjia mentioned this pull request Jul 14, 2025

Use same constant in runtime for unset debug handle #12385

Merged

Gasoonjia mentioned this pull request Jul 14, 2025

Verify intermediate output capturer on exported program #12461

Closed

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 Jul 14, 2025

facebook-github-bot added the fb-exported label Jul 14, 2025

Gasoonjia mentioned this pull request Jul 15, 2025

make operator name consistent before and after serde #12531

Merged

Gasoonjia requested review from JacobSzwejbka and larryliu0820 as code owners July 15, 2025 23:38

angelayi approved these changes Jul 16, 2025

View reviewed changes

Gasoonjia mentioned this pull request Jul 16, 2025

bump pytorch bin to 0716 #12548

Merged

This was referenced Jul 17, 2025

support back propagate debug handle to arbitrary ancestor export graph #12580

Merged

Support export program in intermediate numeric discrepancy detector #12581

Merged

Gasoonjia mentioned this pull request Jul 18, 2025

bump pt core pin to 0723 #12649

Merged

facebook-github-bot merged commit f9cf693 into gh/gasoonjia/23/base Jul 26, 2025
98 of 101 checks passed

facebook-github-bot deleted the gh/gasoonjia/23/head branch July 26, 2025 03:35

facebook-github-bot temporarily deployed to cherry-pick-bot July 26, 2025 03:35 — with GitHub Actions Inactive

pytorchbot mentioned this pull request Jul 26, 2025

Serializing from_node info in et serializer #12876

Merged

Serializing from_node info in et serializer #12462

Serializing from_node info in et serializer #12462

Uh oh!

Conversation

Gasoonjia commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pytorch-bot bot commented Jul 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

✅ You can merge normally! (2 Unrelated Failures)

Uh oh!

facebook-github-bot commented Jul 14, 2025

Uh oh!

github-actions bot commented Jul 14, 2025

This PR needs a release notes: label

Uh oh!

facebook-github-bot commented Jul 15, 2025

Uh oh!

facebook-github-bot commented Jul 15, 2025

Uh oh!

facebook-github-bot commented Jul 16, 2025

Uh oh!

facebook-github-bot commented Jul 16, 2025

Uh oh!

facebook-github-bot commented Jul 17, 2025

Uh oh!

facebook-github-bot commented Jul 18, 2025

Uh oh!

facebook-github-bot commented Jul 18, 2025

Uh oh!

facebook-github-bot commented Jul 18, 2025

Uh oh!

facebook-github-bot commented Jul 24, 2025

Uh oh!

facebook-github-bot commented Jul 24, 2025

Uh oh!

facebook-github-bot commented Jul 25, 2025

Uh oh!

facebook-github-bot commented Jul 25, 2025

Uh oh!

facebook-github-bot commented Jul 25, 2025

Uh oh!

Uh oh!

Uh oh!

Gasoonjia commented Jul 14, 2025 •

edited

Loading

pytorch-bot bot commented Jul 14, 2025 •

edited

Loading

This PR needs a `release notes:` label