chore: Rename DecodedArray to ArrayData - #136
Conversation
|
also worth considering having "Array" mean "an accessor to the chunks" and "ChunkX" mean "An X-flavored chunk (of an array)", e.g. "ChunkBytes" |
|
I think we could probably spend a bunch of time talking about this, so perhaps better to let it sit in the back of our minds while we progress. I think |
|
|
||
| DecodedArray: TypeAlias = Tensor | VariableArray | MaskedTensor | MaskedVariableArray | ||
| """The result of a read: one of the four decoded array layouts. | ||
| ArrayData: TypeAlias = Tensor | VariableArray | MaskedTensor | MaskedVariableArray |
There was a problem hiding this comment.
Perhaps FixedLengthTensor and VariableLengthTensor?
There was a problem hiding this comment.
Perhaps use Optional instead of Masked?
There was a problem hiding this comment.
Tensor: TypeAlias = FixedLengthTensor | VariableLengthTensor ... ?
|
Closing in favor of #172 |
2 participants
Rename
DecodedArraytoArrayDatato be more consistent in the naming across our codebase.Array: a (usually) out of memory Zarr typed arrayArrayBytes: in-memory untyped bytesArrayData: in-memory typed array data