Qiskit / qiskit / 14639528421 / 1
88%
main: 88%

LAST BUILD BRANCH: substitute_pi4_rotations
DEFAULT BRANCH: main
Ran
Files 802
Run time 1min
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
coverage: 87.929% (+0.03%) from 87.896%
14639528421.1

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

74140 of 84318 relevant lines covered (87.93%)

436103.83 hits per line

Source Files on job 14639528421.1