Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: fix-consolidate-blocks-recursion
DEFAULT BRANCH: main
Repo Added
Files 868
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-14799-e9df76a761dda6febaa86e322d7b3e701ff7a234
branch: gh-readonly-queue/main/pr-14799-e9df76a761dda6febaa86e322d7b3e701ff7a234
CHANGE BRANCH
x

coverage: 87.625% (-0.03%) from 87.65%
16787758326

push

github

web-flow 
Python-free path for circuit parameters (#14799)

* Split py and py-free methods

* playing with Symbol as Parameter

-- 606 failing tests

* fix errors

* about to start the mess

* suite passing

* PVE in Rust

* start assign

* add assign

* dealing with unknown params and conflicts

* an attempt at vector fixing

* fix more tests

- name conflicts
- hash(2.3) == hash(ParamExpr(2.3))
- some error messages

* implement name_map as track record

This allows keeping track of which parameters are still in the expression, even though the expression is optimized. E.g. x*0 = 0, but x is still valid to bind.

* The string debacle

- don't use UUID in to_string, only in string_id (using strings for comparisons should probably be avoided in future)
- clippy
- relax Python is-checks to eq-checks

* some housekeeping

* implement _qpy_replay

* implement sympify

* fix copy/deepcopy

* Implement pickle via QPY and Parameter.assign to return Parameter

* fix template matching

this is rather a temporary solution, maybe we ought to disable string parsing in favor or a proper SymPy->parameter expression conversion

* fix rhs - lhs mixup

* fix more tests on binding {x: 0*y}

* fix last failing tests (see numeric(strict=False))

* clippy

* a lot of docs

* rename PyParameterExpression to ParameterExpression

* Thou shan't fake hyperrefs

* start to pull the thread on Param::ParameterExpression

* proper split: (Py)ParameterExpression

* rm pyclass from Symbol

* TODO's and no more RwLock

- rm Arc<RwLock<..>> for inner, since expressions are immutable and not shared, this is not needed
- address a bunch of todos, that include
  - rm rsub/rdiv/rpow
  - introduce try_to_value and _to_symbol
  - make internal data private

* First round of review comments

mainly removing some unnecessary clones, and fixing Hash for ParameterExpression

* rm `name_map` from PyParameterExpression

--- and just reconstruct the Py symbols from ParameterExpression.name_map

* Fix vector... (continued)

610 of 717 new or added lines in 23 files covered. (85.08%)

39 existing lines in 7 files now uncovered.

81814 of 93368 relevant lines covered (87.63%)

547255.04 hits per line

Relevant lines Covered
93368 RELEVANT LINES 81814 COVERED LINES
547255.04 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14799-e9df76a761dda6febaa86e322d7b3e701ff7a234

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
16787758326 gh-readonly-queue/main/pr-14799-e9df76a761dda6febaa86e322d7b3e701ff7a234 Python-free path for circuit parameters (#14799) * Split py and py-free methods * playing with Symbol as Parameter -- 606 failing tests * fix errors * about to start the mess * suite passing * PVE in Rust * start assign * add assign * de... push web-flow github
87.63
See All Builds (13862)