Skip to content
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

Replace matrices.rs with ndarray #3

Open
johanneskoester opened this issue Mar 10, 2017 · 2 comments
Open

Replace matrices.rs with ndarray #3

johanneskoester opened this issue Mar 10, 2017 · 2 comments

Comments

@johanneskoester
Copy link

ndarray has become the de-facto standard for matrix handling in Rust. I suggest to use that instead of building your own matrix implementation.

@klangner
Copy link
Owner

Yes I know and agree.
My problem is that I have a lot of problems using this library. Probably that's because I'm learning Rust.
But also I'm not sure about some aspects of its API. To give constructive feedback to ndarray I decided that first I need to learn how to design this kind of library.

So my plan for the first version is to hide matrix module (don't export it). Use my own matrix implementation to learn how the matrix library could be designed in Rust and then provide feedback to ndarray library owner. Add necessary changes (if any) and switch to it.

I will leave this issue open as a task to replace matrix implementation.

@johanneskoester
Copy link
Author

Ok, that's fine!

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

No branches or pull requests

2 participants