Since the internal storage of xlil_matrix is going to be replaced by a simple index <-> value map, the name of the class (and the file) should be changed to reflect this change.
The xmap_sparse_container should be quite similar to xcontainer: a CRTP base class which gathers common implementation for xmap_sparse_array (dynamic number of dimensions) and xmap_sparse_tensor (static number of dimensions).
Onc ethe basic features (access operators and reshape) are implemented, we can make it inherit from utility base classes from xtensor (xaccessible, xiterable, xexpression).
JohanMabille
commented
4 years ago
Since the internal storage of xlil_matrix is going to be replaced by a simple index <-> value map, the name of the class (and the file) should be changed to reflect this change.
The
xmap_sparse_containershould be quite similar to xcontainer: a CRTP base class which gathers common implementation forxmap_sparse_array(dynamic number of dimensions) andxmap_sparse_tensor(static number of dimensions).Onc ethe basic features (access operators and reshape) are implemented, we can make it inherit from utility base classes from
xtensor(xaccessible,xiterable,xexpression).