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

Draft: Add make_sparse_matrix_op bench #40

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kris524
Copy link

@kris524 kris524 commented Jul 5, 2022

As a start, I wanted to add a new benchmark test and I thought that make_sparse_matrix_op might be a useful one and we don't have an existing one as well

@kris524 kris524 marked this pull request as draft July 5, 2022 23:08
@kris524
Copy link
Author

kris524 commented Jul 8, 2022

I will add more bench tests later. I will switch to writing some examples to get to know the codebase better

@Renmusxd
Copy link
Owner

Renmusxd commented Jul 8, 2022

Sounds good - I definitely need to work on adding documentation (I lot of it was lost when I rewrote things), so if anything is unclear please don't hesitate to ask so I know to add better docs in that area.

@kris524
Copy link
Author

kris524 commented Jul 8, 2022

Is this still relevant then? https://docs.rs/qip/latest/qip/

@Renmusxd
Copy link
Owner

Renmusxd commented Jul 8, 2022

Those docs are for the Cargo version which is behind the github version. I'd recommend building the docs locally with cargo doc

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.

2 participants