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

Rust code #77

Closed
wants to merge 123 commits into from
Closed

Rust code #77

wants to merge 123 commits into from

Conversation

ciaranra
Copy link
Member

@ciaranra ciaranra commented Sep 16, 2024

Including Rust code for stabilizer simulator.

TODO:

  • Have CI workflow that checks Rust tests and formatting
  • Update Makefiles to build etc. both Rust and Python projects given restructuring
  • Have CI workflow that builds and install pecos-rslib and tests pecos-rslib
  • Have CI workflow that tests the usage of pecos-rslib from Python PECOS
  • Add PyO3 wrapping for Rust simulators and make available to Python via pecos-rslib
  • Update tests to test Rust simulators used in Python PECOS
  • Add RSSparseSim to Python test_random_circuits (likely need to add force_outcome measurement)
  • Add Rust tests for the newly added gates for the Rust stabilizer sim
  • Update/modify various Readmes as appropriate...
  • Use symlinks to point to the appropriate legal and Readme stuff
  • Have CI workflow that is generates sdists and wheels for all the Python packages
  • Add version consistency checks for Python packages and Rust crates
  • Update workflow to save artifacts only on success merges, direct pushes, or py- tags in main branches
  • Update README.md if needed

ciaranran and others added 30 commits October 3, 2018 11:31
…a structures so params always given when looping over circuits.
…run both LogicalCircuits and QuantumCircuit. Applying errors to both.
…pper. Minor improvements to Standard and SparseSim.
@ciaranra ciaranra force-pushed the feat-rust branch 12 times, most recently from fd69fd6 to 6db2c7b Compare October 17, 2024 02:23
@ciaranra
Copy link
Member Author

Mostly just tried to work through the development section of the README.

make test-all gave me:

======== short test summary info ========
FAILED docs/api_guide/circuit_runners.rst::circuit_runners.rst
FAILED docs/api_guide/decoders.rst::decoders.rst
FAILED docs/api_guide/error_generators.rst::error_generators.rst
FAILED docs/api_guide/qeccs.rst::qeccs.rst
FAILED docs/api_guide/simulators.rst::simulators.rst
FAILED docs/examples/creating_qecc_class.rst::creating_qecc_class.rst
FAILED docs/examples/stab_code_verification.rst::stab_code_verification.rst
======== 7 failed, 2 passed, 1 warning in 0.82s ========

This should be resolved now. Just removing doc tests for now.

@ciaranra
Copy link
Member Author

Alright, @qartik & @peter-campora. I have updated the makefile to make the .venv more of a "first class citizen" and updated the README appropriately (hopefully). Let me know if you still have issues.

I still need to figure out what is going on with this:

-pecos-rslib==0.6.0.dev6
+pecos-rslib==version = 0.6.0.dev6

@ciaranra ciaranra force-pushed the feat-rust branch 3 times, most recently from 67bae29 to 5b76d77 Compare November 24, 2024 05:01
@ciaranra ciaranra force-pushed the feat-rust branch 3 times, most recently from 444c290 to d9a3b11 Compare November 27, 2024 16:35
@ciaranra ciaranra closed this Nov 27, 2024
@ciaranra ciaranra deleted the feat-rust branch November 27, 2024 22:51
@ciaranra ciaranra mentioned this pull request Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants