Skip to content
This repository has been archived by the owner on Jul 16, 2021. It is now read-only.

Adding column iterators #52

Closed
AtheMathmo opened this issue Apr 22, 2016 · 2 comments
Closed

Adding column iterators #52

AtheMathmo opened this issue Apr 22, 2016 · 2 comments

Comments

@AtheMathmo
Copy link
Owner

We have added row iterators with #48, #49 . These were easy to do because our matrices are row-major. as mentioned in #48 it would be nice to have column iterators too.

This ticket is to track the implementation of column iterators. The first step should be to write up a specification for how exactly these should look.

@AtheMathmo
Copy link
Owner Author

One way to do this would be to have the iterator produce MatrixSlices. This would mean setting the column size of each MatrixSlice to 1 and letting the row_stride be the actual row_stride.

Advantages:

We do not need to copy any data. This is left to the user if they want to use contiguous blocks.

Disadvantages:

Differs from the row iterators which return slices. We opted to return slices in that case to assure the user that the data was indeed contiguous (which provides optimization benefits). If we were to modify this to return MatrixSlices we would not be able to guarantee this so strongly.

@AtheMathmo
Copy link
Owner Author

This issue was moved to AtheMathmo/rulinalg#3

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

No branches or pull requests

1 participant