quil-lang / quilc

The optimizing Quil compiler.
Apache License 2.0
452 stars 73 forks source link

remove RANDOM from FIND-DIAGONALIZER-IN-E-BASIS #848

Closed stylewarning closed 1 year ago

stylewarning commented 1 year ago

fixes #847

braised-babbage commented 1 year ago

This works for me. The role of the RNG previously was just to break possible degeneracies, so making it deterministic (but still allowing the same number of attempts) is an improvement. We can still consider alternative strategies for doing the canonical decomposition in the future.