-
-
Notifications
You must be signed in to change notification settings - Fork 173
Closed
Labels
compat-breakerThis PR is good to merge, but breaks compatibility and needs to wait till we prepare a major releaseThis PR is good to merge, but breaks compatibility and needs to wait till we prepare a major release
Description
Currently we serialize them as k: [v]
or k: []
, which is generally not what people want. We should instead serialize them as k: v
or nothing.
Nested options could probably continue to be serialized boxed in a single-element array. Options in sequences and other things that serialize to arrays can conitnue to be boxed as well.
This would be a breaking change and would need to go into upickle 4.0.0
nemoo, ckipp01, frou, lolgab, LMnet and 1 more
Metadata
Metadata
Assignees
Labels
compat-breakerThis PR is good to merge, but breaks compatibility and needs to wait till we prepare a major releaseThis PR is good to merge, but breaks compatibility and needs to wait till we prepare a major release