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

Improve usability of HashSet/Map type aliases #51

Closed
wants to merge 138 commits into from
Closed

Conversation

ogxd
Copy link
Owner

@ogxd ogxd commented Dec 25, 2023

Extracted from #35

actions-user and others added 20 commits November 21, 2023 23:38
# Conflicts:
#	.github/workflows/bench.yml
#	benches/throughput/main.rs
#	benches/throughput/result_processor.rs
* Improve readme (WIP)

* Update Benchmark Plots

* Update Benchmark Plots

* Fix avx2 in CI

* At least 1 seconds of runtime for throughput benchmark

* Switch to nightly rust for AVX2 bench

* Update Benchmark Plots

* Update Benchmark Plots

* Reduce number of benchmark steps

* Finish

* Fix plots path

* Update Benchmark Plots

* Fix aarch svg

* Update Benchmark Plots

* Remove outliers in throughput benchmark

* Update Benchmark Plots

* Small tweaks

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
Co-authored-by: Author <actions@github.com>
Co-authored-by: Benchmark Bot <benchmark-bot@noreply.com>
* Use different artifact to prevent conflicts

* Update Benchmark Plots

* LS

* Update Benchmark Plots

* Delete .svg before downloading artifacts

* Update Benchmark Results

* Verbose rm

* Update Benchmark Results

* Wtf it does not work ?

* Update Benchmark Results

* Add fricking LS back

* Update Benchmark Results

* Try using buildjet cache instead

* Update Benchmark Results

* Fix CI

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
Co-authored-by: Benchmark Bot <benchmark-bot@noreply.com>
* Added `Clone` and `Debug` traits to `GxHasher` and `GxBuildHasher`.

* Updated deps.

* Made clippy (almost) happy.

* rustfmt.

---------

Co-authored-by: Olivier Giniaux <oginiaux@gmail.com>
Co-authored-by: Ian Qvist <Genbox@users.noreply.github.com>
Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
* Add specialized write_u32

* Fix hashset benchmark accurracy

* Implement all types

* Fix build

* Fix avx2

* Add mandatory 0 length check to avoid SIGSEGV and add test

* Version 2.3.0

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
* Create dedicated quality bench

* Add avalanche

* Code cleanup

* More cleanup

* Add t1ha

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
* Fix permutation quality issue

* Add more quality tests

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
…in a hybrid way for maximum performance

* Change construction to allow stable hashes even with AVX2 and use it in a hybrid way for maximum performance

* Add unit test on permutations

* Fix permutations

* Recursive chunked stepped permute

* Reduce bytecode size from duplicated AES keys

* Fix is_stable test

* Update Benchmark Results

---------

Co-authored-by: Olivier Giniaux <oginiaux@smartadserver.com>
Co-authored-by: Benchmark Bot <benchmark-bot@noreply.com>
@ogxd
Copy link
Owner Author

ogxd commented Dec 30, 2023

Had to fixup an issue with my git config before repo gets too large / too many contributors to do it. It messed up this PR but no worry I have rebased it: #56

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

Successfully merging this pull request may close these issues.

Tracking: missing compatibility with std::HashMap & std::HashSet
4 participants