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

Docs/benchmark postproc #55

Merged
merged 20 commits into from
Oct 7, 2024
Merged

Docs/benchmark postproc #55

merged 20 commits into from
Oct 7, 2024

Conversation

jamesETsmith
Copy link
Contributor

Summary

This PR adds benchmarks for three operations and compares fast-pauli and qiskit for all three. It also adds some discussion around them on the docs page.

Details

We focus on three operations:

  1. Pauli string applied to a state (shows that we can do things quickly)
  2. Pauli operators applied to a states (shows that we're still faster even for more complex objects) for a range of different sizes of Pauli operators).
  3. Expectation value for a batch of states (shows that we can do expectation values and work with batches or states)

@jamesETsmith jamesETsmith added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 4, 2024
@jamesETsmith jamesETsmith self-assigned this Oct 4, 2024
fast_pauli/cpp/include/__pauli_op.hpp Outdated Show resolved Hide resolved
tests/benchmarks/process_benchmark_data.py Show resolved Hide resolved
tests/benchmarks/process_qiskit_benchmarks.py Outdated Show resolved Hide resolved
tests/benchmarks/process_qiskit_benchmarks.py Outdated Show resolved Hide resolved
tests/benchmarks/test_qiskit_adv.py Outdated Show resolved Hide resolved
@jamesETsmith
Copy link
Contributor Author

I'm going to delete the unused figure/benchmarks to reduce clutter and tech debt. When we revisit them, this is a pointer to a commit, where the code exists in commented out form.

…nchmark.rst

* Adding new benchmark numbers and cleaning up the scripts

* Cleaning up some clutter introduced to hpp files during benchmarking
docs/benchmarks.rst Outdated Show resolved Hide resolved
tests/benchmarks/process_benchmark_data.py Show resolved Hide resolved
tests/benchmarks/process_benchmark_data.py Outdated Show resolved Hide resolved
alexanderlerner and others added 4 commits October 7, 2024 11:48
This reverts commit ac73757.
accidentally pushed to wrong branch
…nchmark.rst

* Adding new benchmark numbers and cleaning up the scripts

* Cleaning up some clutter introduced to hpp files during benchmarking
Makefile Show resolved Hide resolved
docs/benchmarks.rst Outdated Show resolved Hide resolved
docs/benchmarks.rst Outdated Show resolved Hide resolved
Copy link
Collaborator

@stand-by stand-by left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@jamesETsmith jamesETsmith merged commit 781d501 into main Oct 7, 2024
10 checks passed
@jamesETsmith jamesETsmith deleted the docs/benchmark_postproc branch October 7, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants