Qiskit / qiskit / 23207675357

88%

main: 87%

Build Type

push

github

Committed by web-flow

Commit Message

SynthesizeRZRotations pass  (#15641)

* SynthesizeRZRotations pass (WIP)

* formatting fixes

* suggested changes to synthesize_rz_rotations.rs

* changes to synthesize_rz_rotations.py

* changes to synthesize_rz_rotations.py

* changes to release notes

* change tau to two_pi

* added more functions for canonicalization and included code for reusing synthesis of same angles

* canonical representation corrections

* removes square brackets

* added a param test

* angle approximation logic implemented and epsilon reduced

* documentation added and epsilon modified to approximation degree

* added more tests

* change to t_count test

* fix lint and doc issues

* empty-commit-deleted-empty-lines-to-rerun-checks

* lint fixes

* header changes

* typo correct

* reorder pass import in init file

* add to autosummary

* documentation corrections

* epsilon-related modifications

* edited comments

* release note correction

* make clippy happy

* minor test edits

* documentation corrections

* improve angle canonicalization test

* lint fix

* doc changes/ remove debug assert

* Add description to doc

* fix pylint expression not assigned issue for list comprehension

* fix comments

* use Result instead of PyResult & anyhow context to covert/wrap error for Result. Also convert panic! to unreachable! since ross_sellinger handles return of nonstandard gates

* convert back to PyErr at call

* formatting change

* undo changes

* separate rust and python calls for gridsynth_rz

* pyres to res changes

* pauli rotns fixed

* fix panic and unwrap with expect

* formatting changes

* Adding a path to specify error budget for synthesis and for caching separately

* Shortening the description of the pass based on review comments

* Updating (and renaming) release note

* minor tweak of pass docstring

* clippy

* typos mentioned in the review

* adding tests for cache_error and synthesis_error

* fixing merge conflict with main

* Small fixes on metrics & tes... (continued)

Run Details

104 of 106 new or added lines in 7 files covered. (98.11%)

1 existing line in 1 file now uncovered.

101263 of 115729 relevant lines covered (87.5%)

1166955.1 hits per line