Skip to content
This repository was archived by the owner on Jul 15, 2025. It is now read-only.

Initializers #13

Closed
wants to merge 3 commits into from
Closed

Conversation

karllessard
Copy link
Contributor

@karllessard karllessard commented Dec 5, 2022

Initializes a just-allocated array using a new API that focuses on a stateful builder approach rather than the positional endpoints setXXX(value, coords) exposed by the NdArray itself.

Not only it simplifies the task of initializing an array with explicit values but also it allows us to allocate an empty sparse array and then let the user initialize its data (right now, the user have to allocate and initialize the data of the indices and values subarrays first before being able to allocate a sparse array).

@karllessard karllessard requested a review from Craigacp December 5, 2022 04:05
@karllessard karllessard mentioned this pull request Dec 5, 2022
@karllessard
Copy link
Contributor Author

karllessard commented Dec 5, 2022

Again, now just the implementation for basic object arrays and double arrays is present in the PR. Once we are happy with them, I'll add more tests and support for all other type of arrays.

@karllessard karllessard closed this Dec 5, 2022
@karllessard
Copy link
Contributor Author

Since it is not ready to be merged yet (missing types, etc.), this should be reopened as a draft, sorry for the noise

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant