Skip to content

Move qubit management transformers from Cirq-FT to Cirq-core#6319

Merged
tanujkhattar merged 3 commits intoquantumlib:masterfrom
tanujkhattar:move_qubit_allocation_transformer_to_cirq
Oct 18, 2023
Merged

Move qubit management transformers from Cirq-FT to Cirq-core#6319
tanujkhattar merged 3 commits intoquantumlib:masterfrom
tanujkhattar:move_qubit_allocation_transformer_to_cirq

Conversation

@tanujkhattar
Copy link
Copy Markdown
Collaborator

@tanujkhattar tanujkhattar commented Oct 17, 2023

@tanujkhattar tanujkhattar requested review from a team, cduck and vtomole as code owners October 17, 2023 23:13
@CirqBot CirqBot added the size: L 250< lines changed <1000 label Oct 17, 2023
@tanujkhattar tanujkhattar enabled auto-merge (squash) October 17, 2023 23:26
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 18, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (105d975) 97.89% compared to head (36f7cc0) 97.89%.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #6319      +/-   ##
==========================================
- Coverage   97.89%   97.89%   -0.01%     
==========================================
  Files        1106     1108       +2     
  Lines       96180    96192      +12     
==========================================
+ Hits        94158    94169      +11     
- Misses       2022     2023       +1
Files Coverage Δ
cirq-core/cirq/__init__.py 100.00% <ø> (ø)
cirq-core/cirq/transformers/__init__.py 100.00% <100.00%> (ø)
...transformers/qubit_management_transformers_test.py 100.00% <100.00%> (ø)
...t/cirq_ft/algos/qubitization_walk_operator_test.py 100.00% <100.00%> (ø)
...-ft/cirq_ft/algos/reflection_using_prepare_test.py 100.00% <100.00%> (ø)
...-ft/cirq_ft/infra/qubit_management_transformers.py 100.00% <100.00%> (+1.88%) ⬆️
...irq_ft/infra/qubit_management_transformers_test.py 100.00% <100.00%> (ø)
...cirq/transformers/qubit_management_transformers.py 96.36% <96.36%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tanujkhattar tanujkhattar merged commit 96b3842 into quantumlib:master Oct 18, 2023
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
@tanujkhattar
Move qubit management transformers from Cirq-FT to Cirq-core (quantum…
f540ddc 
…lib#6319)

* Move qubit management transformers from Cirq-FT to Cirq-core

* Fix mypy error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NoureldinYosri NoureldinYosri NoureldinYosri approved these changes

@vtomole vtomole Awaiting requested review from vtomole

@cduck cduck Awaiting requested review from cduck

@pavoljuhas pavoljuhas Awaiting requested review from pavoljuhas pavoljuhas was automatically assigned from quantumlib/cirq-maintainers

Assignees

@mpharrigan mpharrigan

@fdmalone fdmalone

Labels

size: L 250< lines changed <1000

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@tanujkhattar @NoureldinYosri @mpharrigan @fdmalone @CirqBot