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 <[email protected]>
Co-authored-by: Author <[email protected]>
Co-authored-by: Benchmark Bot <[email protected]>
* 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 <[email protected]>
Co-authored-by: Benchmark Bot <[email protected]>
* Added `Clone` and `Debug` traits to `GxHasher` and `GxBuildHasher`.

* Updated deps.

* Made clippy (almost) happy.

* rustfmt.

---------

Co-authored-by: Olivier Giniaux <[email protected]>
* 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 <[email protected]>
* Create dedicated quality bench

* Add avalanche

* Code cleanup

* More cleanup

* Add t1ha

---------

Co-authored-by: Olivier Giniaux <[email protected]>
* Fix permutation quality issue

* Add more quality tests

---------

Co-authored-by: Olivier Giniaux <[email protected]>
…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 <[email protected]>
Co-authored-by: Benchmark Bot <[email protected]>
@ogxd ogxd force-pushed the no-prefix-collections branch from 61b9fdd to 789c19a Compare December 25, 2023 22:05
@ogxd ogxd linked an issue Dec 25, 2023 that may be closed by this pull request
@ogxd ogxd force-pushed the main branch 2 times, most recently from afe803f to 7629ab1 Compare December 30, 2023 21:26
@ogxd ogxd closed this Dec 30, 2023
@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