Skip to content

Conversation

@perlinm
Copy link
Contributor

@perlinm perlinm commented Dec 15, 2024

I overlooked this in #96. Sorry.

@perlinm perlinm requested a review from ciaranra as a code owner December 15, 2024 04:45
Copy link
Member

@ciaranra ciaranra left a comment

Choose a reason for hiding this comment

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

No worries. LGTM (besides the annoying regression tests)

@ciaranra ciaranra merged commit 07aafb7 into PECOS-packages:development Dec 15, 2024
17 checks passed
@perlinm perlinm deleted the unroll-face-rots branch December 15, 2024 23:40
@perlinm
Copy link
Contributor Author

perlinm commented Dec 16, 2024

Thanks for handling the regression tests!

ciaranra added a commit to ciaranra/PECOS-alt that referenced this pull request May 25, 2025
* Unroll broadcast face rotations (PECOS-packages#110)

* Upgrade to using uv instead of pip (PECOS-packages#111)

* Adding uv to Makefile

* Turn on artifact generation (PECOS-packages#117)

* Add T gates with active error correction (PECOS-packages#118)

* remove t_tel_cor

* bugfix in tdg_cor

* fix t_cor

* Permute all variables of the Steane code (PECOS-packages#114)

* compute the swap of CRegs

* Add Steane-type and teleportation-based QEC for the Steane code (PECOS-packages#115)

* fix qec_steane_z

* set flag bit correctly

* add qec_steane_.*tel templates

* fix Steane.qec_steane_tel

* permute all qubits for teleportation

* fix teleportation-based QEC

* two flag bits for Steane.qec_tel

* rename flag_bit --> flag, in agreement with elsewhere in the repo

* fix If statement

* fix Steane QEC and an incorrect type hint

---------

Co-authored-by: Michael A. Perlin <[email protected]>
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