-
Notifications
You must be signed in to change notification settings - Fork 58
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
[Submission]: Optimal Pauli string grouping for dense operators #488
Comments
Successfull submission! ✨ PR #489 |
Tests with latest version of Qiskit release passed! ✨ |
Tests with development version of Qiskit release passed! ✨ |
Awesome 🚀 |
Submission PR #489 has been created with errors in tests |
Tests with latest version of Qiskit release failed! |
Tests with development version of Qiskit release failed! |
@atlytle seems your last commit make the tests doesn"t pass anymore :/ error : 2023-08-16 14:21:15 ecosystem INFO file /home/runner/work/ecosystem/ecosystem/ecosystem/resources/cloned_repo_directory/dense-ev/src/dense_ev/rmatrix.py, line 70
2023-08-16 14:21:15 ecosystem INFO def test_random_H(m):
2023-08-16 14:21:15 ecosystem INFO E fixture 'm' not found
2023-08-16 14:21:15 ecosystem INFO > available fixtures: cache, capfd, capfdbinary, caplog, capsys, capsysbinary, doctest_namespace, monkeypatch, pytestconfig, record_property, record_testsuite_property, record_xml_attribute, recwarn, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory |
I had expanded the test coverage slightly, locally it runs ok with tox and unittest. I changed the function name to check_random_H(m) in case there is something automatic with functions "test*()". Do you have any suggestions? Thanks! |
@atlytle never mind Can you just do an edit in your issue and modify a space or a dot and save, that'll launch the wf again :) So still awesome your project passed first time 🚀 |
Successfull submission! ✨ PR #489 |
Tests with latest version of Qiskit release passed! ✨ |
Tests with development version of Qiskit release passed! ✨ |
Add dense-ev to list. Closes #488 Co-authored-by: atlytle <atlytle@users.noreply.github.com>
hello @atlytle! You joined the Qiskit Ecosystem a little while ago and I just wanted to check in and let you know of some follow up actions you can take to make the most of joining the program, if you're interested: 💬 Join other Qiskit ecosystem developers in the Slack channel #qiskit-ecosystem (if you are not in the Qiskit workspace yet, you can sign up here: https://qisk.it/join-slack) 🧠 Help us define the future of the Qiskit Ecosystem: we recently started an RFC on how the Ecosystem should be structured. It is very draft-y at the moment, but you can subscribe for notifications here Qiskit/RFCs#46 to participate in that discussion. 📣 Would you like to showcase your project to the Qiskit community? A couple of things you can do to raise awareness for your project:
Great having you as a part of the ecosystem! |
Thank you, @1ucian0! |
Github repo
https://github.com/atlytle/dense-ev
Description
Implements expectation value measurements in Qiskit using optimal dense grouping. Dense-ev provides an improvement of ~2^m over naive grouping and (3/2)^m over qubit-wise commuting groups. Based on arXiv:2305.11847.
Email
atlytle@gmail.com
Alternatives
No response
License
Apache License 2.0
Affiliations
University of Illinois at Urbana-Champaign
Tags
paper implementation
Website
https://github.com/atlytle/dense-ev
The text was updated successfully, but these errors were encountered: