5592160564
87%

Ran 18 Jul 2023 09:21PM UTC

Jobs 1

Files 1265

Run time 25s

Badge

Embed ▾

pending completion

Build # 5592160564

Build Type

push

github

Committed by

web-flow

Commit Message

Remove cyclic-definition handling from `Clifford.from_circuit` (#10441)

* Remove cyclic-definition handling from `Clifford.from_circuit`

This reliance on `RecursionError` could lead to CPython crashing, when
the user had raised the recursion limit beyond what their operating
system could actually support in terms of Python frames.  This was
particularly an issue with Windows when in a context that `jedi` is
active (such as in an IPython session, or if `seaborn` was imported),
since `jedi` unilaterally sets the recursion limit to 3000, while
CPython tends to overflow the stack on Windows at around 2700 frames.

Recursive `definition` fields are not valid data in the Qiskit model, as
the definition is supposed to be hierarchical, and a decomposition in
terms of gates that do not involve the current one in any form.  For
defining equivalences that may involve cycles, one should use the
`EquivalenceLibrary` objects that Terra manages, and the transpiler
takes advantage of via the `BasisTranslator`.

* Remove unused import

Run Details

72398 of 84137 relevant lines covered (86.05%)

45975.89 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	5592160564.1	18 Jul 2023 09:21PM UTC	0	86.05	GitHub Action Run

Source Files on build 5592160564

Detailed source file information is not available for this build.

Qiskit / qiskit-terra / 5592160564
87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5592160564

Qiskit / qiskit-terra / 5592160564 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 5592160564

Qiskit / qiskit-terra / 5592160564
87%

README BADGES
x