Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: commutation-check-paulis
DEFAULT BRANCH: main
Repo Added
Files 880
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-14958-77c79fcab58790a19816ebe05865c5eab8735d50
branch: gh-readonly-queue/main/pr-14958-77c79fcab58790a19816ebe05865c5eab8735d50
CHANGE BRANCH
x

coverage: 88.207% (-0.003%) from 88.21%
19510335870

push

github

web-flow 
Add C API testing framework to Python tests (#14958)

* Add C API testing framework to Python tests

Right now the Qiskit C API (and by extension the internal rust code) is
lacking some of the helper utilities necessary to do deeper validation
of the output circuits. Mainly `Operator.from_circuit` to do unitary
equivalence checks accounting for a layout introduced by the transpiler.
But also things like just circuit equality checks are easier in Python
because the tooling doesn't exist in pure-Rust/C yet. As we're building
the C API to be the universal interface to Qiskit from any programming
language it is very easy to leverage the C API from Python.

This commit adds a new testing framework to the Python tests that builds
a C FFI using ctypes to call into the loaded compiled extension (which
already bundles the C API). It then wraps that ffi in a couple of helper
functions to go from a Python circuit to a C circuit, build a c target
from basis gates and a coupling map, and then a function to transpile a
c circuit using the C api which returns a Python circuit. This enables
us to write tests in Python and validate the output circuits are
equivalent accounting for layout and permutations the transpiler
applies. It also lets us use any other tooling we have in Python for
testing.

This framework does not eliminate the need for
testing the C API directly from C, it is only intended to do deeper
testing while the functionality exposed to C is more limited
specifically for the transpiler.

The tests added in this PR are just a starting point and basically are
copies of Python space full path various transpilation tests. This is
not expected to be the final set of tests, but more just serve as a
starting point.

* Add safety comment

* Add missing return types to ctypes interface

* Fix ctypes interface def order

* Add helper function for building c arrays

* Add free on cleanup to tests

* Update test/python/c_api/ffi.py

Co-authored-by: Julien Gacon <ga... (continued)

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

33 existing lines in 4 files now uncovered.

94324 of 106935 relevant lines covered (88.21%)

1143176.87 hits per line

Relevant lines Covered
106935 RELEVANT LINES 94324 COVERED LINES
1143176.87 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14958-77c79fcab58790a19816ebe05865c5eab8735d50

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
19510335870 gh-readonly-queue/main/pr-14958-77c79fcab58790a19816ebe05865c5eab8735d50 Add C API testing framework to Python tests (#14958) * Add C API testing framework to Python tests Right now the Qiskit C API (and by extension the internal rust code) is lacking some of the helper utilities necessary to do deeper validation of ... push web-flow github
88.21
See All Builds (14864)