1
89%
main: 88%

Ran 03 Sep 2024 03:25PM UTC

Files 874

Run time 57s

Badge

Embed ▾

Committed 03 Sep 2024 02:56PM UTC coverage: 89.175% (+0.04%) from 89.139%

Job # 10685536623.1

Build Type

push

github

Committed by

web-flow

Commit Message

Avoid `ExtraInstructionAttributes` allocation on `unit="dt"` (#13078)

The default value for `Instruction.unit` is `"dt"`.  Previously, the
`OperationFromPython` extraction logic would only suppress allocation of
the extra instruction attributes if all the contained fields were
`None`, but `None` is not actually a valid value of `Instruction.unit`
(which must be a string).  This meant that `OperationFromPython` would
always allocate and store extra attributes, even for the default cases.
This did not affect standard gates appended using their corresponding
`QuantumCircuit` methods (since no Python-space extraction is
performed in that case), but did affect standard calls to `append`, or
anything else that entered from Python space.

This drastically reduces the memory usage of circuits built by
`append`-like methods. Ignoring the inefficiency factor of the
heap-allocation implementation, this saves 66 bytes plus
small-allocation overhead for 2-byte heap allocations (another 14 bytes
on macOS, but will vary depending on the allocator) per standard
instruction, which is on the order of 40% memory-usage reduction.

Run Details

72464 of 81260 relevant lines covered (89.18%)

405879.13 hits per line

Qiskit / qiskit / 10685536623 / 1
89%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10685536623.1

Qiskit / qiskit / 10685536623 / 1 89% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10685536623.1

Qiskit / qiskit / 10685536623 / 1
89%
main: 88%

README BADGES
x