Qiskit / qiskit
90%
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-12087-cc7c47dc7b435eee9055cd2ec41f3e0b28481b15
branch: gh-readonly-queue/main/pr-12087-cc7c47dc7b435eee9055cd2ec41f3e0b28481b15
CHANGE BRANCH
x

coverage: 89.644% (+0.01%) from 89.631%
8940031400

push

github

web-flow 
Adapt crates/qasm3 to work with recent versions of openqasm3_parser (#12087)

* Adapt crates/qasm3 to work with recent versions of openqasm3_parser

This commit adds no new features or capabilities to the importer. But it brings
the importer up to date with the latest version of openqasm3_parser as a preliminary
step in improving the importer.

The biggest change in openqasm3_parser is in handling stdgates.inc.  Upon encountering
`include "stdgates.inc" openqasm3_parser reads no file, but rather adds symbols to the
symbol table for gates in the standard library.

The function `convert_asg` in the importer calls oq3_semantics::symbols::SymbolTable.gates
which returns a `Vec` of information about each "declared" gate. The information is the
gate's name and signature and SymbolID, which is sufficient to do everything the importer
could do before this commit.

Encountering `Stmt::GateDefinition` now is a no-op rather than an error.  (This was
previously called `Stmt::GateDeclaration`, but importantly it is really a definition.)

How the standard library, and gates in general, are handled will continue to evolve.

A behavioral difference between this commit and its parent: Before if the importer
encountered a gate call before the corresponding definition an error would be raised. With
the current commit, the importer behaves as if all gate definitions were moved to the top
of the OQ3 program. However, the error will still be found by the parser so that the
importer never will begin processing statements.

The importer depends on a particular branch of a copy of openqasm3_parser. When
the commit is ready to merge, we will release a version of openqasm3_parser and
depend instead on that release.

See https://github.com/openqasm/openqasm/pull/517

* Remove unnecessary conversion `name.to_string()`

Response to reviewer comment https://github.com/Qiskit/qiskit/pull/12087/files#r1586949717

* Remove check for U gate in map_gate_ids

* This requires modifying th... (continued)

14 of 27 new or added lines in 2 files covered. (51.85%)

12 existing lines in 3 files now uncovered.

62257 of 69449 relevant lines covered (89.64%)

231164.78 hits per line

Relevant lines Covered
69449 RELEVANT LINES 62257 COVERED LINES
231164.78 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-12087-cc7c47dc7b435eee9055cd2ec41f3e0b28481b15

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
8940031400 gh-readonly-queue/main/pr-12087-cc7c47dc7b435eee9055cd2ec41f3e0b28481b15 Adapt crates/qasm3 to work with recent versions of openqasm3_parser (#12087) * Adapt crates/qasm3 to work with recent versions of openqasm3_parser This commit adds no new features or capabilities to the importer. But it brings the importer up to... push web-flow github
89.64
See All Builds (9023)