-
Notifications
You must be signed in to change notification settings - Fork 41
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
c# port of the java version #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this contribution, this is great!
Feel free to add yourself to the acknowledgments section if you want: https://github.com/hbf/miniball#authors--acknowledgements
Regarding the unit tests, the three that fail succeed in Java, or what’s the context around these?
Thanks you, I added to acknowledgements. This happens from java test cd java
mvn test results
and corresponding c# cd csharp
dotnet test resulting
details follows: test_cocircular_points_large_radius_2test_simplex_15test_simplex_10notes on how to debug test c#prerequisites open folder shortcuts:
|
Ah, I see, they fail in Java as well. That definitely needs some cleanup/addressing some time but I don’t want you to be blocked by this. Not sure you can merge, if not lmk when it’s ready from your side and I’ll merge. |
I have no more commit, for me it can be merged. |
This PR adds c# support for the miniball library.
Complete of unit tests ( 11 passed, 3 failed, 1 skipped ) like the Java implementation results, with an example like the c++ version and the Getting Started doc.