Skip to content

ScheduleGettable formats data with trigger_count acquisition nicely

Konstantin Lehmannrequested to merge
schedulegettable-formats-trigger-count into main

Explanation of changes

An additional if statement checks the acquisition mode and treats trigger_count acquisition differently. Only one array is returned. The previous check for gettable labels assumed that acquisitions are I/Q values. This check is removed now.

This MR has been tested on a development branch including several non-merged features. Don't merge before:

Motivation of changes

Introduction of the TriggerCount acquisition mode requires ScheduleGettable to format the data differently than with I/Q measurements.

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