-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add (untested) slow path and make performance test compile #27
Conversation
solimantwali
commented
Nov 30, 2023
•
edited
Loading
edited
- Rename utilities to utility_lib
- Debug performance tests
… around, deleted get_real_index()
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.
I didn't check the algorithm, I mostly checked for code hygiene.
Might as well merge it now with a broken algorithm so that we can run our perf tests sooner rather than later #️⃣approximatecomputing
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
…ad of utilities.hpp Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
… in insert in parallel.cppcomment for Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
Co-authored-by: David Chu <45278895+thedavidchu@users.noreply.github.com>
* utilities -> utility * changed traces folder to trace * changed generate_random_trace to generate_random_traces
* move hash function source comment from sequential.hpp to utility.cpp * alphabetize includes in performance_test/main.cpp * add more descriptive comment for shifting entries in remove() in parallel.cpp * use capacity_with_buffer in find_next_index_lock()
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.
All conversations resolved
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.
lgtm (on the condition it compiles with the changes!)