Qiskit / qiskit
89%
main: 89%

LAST BUILD BRANCH: BD-add-alt-text
DEFAULT BRANCH: main
Repo Added
Files 913
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-13032-7c409123a369c7df0b86f99ae00fefd41bc142fd
branch: gh-readonly-queue/main/pr-13032-7c409123a369c7df0b86f99ae00fefd41bc142fd
CHANGE BRANCH
x

coverage: 89.135% (+0.001%) from 89.134%
10741843551

push

github

web-flow 
Add method to add instructions to a DAGCircuit from an iterator of PackedInstruction (#13032)

* Initial: Add add_from_iter method to DAGCircuit
- Introduce a method that adds a chain of `PackedInstruction` continuously avoiding the re-linking of each bit's output-node until the very end of the iterator.
   - TODO: Add handling of vars
- Add header for a `from_iter` function that will create a `DAGCircuit` based on a chain of `PackedInstruction`.

* Fix: leverage new methods in layers
- Fix incorrect re-insertion of last_node.

* Fix: Keep track of Vars for add_from_iter
- Remove `from_iter`

* Fix: Incorrect modification of last nodes in `add_from_iter`.
- Use `entry` api to either modify or insert a value if missing.

* Fix: Cycling edges in when adding vars.
- A bug that adds duplicate edges to vars has been temporarily fixed. However, the root of this problem hasn't been found yet. A proper fix is pending. For now skip those instances.

* Fix: Remove set collecting all nodes to be connected.
- A set collecting all the new nodes to connect with a new node was preventing additional wires to connect to subsequent nodes.

* Fix: Adapt to #13033

* Refactor: `add_from_iter` is now called `extend` to stick with `Rust` nomenclature.

* Fix docstring

- Caught by @ElePT

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>

* Fix: Remove duplicate vars check

* Fix: Corrections from code review.
- Use Entry API to modify last nodes in the var.
- Build new_nodes with an allocated vec.
- Add comment explaining the removal of the edge between the output node and its predecessor.

* Fix: Improper use of `Entry API`.
- Use `or_insert_with` instead of `or_insert` to perform actions before inserting a value.

---------

Co-authored-by: Elena Peña Tapia <57907331+ElePT@users.noreply.github.com>

110 of 110 new or added lines in 1 file covered. (100.0%)

17 existing lines in 4 files now uncovered.

72721 of 81585 relevant lines covered (89.14%)

388325.2 hits per line

Relevant lines Covered
81585 RELEVANT LINES 72721 COVERED LINES
388325.2 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-13032-7c409123a369c7df0b86f99ae00fefd41bc142fd

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
10741843551 gh-readonly-queue/main/pr-13032-7c409123a369c7df0b86f99ae00fefd41bc142fd Add method to add instructions to a DAGCircuit from an iterator of PackedInstruction (#13032) * Initial: Add add_from_iter method to DAGCircuit - Introduce a method that adds a chain of `PackedInstruction` continuously avoiding the re-linking of ... push web-flow github
89.14
See All Builds (9023)