-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Strided containers #166
Comments
The short answer is not The long answer is yes and I was working on implementing this in Bau. It wasn't a show stopper but I quickly arrived at the conclusion that one needed Perhaps this is a good spot as any to write down my thoughts and experiences on the subject. I think there are several issues in this space, some of which interact in various ways:
All of this shouldn't discourage you from submitting a PR; working code is infinitely more valuable than no code. I just want to share my thinking on the subject. |
A bit OT but note that things like flambda are not available in bytecode which is what Which leads me to something I wanted to ask for a long time, how much of |
I did not realize that bigarrays are available on I've reserved a bit of time this and next week to see how much of |
@dbuenzli If you have a chance take a look at |
@rleonid Caveat, I have little knowledge on how My approach would be to rather try to reorganize the API so that |
Have you considered making
Vector.t
strided (c.f. NumPy)?The major benefits of this approach are:
Memory efficiency. No pointer chasing for whole-matrix operations.
Code reuse between
Vectors
andMatrices
, e.g.Update: I probably should've added this as a comment to #27.
The text was updated successfully, but these errors were encountered: