return truncated acquisition sample size (QAE-292)
Explanation of changes
Use integration_length param of QRMAcquisitionManager
in qblox instrument coordinator file to truncate returned acquisitions to actual length. Also fix bug where every QRMAcquisitionManager
stored only one integration_length (as opposed to 6, one per sequencer). Renamed integration_length
param of QRMAcquisitionManager
to acq_duration
as it is now not only used for integration type acq's anymore
Motivation of changes
Currently the number of samples returned when an acquisition is performed is always 16384 (for QBLOX hardware), there is no way for a user to specify less samples. Samples are directly correlated to acq duration as sampling time is constant, therefore it should be possible to return the number of samples actually measured.
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
andAUTHORS.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.