Remove normalization in virtual AwgOffsetStrategy

Timo van Abswouderequested to merge
remove-renormalization-virtual-awgoffsetstrategy into main

Explanation of changes

Removes the renormalization by qasm_program.static_hw_properties.max_awg_output_voltage in qblox.operation_handling.virtual.AwegOffsetStrategy.

Motivation of changes

This renormalization caused an exception when compiling an AWG offset, since !652 (merged) made the default renormalization constant None. Since in !652 (merged), similar renormalizations were removed from other modules in operation_handling, I believe removing it here as well is the correct approach, but please correct me if it's not.

Merge checklist

See also merge request guidelines

  • Merge request has been reviewed (in-depth by a knowledgeable contributor), and is approved by a project maintainer.
  • New code is covered by unit tests (or N/A).
  • New code is documented and docstrings use numpydoc format (or N/A).
  • New functionality: considered making private instead of extending public API (or N/A).
  • Public API changed: added @deprecated (or N/A).
  • Tested on hardware (or N/A).
  • CHANGELOG.md and AUTHORS.md have been updated (or N/A).
  • Windows tests in CI pipeline pass (manually triggered by maintainers before merging).

For reference, the issues workflow is described in the contribution guidelines.

Edited by Edgar Reehuis

Merge request reports