-
Notifications
You must be signed in to change notification settings - Fork 0
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
Kd/improve solver #62
Conversation
Please fix test coverage. |
I have removed many tested functions (with tests), hence the coverage has fallen. I have added some new tests in contractions. Nevertheless there is no tests in cuda, detailed_mps, and some tested are needed in utils. I am not qualified there |
What is the purpose of |
I see now, you asked to store somewhere the old (my) mps implementation. I did not noticed it has an impact to the coveralls. Where shall I move it, benchamrks? |
Or remove from repo? |
As far as I remebmer this implementation has a slightly different idea than ours, I would store it in someplace. |
done |
No description provided.