Skip to content
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

RFC: document compatibility properties of serialize #41520

Merged
merged 1 commit into from
Nov 9, 2021
Merged

Conversation

JeffBezanson
Copy link
Sponsor Member

For every 1.x release so far, we have kept serialize backwards compatible for reading. Since this has been possible for 7 releases, the time has probably come to officially promise it. While the "syntax" of the data remains readable, there are still some caveats I have tried to point out. Did I miss anything?

@JeffBezanson JeffBezanson added the domain:docs This change adds or pertains to documentation label Jul 8, 2021
@DhairyaLGandhi
Copy link

This is great! Thank you! One thought I had was that in ML it is fairly common to specify lots of small details as anonymous functions and closures all over the place. Serializing those and reading them back in (with the intention to train further) is also pretty crucial since it can be quite difficult to train some of these models over and over again. We have used BSON.jl so far, and while that has its quirks, it has been a good tool in the arsenal. We also have plans on a "data only" storage for optional cases, but I guess that is somewhat separate.

What do you think would be a good way of maintaining compatibility with serialized anonymous functions?

@ViralBShah
Copy link
Member

@JeffBezanson This would be great to merge.

@JeffBezanson JeffBezanson added the status:merge me PR is reviewed. Merge when all tests are passing label Nov 3, 2021
@timholy
Copy link
Sponsor Member

timholy commented Nov 3, 2021

#42016 changes a couple of tags, not sure whether that's incompatible (some of the common symbols had to be jettisoned to make space for new "real" tags).

@vtjnash vtjnash merged commit 7e85914 into master Nov 9, 2021
@vtjnash vtjnash deleted the jb/sercompatdocs branch November 9, 2021 18:44
@DilumAluthge DilumAluthge removed the status:merge me PR is reviewed. Merge when all tests are passing label Nov 10, 2021
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants