Skip to content

Implement conversions to and from std::Vec and slice#25

Merged
HaoZeke merged 1 commit intomainfrom
vec-slice
Mar 18, 2026
Merged

Implement conversions to and from std::Vec and slice#25
HaoZeke merged 1 commit intomainfrom
vec-slice

Conversation

@Luthaf
Copy link
Copy Markdown
Member

@Luthaf Luthaf commented Mar 13, 2026

No description provided.

@Luthaf Luthaf requested a review from HaoZeke March 13, 2026 16:10
HaoZeke added a commit to HaoZeke/metatensor that referenced this pull request Mar 13, 2026
…belValue=i32

Stack PR metatensor#1063 (opaque mts_labels_t, array-primary Labels) on top of
PR metatensor#1073 (LabelValue newtype -> type alias). Resolves conflicts in
labels.rs Debug impl and adapts all LabelValue::new/i32()/usize()/isize()
call sites to work with plain i32.

Includes pickle fix: np.asarray strips LabelsValues subclass before
np.savez to avoid numpy 2.2+ allow_pickle requirement.

Updates dlpk dependency to vec-slice branch (metatensor/dlpk#25) for
Vec/slice conversions needed by labels array backing.

Temporarily disables wheel CI (git deps in Cargo.toml).
HaoZeke added a commit to HaoZeke/metatensor that referenced this pull request Mar 18, 2026
…belValue=i32

Stack PR metatensor#1063 (opaque mts_labels_t, array-primary Labels) on top of
PR metatensor#1073 (LabelValue newtype -> type alias). Resolves conflicts in
labels.rs Debug impl and adapts all LabelValue::new/i32()/usize()/isize()
call sites to work with plain i32.

Includes pickle fix: np.asarray strips LabelsValues subclass before
np.savez to avoid numpy 2.2+ allow_pickle requirement.

Updates dlpk dependency to vec-slice branch (metatensor/dlpk#25) for
Vec/slice conversions needed by labels array backing.

Temporarily disables wheel CI (git deps in Cargo.toml).
Copy link
Copy Markdown
Member

@HaoZeke HaoZeke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great and helps with metatensor.

@HaoZeke HaoZeke merged commit 4369181 into main Mar 18, 2026
5 checks passed
@HaoZeke HaoZeke deleted the vec-slice branch March 18, 2026 16:35
HaoZeke added a commit to HaoZeke/metatensor that referenced this pull request Mar 31, 2026
…belValue=i32

Stack PR metatensor#1063 (opaque mts_labels_t, array-primary Labels) on top of
PR metatensor#1073 (LabelValue newtype -> type alias). Resolves conflicts in
labels.rs Debug impl and adapts all LabelValue::new/i32()/usize()/isize()
call sites to work with plain i32.

Includes pickle fix: np.asarray strips LabelsValues subclass before
np.savez to avoid numpy 2.2+ allow_pickle requirement.

Updates dlpk dependency to vec-slice branch (metatensor/dlpk#25) for
Vec/slice conversions needed by labels array backing.

Temporarily disables wheel CI (git deps in Cargo.toml).
Luthaf pushed a commit to HaoZeke/metatensor that referenced this pull request Apr 1, 2026
…belValue=i32

Stack PR metatensor#1063 (opaque mts_labels_t, array-primary Labels) on top of
PR metatensor#1073 (LabelValue newtype -> type alias). Resolves conflicts in
labels.rs Debug impl and adapts all LabelValue::new/i32()/usize()/isize()
call sites to work with plain i32.

Includes pickle fix: np.asarray strips LabelsValues subclass before
np.savez to avoid numpy 2.2+ allow_pickle requirement.

Updates dlpk dependency to vec-slice branch (metatensor/dlpk#25) for
Vec/slice conversions needed by labels array backing.

Temporarily disables wheel CI (git deps in Cargo.toml).
HaoZeke added a commit to HaoZeke/metatensor that referenced this pull request Apr 3, 2026
…belValue=i32

Stack PR metatensor#1063 (opaque mts_labels_t, array-primary Labels) on top of
PR metatensor#1073 (LabelValue newtype -> type alias). Resolves conflicts in
labels.rs Debug impl and adapts all LabelValue::new/i32()/usize()/isize()
call sites to work with plain i32.

Includes pickle fix: np.asarray strips LabelsValues subclass before
np.savez to avoid numpy 2.2+ allow_pickle requirement.

Updates dlpk dependency to vec-slice branch (metatensor/dlpk#25) for
Vec/slice conversions needed by labels array backing.

Temporarily disables wheel CI (git deps in Cargo.toml).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants