Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: use-fprintf-over-printf-15097
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-14837-cde5f55d843ab8e932df6d3cf0380856382d3f2d
branch: gh-readonly-queue/main/pr-14837-cde5f55d843ab8e932df6d3cf0380856382d3f2d
CHANGE BRANCH
x

coverage: 88.249% (-0.05%) from 88.295%
18168441810

push

github

web-flow 
Add ``QkParam`` to the C API (#14837)

* Add C-API for ParameterExpression

* add test cases for Parameter C-API

* rename qk_paramter_ to qk_param_, fix array format of bind/subs functions

* fix test for param

* use existing QkParam to save results of operations

* fix test_param.c

* fix test_param.c

* fix for lint, add releasenote

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* Update crates/cext/src/param.rs

Co-authored-by: Julien Gacon <gaconju@gmail.com>

* add qk_param_zero(), rename qk_param_new to qk_param_new_symbol

* remove unsafe from qk_param_zero

* rename ``qk_param_str`` for consistency

compare to ``qk_obs_str``

* Cleanup tests

- use goto for shorter code
- handle QkExitCode from arithmetic functions
- add docstrings

* Doc chores

- proper code highlighting
- fix C code sections
- fix safety sections
- update exit code docs

* Docs & small improvements for bind/subs

* More tests

* Update reno

* Jake's review comments - light edition

the others are for follow ups :)

* drop bind/subs for now

there are open discussions on what the right interface to reduce the allocations in practice. Let's unblock parameters while we have this discussion

* one more go at safer print

* update reno

* fix test artifact

* fix rust edition 2024

* don't use VLA (thanks MSVC)

* rm string prints

* rm ctype

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Julien Gacon <jules.gacon@googlemail.com>

240 of 347 new or added lines in 3 files covered. (69.16%)

6 existing lines in 3 files now uncovered.

93132 of 105533 relevant lines covered (88.25%)

1161819.3 hits per line

Relevant lines Covered
105533 RELEVANT LINES 93132 COVERED LINES
1161819.3 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14837-cde5f55d843ab8e932df6d3cf0380856382d3f2d

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
18168441810 gh-readonly-queue/main/pr-14837-cde5f55d843ab8e932df6d3cf0380856382d3f2d Add ``QkParam`` to the C API (#14837) * Add C-API for ParameterExpression * add test cases for Parameter C-API * rename qk_paramter_ to qk_param_, fix array format of bind/subs functions * fix test for param * use existing QkParam to save res... push web-flow github
88.25
18166340539 gh-readonly-queue/main/pr-14837-cde5f55d843ab8e932df6d3cf0380856382d3f2d Add ``QkParam`` to the C API (#14837) * Add C-API for ParameterExpression * add test cases for Parameter C-API * rename qk_paramter_ to qk_param_, fix array format of bind/subs functions * fix test for param * use existing QkParam to save res... push web-flow github
88.25
See All Builds (13942)