fix(compare_maps): slice ccvs correctly and some other small fixes (!46) · Merge requests · calibration / calephant

Karim Ahmed requested to merge fix/slice_timestamps_by_num_constants into master May 27, 2024

Description

This MR addresses and fixes the issue of incorrect data being associated with the wrong timestamps during the loading of calibration data. Problem

Previously, the code had a bug that resulted in the incorrect number of calibration constants (CCVs) being attached for timestamps for each module. This issue led to confusing plots where, if one module missed some CCVs for certain timestamps, it would appear that all other modules lacked data for other timestamps. This caused false interpretations of the data's availability and trends.

This fix improves the accuracy and reliability of the calibration trends plots, ensuring that users can trust the visualized data to reflect the true availability and trends of calibration constants across all modules.

Changes Made

Screenshots

Related Issues

Tests

Additional Notes

Merge Request Checklist

Code follows project coding guidelines.
Documentation reflects the changes made.
Unit tests are added or updated to cover the changes.
The application runs without errors and the new functionality works as expected.

Edited May 27, 2024 by Karim Ahmed

fix(compare_maps): slice ccvs correctly and some other small fixes