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

Port and improve Perf_Marvin #44

Closed
3 tasks done
adamsitnik opened this issue Jun 26, 2018 · 0 comments
Closed
3 tasks done

Port and improve Perf_Marvin #44

adamsitnik opened this issue Jun 26, 2018 · 0 comments

Comments

@adamsitnik
Copy link
Member

adamsitnik commented Jun 26, 2018

  • Port from xunit-performance to BenchmarkDotNet
  • investigate why the benchmarks are not calling Marvin API from CoreFX, but instead, have it's own copy of the hashing algorithm. IMHO it creates a need to synchronize the code between repositories and makes it possible to miss a regression if somebody changes the implementation in CoreFX
  • reduce the number of permutations, it's more a unit test than a benchmark today.
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