make et.export support etrecord generation #13164

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

Open

Gasoonjia wants to merge 8 commits into gh/gasoonjia/38/base from gh/gasoonjia/38/head

Contributor

Gasoonjia commented Aug 6, 2025 •

edited

Loading

Stack from ghstack (oldest at bottom):

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: D79741917


          make et.export support etrecord generation

273bd0a

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

This was referenced Aug 6, 2025

raise error when trying to save an etrecord missing essential info #13143

Merged

make to_edge support etrecord generation #13151

Open

pytorch-bot bot commented Aug 6, 2025 •

edited

Loading

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 4 New Failures, 1 Unrelated Failure

As of commit e8df9ca with merge base a8fe653 ():

NEW FAILURES - The following jobs have failed:

Build documentation / build (buck2) / Build doc (gh)
At least one of the pre-conditions you specified did not hold
pull / test-moshi-linux / linux-job (gh)
RuntimeError: Command docker exec -t 269d8e4fdff1347778c6f6e2e3ef21f4643569d88f81bd1b30b208aa89652185 /exec failed with exit code 1
pull / unittest / linux / linux-job (gh)
examples/models/llama/tests/test_ring_attention.py::TestRingAttention::test_single_token_processing_quantized
pull / unittest-editable / linux / linux-job (gh)
examples/models/llama/tests/test_ring_attention.py::TestRingAttention::test_single_token_processing_quantized

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

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

pull / test-openvino-linux / linux-job (gh) (trunk failure)
AttributeError: '_OpNamespace' 'quantized_decomposed' object has no attribute 'convert_element_type'

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

meta-cla bot added the CLA Signed label

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

c6aeb42

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

ghstack-source-id: 301172282
Pull Request resolved: #13164

Contributor

facebook-github-bot commented Aug 6, 2025

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

facebook-github-bot added the fb-exported label


          Update on "make et.export support etrecord generation"

c9e5e0a

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

b22bf3e

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)
ghstack-source-id: 301175953

Contributor

facebook-github-bot commented Aug 6, 2025

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

Gasoonjia added the release notes: none label


          Update on "make et.export support etrecord generation"

6593cc3

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

765d22c

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301220396

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 6, 2025

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

abhinaykukkadapu approved these changes

View reviewed changes


          Update on "make et.export support etrecord generation"

9187fb0

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

d94ac82

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301336309

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 7, 2025

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

Contributor

abhinaykukkadapu commented Aug 7, 2025

@Gasoonjia i think unittests of export session needs to be updated.

Gasoonjia mentioned this pull request

make export flow etrecord generation supportive #12961

Closed


          Update on "make et.export support etrecord generation"

ab0983f

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

aa99de4

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301468548

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 7, 2025

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


          Update on "make et.export support etrecord generation"

71a6634

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

5a63a3e

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301482196

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 7, 2025

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


          Update on "make et.export support etrecord generation"

f8e4d65

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

a86a573

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301624415

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 8, 2025

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


          Update on "make et.export support etrecord generation"

e8df9ca

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 has completed.

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

[ghstack-poisoned]

Gasoonjia added a commit that referenced this pull request


          make et.export support etrecord generation

ead6166

Pull Request resolved: #13164

this diff makes et.export etrecord generation supportive. Details can be found in #12925.

After this change, all things in #12925 have been completed.
ghstack-source-id: 301624415

Differential Revision: [D79741917](https://our.internmc.facebook.com/intern/diff/D79741917/)

Contributor

facebook-github-bot commented Aug 8, 2025

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

github-actions bot commented Oct 8, 2025

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

github-actions bot added the stale label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed fb-exported release notes: none stale