Skip to content

Conversation

@lucylq
Copy link
Contributor

@lucylq lucylq commented Sep 29, 2025

Stack from ghstack (oldest at bottom):

Reuse DataEntry from data_serializer.py, in _named_data_store.py.

Motivation

  • deserialize from flat tensor to named data store output
  • support tensor layout in named data store

Differential Revision: D83490345

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 29, 2025

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

❌ 1 New Failure

As of commit 3ea974a with merge base 09408e3 (image):

NEW FAILURE - The following job has failed:

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

lucylq added a commit that referenced this pull request Sep 29, 2025
Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

ghstack-source-id: 312807877
Pull Request resolved: #14667
@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 Sep 29, 2025
@facebook-github-bot
Copy link
Contributor

@lucylq has exported this pull request. If you are a Meta employee, you can view the originating diff in D83490345.

@github-actions
Copy link

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.

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 14, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
ghstack-source-id: 316164323
Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 14, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store
ghstack-source-id: 316169224

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
Copy link
Contributor

@larryliu0820 larryliu0820 left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@lucylq lucylq requested a review from larryliu0820 October 28, 2025 23:51
Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 29, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store
ghstack-source-id: 319563768

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 29, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store
ghstack-source-id: 319566810

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 29, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store
ghstack-source-id: 319566810

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
Copy link
Contributor

@larryliu0820 larryliu0820 left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store

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

[ghstack-poisoned]
lucylq added a commit that referenced this pull request Oct 30, 2025
Pull Request resolved: #14667

Reuse `DataEntry` from data_serializer.py, in _named_data_store.py.

Motivation
- deserialize from flat tensor to named data store output
- support tensor layout in named data store
ghstack-source-id: 319685093

Differential Revision: [D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
@meta-codesync meta-codesync bot merged commit 4f2e061 into gh/lucylq/115/base Oct 30, 2025
149 of 153 checks passed
@meta-codesync meta-codesync bot deleted the gh/lucylq/115/head branch October 30, 2025 03:21
lucylq added a commit that referenced this pull request Oct 30, 2025
This PR was created by the merge bot to help merge the original PR into
the main branch.
ghstack PR number: #14667 by
@lucylq
^ 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/lucylq/115/base
ghstack PR head:
https://github.com/pytorch/executorch/tree/gh/lucylq/115/head
Merge bot PR base: https://github.com/pytorch/executorch/tree/main
Merge bot PR head:
https://github.com/pytorch/executorch/tree/gh/lucylq/115/orig
Differential Revision:
[D83490345](https://our.internmc.facebook.com/intern/diff/D83490345/)
@diff-train-skip-merge

Co-authored-by: lucylq <[email protected]>
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 meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants