Compensated integration time for Qblox QRM IC component

Explanation of changes

Integration time is now compensated for when retrieving integrated acquisition.

Motivation of changes

Integration data returns

V_{int} = \sum_{n=0}^N V[n]

with N being the integration length. This is changed to

\widetilde{V}_{int} = \dfrac{1}{N}\sum_{n=0}^N V[n]

since now the returned data simply has units volt which is more intuitive and is in agreement with the rest of quantify assumes this already (previously the units were \mathrm{nVs} due the the 1 GHz sample rate). Changes tested to work on hardware.


Merge checklist

See also merge request guidelines

  • Merge request has been reviewed and approved by a project maintainer.
  • Merge request contains a clear description of the proposed changes and the issue it addresses.
  • Merge request made onto appropriate branch (develop for most MRs).
  • New code is fully tested.
  • New code is documented and docstrings use numpydoc format.
  • CHANGELOG.rst and AUTHORS.rst have been updated (when applicable).
  • CI pipelines pass
    • black code-formatting passes (gitlab-ci),
    • test suite passes (gitlab-ci),
    • no degradation in code-coverage (codacy),
    • no (serious) new pylint code quality issues introduced (codacy),
    • documentation builds successfully (CI and readthedocs),
    • windows tests pass (manually triggered by maintainers before merging).

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

Edited by Victor Negîrneac

Merge request reports

Loading