Skip to content

[cherry-pick][export] Update schema version #115952

Merged
malfet merged 1 commit intorelease/2.2from
angelayi/115712
Dec 18, 2023
Merged

[cherry-pick][export] Update schema version #115952
malfet merged 1 commit intorelease/2.2from
angelayi/115712

Conversation

@angelayi
Copy link
Copy Markdown
Contributor

@angelayi angelayi commented Dec 15, 2023

Since pytorch 2.1 release we've made some BC breaking changes to the serialized schema. We should update it in time for the 2.2 release. Some of the changes include:

* #114371 - custom class objects / pybinded objects are no longer saved directly to the `ExportedProgram` structure. Instead, the name is serialized inside of the program, and the actual bytes are stored. in a separate location from the exported program, allowing it to be saved to a different location.
* #111204 - `GraphSignature` structure changed and `call_spec` is removed from the `GraphModule` schema
* #111407 - `loss_outout` -> `loss_output`
* #113075 - `example_inputs` removed from the `ExportedProgram` structure (this originally did not store anything), `dialect` added to the `ExportedProgram` structure.
* #113689 - tensor constants are now lifted as inputs to the graph, and their locations are stored in the `GraphSignature`
* #114172 - removed `equality_constraints` and added a `SymExprHint` for all symbolic expressions.
Pull Request resolved: #115712
Approved by: https://github.com/gmagogsfm
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Dec 15, 2023

🔗 Helpful Links

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

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

✅ No Failures

As of commit 30a919a with merge base a8bd593 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@malfet malfet merged commit e8ebe2c into release/2.2 Dec 18, 2023
@malfet malfet deleted the angelayi/115712 branch December 18, 2023 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants