[Jungfrau][Correct] fix: Skip exposure_timeout for gain constant retrieval

Description

• Remove exposure timeout from gain constants conditions
• use calcat_interface2

How Has This Been Tested?

• Will test the failed run from production :
• integration test

Relevant Documents (optional)

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@schmidtp @kluyvert @mramilli

changed title from fix: Skip exposure_timeout for gain constant retrieval to [Jungfrau][Correct] fix: Skip exposure_timeout for gain constant retrieval

Other than those things, LGTM

That was very fast and efficient. Thanks a lot Thomas

added 1 commit

• 9e51056b - fix: display ccv table and rename force_fixed_gain_constants

Compare with previous version

changed milestone to %3.15.5

marked the checklist item Will test the failed run from production as completed

changed the description

Apologies for missing this earlier. We had verified that it should not apply to gain constants for now, but I failed to realize this way it would still use it for querying of course...

I'm mildly unsettled by the large change done quickly. Shall we rebase this on top of master (if it isn't already) and run the entire test suite, or at least the affected detectors (LPD, JF, GH2)?

I think only JUNGFRAU is affected? The calcat_interface2 changes are in JUNGFRAUConditions. Or do you mean everything we've touched recently?

Yes, touched since the last deployment of the main branch.

Makes sense, yes. It does need rebasing, unfortunately.

added 21 commits

• 9e51056b...d6de823a - 19 commits from branch master
• 92f842c0 - fix: Skip exposure_timeout for gain constant retrieval
• 2ebc4a4a - fix: display ccv table and rename force_fixed_gain_constants

Compare with previous version

marked the checklist item integration test as completed

The tests are finished. LPD failed

So for darks, it looks like one of the new sanity checks failed. I'm a bit mystified by the failure in correct:

TypeError: correct_lpd_frames() takes at least 12 positional arguments (8 given)

The failed tests are fixed in the following MRs:

• !1113 (merged)
• !1093 (merged)

merged

mentioned in commit 638788ef