Skip to content

Commit 6ce42f5

Browse files
mtfishmanclaude
andcommitted
Narrow NDTensors serialization public API to just the schema struct types
Drops `serialized_type`, `serialize_convert`, and `deserialize_convert` from the `public` declaration. These remain reachable from the JLD2 extension (and from `Base.convert` via the shims) but are no longer part of the package's stable public surface — no current external consumer calls them directly, so committing to them now is premature. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 38fb5f5 commit 6ce42f5

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

NDTensors/src/NDTensors.jl

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,12 +91,9 @@ include("empty/adapt.jl")
9191
# Serialization (schema types + backend-agnostic conversions, used by NDTensorsJLD2Ext)
9292
#
9393
include("serialization.jl")
94-
# Mark the on-disk schema types and the serialization conversion entry points as
95-
# public API (per Julia docs recommendation for backwards-compatible `public`
96-
# declarations on pre-1.11 Julia).
9794
VERSION >= v"1.11.0-DEV.469" && eval(
9895
Meta.parse(
99-
"public serialized_type, serialize_convert, deserialize_convert, SerializedDense, SerializedBlockSparse, SerializedDiag, SerializedUniformDiag, SerializedDiagBlockSparse, SerializedUniformDiagBlockSparse, SerializedEmptyStorage"
96+
"public SerializedDense, SerializedBlockSparse, SerializedDiag, SerializedUniformDiag, SerializedDiagBlockSparse, SerializedUniformDiagBlockSparse, SerializedEmptyStorage"
10097
)
10198
)
10299

0 commit comments

Comments
 (0)