Make vector hashable about rulinalg HOT 3 CLOSED

Thanks for posting the issue!

I appreciate that this is a very quick and easy fix - but could you explain the use case for me? I ask as I don't see anything obvious myself and this will increase compile time (probably negligibly).

Also - would we not want this to also be possible for Matrix?

from rulinalg.

In my case, I have a structure containing a vector and I use that structure as the key of my HashMap. To be able to do this, my structure has to implement the Hash trait and this Vector needs it too (to be able to use derive).

Maybe there are some other use cases but I have not experienced them.
(If it is considered a valid argument, the native Vec also implements Hash)

Finally, if it is implemented in Vector it should probably be implemented in Matrix for consistency.

from rulinalg.

I see, thanks! I've added a comment to PR #11 . In short I'm happy to see it added unless there is a significant increase in compile time (which I don't see happening).

from rulinalg.