Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15036-7f641edc873829ad1ffd795765f769b215d4d5db
DEFAULT BRANCH: main
Repo Added
Files 871
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/main/pr-14421-c9ad20cd7df88e5b694f909f0a1ea428aaf4983c
branch: gh-readonly-queue/main/pr-14421-c9ad20cd7df88e5b694f909f0a1ea428aaf4983c
CHANGE BRANCH
x

coverage: 87.763% (+0.02%) from 87.748%
16146286725

push

github

web-flow 
Move classical variables and stretches to CircuitData (#14421)

* Add groundwork for vars storage in CircuitData

Adding basic support for vars and identifier storage and access, using
the same approach implemented in DAGCircuit with `ObjectRegsitry` and
various var-types structs. In addition updating `QuantumCircuit` to use
`CircuitData` for input vars.

* Handle captured and declared vars in QuantumCircuit

* Move stretch to CircuitData

* Handle vars & stretches in Rust in dag_to_circuit

* Add pickle/unpickle support + minor misc. stuff

* Use CircuitData'a vars/stretches natively in QuantumCircuitData

* Clone interners in CircuitData::copy_empty_like

* Add scoping rule checks to CircuitData

This commit adds checks to CircuitData::add_var and CircuitData::add_stretch
functions to enforce variable scoping rules. It also includes minor refactoring
and miscellaneous cleanups.

* Update BlueprintCircuit & twirling and cleanup

* Add unit tests. Some refactoring and cleanups

* Minor touches

* Lint and comment

* Fix metadata handling following changes in high_level_synthesis.rs

Since this PR no longer uses a custom function to convert a circuit to DAG in
run_high_level_synthesis, we should use DAGCircuit::from_circuit
with QuantumCircuitData initialized with both name and metadata from
the original DAG.

* Address some of Jake's review comments

Relax equality checking to allow variable order independence (modulo the declared stretches).
Minor refactoring.

* Use the `VarsMode` enum in Python's `copy_empty_like`

Following PR #14557, this commit switches to using the `VarsMode` enum
in the Pyo3 function `CircuitData::copy_empty_like`.
Additionaly, since the enum is now used both by `CircuitData` and `DAGCircuit`
and in various placed, it has been moved to the crate level.

* Update crates/circuit/src/circuit_data.rs

Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com>

* Update qiskit/circuit/quantumcircuit.py

Co-authore... (continued)

570 of 604 new or added lines in 11 files covered. (94.37%)

25 existing lines in 7 files now uncovered.

81609 of 92988 relevant lines covered (87.76%)

547445.01 hits per line

Relevant lines Covered
92988 RELEVANT LINES 81609 COVERED LINES
547445.01 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14421-c9ad20cd7df88e5b694f909f0a1ea428aaf4983c

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
16146286725 gh-readonly-queue/main/pr-14421-c9ad20cd7df88e5b694f909f0a1ea428aaf4983c Move classical variables and stretches to CircuitData (#14421) * Add groundwork for vars storage in CircuitData Adding basic support for vars and identifier storage and access, using the same approach implemented in DAGCircuit with `ObjectRegsit... push web-flow github
87.76
See All Builds (14018)