-
Notifications
You must be signed in to change notification settings - Fork 5
Rename TensorDynLen to IdxTensor for ITensor compatibility #374
Copy link
Copy link
Open
Description
Motivation
TensorDynLen is the core tensor type in tensor4all-rs, analogous to ITensor's ITensor type. The current name describes implementation details (dynamic-length indices) rather than what the type conceptually represents. To improve discoverability and emphasize compatibility with the ITensor ecosystem, we should rename it to IdxTensor — a name that highlights the index-based tensor semantics shared with ITensor.
Proposal
- Rename
pub struct TensorDynLen→pub struct IdxTensorincrates/tensor4all-core/src/defaults/tensordynlen.rs - Keep
pub type TensorDynLen = IdxTensor;as a deprecated type alias for backward compatibility during the transition - Update all internal usage (library code, tests, examples, docs) to use
IdxTensor - Update C API and language bindings as needed
Scope
- 152 files, ~1926 occurrences of
TensorDynLenacross the workspace - Struct definition:
crates/tensor4all-core/src/defaults/tensordynlen.rs - Re-exported from
crates/tensor4all-core/src/lib.rs
Notes
- Since the project is in early development with no backward compatibility guarantees, the alias is mainly for a smooth transition period and can be removed later.
- Consider also renaming the source file from
tensordynlen.rstoidx_tensor.rs.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels