Skip to content

Ensure S-bit rate scan and DAC scan completeness and reliability

Laurent Petre requested to merge bugfix/scans-completeness into main

Description

The title says it all... The second commit message is quite explicit on the MR goal:

Originally, the S-bit rate vs THR_ARM_DAC scan and DAC scan were made robust against communication instabilities by trying to extract as much reliable data as possible. Partial scans would occur, either by skipping some points or by stopping the scan partway. Experienced proved that such scans are difficult - and useless - to analyze.

This commit instead ensures that only complete scans are returned to the user and thus saved. Partial scans are simply considered failed, similar to what is implemented for the calibration pulse scan. The scans should be faster in the presence of communication instabilities and easier to analyze.

The first commit is a technical change to ensure compatibility with the new firmware behavior regarding the cached VFAT ADC reads. See emu/0xbefe!157 (merged) for more details.

Related Issue

How Has This Been Tested?

No setup is currently available. Our test bed will be the production system at P5. 😕

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Merge request reports