Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-15239-3ca582178839d0a7b01437752b293ab4e78baf5c
DEFAULT BRANCH: main
Repo Added
Files 872
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/stable/2.2/pr-15231-e06b870d75a41d23ed6d153cd64583456d157e8c
branch: gh-readonly-queue/stable/2.2/pr-15231-e06b870d75a41d23ed6d153cd64583456d157e8c
CHANGE BRANCH
x

coverage: 88.42% (+0.01%) from 88.408%
18712519024

push

github

web-flow 
Document subclass considerations for `Target` (#15220) (#15231)

* Document subclass considerations for `Target`

The documentation of `Target` previously implied that subclassing it is
ok, and we have users that have done that.  However, subclassing it is
fragile, does not affect any transpiler pass that reads from the
`Target` in Rust space, and requires the subclass to arrange for
`__new__` to be called correctly.  I re-added a `**kwargs` variadic in
`Target.__new__` as a minor convenience; subclasses that only add
keyword-only arguments (which are generally the safest things to add)
will remain functional.  Had `Target.__new__` been defined in terms of
keyword-only arguments, we could also have added an `*args` munch too,
but as it is, adding or modifying the positional arguments to a `Target`
subclass constructor _always_ requires overriding `__new__` as well to
avoid clobbering positional `description`, `num_qubits`, etc, so there's
no benefit to having `*args`.

We should be discouraging users from subclassing `Target`; even our own
subclass of the Python-space `Target` from the Rust-space `Target` is
rather fragile with how we attempt to keep data in sync between the two.
Most APIs exposing a `Target` to their users would probably be better as
constructor functions; this is supposed to be the default way of
building a `Target`.

* Reexplain constructors vs subclassing

---------


(cherry picked from commit b6484afde)

Co-authored-by: Jake Lishman <jake.lishman@ibm.com>
Co-authored-by: Shelly Garion <46566946+ShellyGarion@users.noreply.github.com>

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

10 existing lines in 3 files now uncovered.

92391 of 104491 relevant lines covered (88.42%)

539354.87 hits per line

Relevant lines Covered
104491 RELEVANT LINES 92391 COVERED LINES
539354.87 HITS PER LINE
Source Files on gh-readonly-queue/stable/2.2/pr-15231-e06b870d75a41d23ed6d153cd64583456d157e8c

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
18712519024 gh-readonly-queue/stable/2.2/pr-15231-e06b870d75a41d23ed6d153cd64583456d157e8c Document subclass considerations for `Target` (#15220) (#15231) * Document subclass considerations for `Target` The documentation of `Target` previously implied that subclassing it is ok, and we have users that have done that. However, subclass... push web-flow github
88.42
See All Builds (14207)