Skip to content

Conversation

@pytorchbot
Copy link
Collaborator

This PR was created by the merge bot to help merge the original PR into the main branch.
ghstack PR number: #13020 by @Gasoonjia
^ Please use this as the source of truth for the PR details, comments, and reviews
ghstack PR base: https://github.com/pytorch/executorch/tree/gh/gasoonjia/31/base
ghstack PR head: https://github.com/pytorch/executorch/tree/gh/gasoonjia/31/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/gasoonjia/30/orig
Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/gasoonjia/31/orig
@diff-train-skip-merge

Pull Request resolved: #13020

Previously we have to provide all essentail infos at the same time to generate etrecord; however if we want to generate it through export flow we can not find a stage that having all essential infos so that we need to have a new way to contruct it on-the-fly.

This diff makes the target happen by adding three functions: `add_exported_program`, `add_edge_dialect_program` and `add_executorch_program` so that whenever we have the required info we can equip it into etrecord.

Also update test case for test coverage.
ghstack-source-id: 300161202

Differential Revision: [D79294945](https://our.internmc.facebook.com/intern/diff/D79294945/)
@pytorchbot pytorchbot requested a review from Gasoonjia as a code owner August 2, 2025 00:00
@pytorch-bot
Copy link

pytorch-bot bot commented Aug 2, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 7 Pending, 1 Unrelated Failure

As of commit 867467d with merge base ec35f56 (image):

NEW FAILURES - The following jobs have failed:

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

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

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

@meta-cla meta-cla 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 Aug 2, 2025
@Gasoonjia Gasoonjia closed this Aug 4, 2025
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants