-
Notifications
You must be signed in to change notification settings - Fork 29
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# version #5
Comments
Hi @Genbox ! I'm glad to see it had someone attention 😊 I've done my best to test it thoroughly, but having someone else give it a spin would be awesome. Really curious to see how it holds up in your benchmarks and open to any feedback you've got. I'll create a new repository tomorrow so I can share the source code and push improvements until it matches this rust implementation performance-wise. I'll also include a few basic tests. |
Here it is: gxhash-csharp Actual implementation of the algorithm is here: https://github.com/ogxd/gxhash-csharp/blob/main/GxHash/GxHash.cs I have added a few more optimization. It might not be "the most optimal" but I think it is at least conform to the spec and it already outperforms XxH3 and Marvin both on my MacBook m1 pro (ARM) and my x86 machine. Here are the benchmark results on a github runner:
|
Thanks Olivier. I'll check it out. Closing this issue for now. |
I saw your post on dotnet/runtime#85206 about a C# port. Are you able to share it? I'd like to benchmark it both in terms of speed and correctness.
The text was updated successfully, but these errors were encountered: