Skip to content

Introduce write_to_file api #2307

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.

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

Closed
wants to merge 2 commits into from
Closed

Conversation

lucylq
Copy link
Contributor

@lucylq lucylq commented Mar 7, 2024

Summary: Update callsites that save to file to use write_to_file api instead of .buffer

Reviewed By: dbort

Differential Revision: D54526788

Copy link

pytorch-bot bot commented Mar 7, 2024

🔗 Helpful Links

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

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit 09ea15e with merge base 9d6bf72 (image):

NEW FAILURE - The following job has 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.

@facebook-github-bot 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 Mar 7, 2024
@facebook-github-bot
Copy link
Contributor

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

lucylq added 2 commits March 7, 2024 14:41
Summary: And update existing callsites to use bytes(serialize_pte_binary..), or write_to_file

Differential Revision: D54524444
Summary:
Pull Request resolved: pytorch#2307

Update callsites that save to file to use write_to_file api instead of .buffer

Reviewed By: dbort

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

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

@lucylq lucylq force-pushed the export-D54526788 branch from fed00e9 to 09ea15e Compare March 7, 2024 22:43
@guangy10
Copy link
Contributor

guangy10 commented Mar 8, 2024

@pytorchbot label ciflow/trunk

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5316e69.

robell added a commit to robell/executorch that referenced this pull request Mar 12, 2024
Signed-off-by: Rob Elliott <[email protected]>
Change-Id: I584410307d4b9c2635ae6647cd9bd138bf596c45
facebook-github-bot pushed a commit that referenced this pull request Mar 19, 2024
Summary:
Fixes for a few things:

- Move C++ std to 17 to enable changes for #1733 on Arm builds
- fix after #2307 regressed save_pte_program on aot_arm_compiler
- add aot_arm_compiler support for examples/models (they don't all compile but can be consumed)
- Move the Vela pin in anticipation of MNV2 tosa fixes in #2371

Pull Request resolved: #2372

Reviewed By: mcr229

Differential Revision: D54817009

Pulled By: digantdesai

fbshipit-source-id: 5ca3c7a03e8ae9f6a784335d53654b8db05a3cb8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants