Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15036-7f641edc873829ad1ffd795765f769b215d4d5db
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-14210-6bf8f1c11ececae79aa0d0f505400452ae60e05e
branch: gh-readonly-queue/main/pr-14210-6bf8f1c11ececae79aa0d0f505400452ae60e05e
CHANGE BRANCH
x

coverage: 87.929% (+0.03%) from 87.896%
14639528421

push

github

web-flow 
Add custom `Qargs` implementation for `Target` (#14210)

* Initial: Add custom `Qargs` implementation for `Target`.

A custom implementation of `Qargs` was added to `Target` allowing us to directly fix an ownership issue that was currently being solved by `NullableIndexMap`. These additions will ensure the following:
- Seamless conversion from Python `tuple[int]` and Rust `SmallVec<[PhysicalQubit; 2]>`.
- Usage of `Option<&[PhysicalQubit]>` as a valid query for the `Target`.

* Refactor: `TargetQargs` is now `Qargs`.
- `TargetQargsRef` is now `QargsRef`.

* Fix: Use generics for querying into the `Target`.
 Use a `pub use` to export `Qargs`, and `QargsRef` directly from the `target_transpiler` module.
 - Add additional conversion to `&Qargs` to `PyTuple`.
 - Add `From<[T; N]>` for `Qargs` and `QargsRef`.
 - Enable use of generics to query into the `Qargs` present in `Target`. These are based on `Into<QargRef>` and include `&Qarg`, `&[PhysicalQubit]` and others.

* Fix: Make `Qargs` and `QargsRef` more lean
- Remove unnecessary methods from `Qargs` as pointed out by @jakelishman.
- Remove macro derives `Default`, `Ord`, and `PartialOrd` as they had incorrect semantic implications.
- Added more generic `From` and `FromIterator` implementations to make usage more versatile.

* Fix: `update_instruction_properties` signature.

* Fix: update signatures for `for_qargs` methods.

* Refactor: Remove wrongful use of underscore prefixed names.
- Fix docstrings documenting `Qargs` as `TargetQargs`.

* Fix: Address review comments
- `extract_target_basis` uses `SmallVec` instead of `Qargs`.
- Use`let Some(val) = conscript else { // }` pattern.
- Fix docstring for `py_qargs_for_operation_name`.
- Move `pub use` of qargs to top of document.

* Docs: Remove stray comment

223 of 280 new or added lines in 11 files covered. (79.64%)

7 existing lines in 2 files now uncovered.

74140 of 84318 relevant lines covered (87.93%)

436103.83 hits per line

Relevant lines Covered
84318 RELEVANT LINES 74140 COVERED LINES
436103.83 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14210-6bf8f1c11ececae79aa0d0f505400452ae60e05e

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
14639528421 gh-readonly-queue/main/pr-14210-6bf8f1c11ececae79aa0d0f505400452ae60e05e Add custom `Qargs` implementation for `Target` (#14210) * Initial: Add custom `Qargs` implementation for `Target`. A custom implementation of `Qargs` was added to `Target` allowing us to directly fix an ownership issue that was currently being s... push web-flow github
87.93
See All Builds (14018)