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.rstandAUTHORS.rsthave 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