Add computation of std of drawer temperatures to the DQM

[alessandromontanari]

feat(src/nectarchain/dqm):

• camera_monitoring.py: add computation of std of temperature for each drawer

This PR adds the computation of the std of drawer temperatures to the processor camera_monitoring.py, following the same code logic as for the temperature mean.

[codecov]

Codecov Report

❌ Patch coverage is 3.92157% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.09%. Comparing base (bcb16b2) to head (8b6cc70).

Files with missing lines	Patch %	Lines
src/nectarchain/dqm/camera_monitoring.py	3.92%	49 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #309      +/-   ##
==========================================
- Coverage   49.33%   49.09%   -0.25%     
==========================================
  Files          85       85              
  Lines        7795     7840      +45     
==========================================
+ Hits         3846     3849       +3     
- Misses       3949     3991      +42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[hashkar]

Looks good to me.

[hashkar]

All is ok to me for the changes in the DQM. Please ensure that all automatic checks have passed (the ones that are possible) and green light for merging.

[jlenain]

All is ok to me for the changes in the DQM. Please ensure that all automatic checks have passed (the ones that are possible) and green light for merging.

Hi @hashkar , @alessandromontanari ,

Many thanks for this PR, @alessandromontanari , and your review, @hashkar !
I just first would like to process a DQM run locally, to check that everything runs fine, since the DQM code is not fully covered with unit tests.

[hashkar]

@jlenain @alessandromontanari also 310 should have been in the same PR as this one and tested. I will test 310.

[jlenain]

@jlenain @alessandromontanari also 310 should have been in the same PR as this one and tested. I will test 310.

Well, to me it is OK to have two separate PRs: I am rather an adept of "two different features introduced, two different pull requests".

[jlenain]

At the moment, a local DQM processing gives exactly the same plots for the mean and standard deviation:

The same applies between mean and std when taking the temperature sensors "FEB1" or "FEB2" individually.

I think I see why, I'll fix it.

[hashkar]

@jlenain problem is in line 23, 248 and 264 it is still plotting the mean. Sorry I missed it.

[alessandromontanari]

@jlenain @alessandromontanari also 310 should have been in the same PR as this one and tested. I will test 310.

Well, to me it is OK to have two separate PRs: I am rather an adept of "two different features introduced, two different pull requests".

I tend to prefer this logic too

[jlenain]

@jlenain problem is in line 23, 248 and 264 it is still plotting the mean. Sorry I missed it.

OK, much better now: