[AGIPD] Read bias_voltage for AGIPD at SPB and MID from RUN source

get_bias_voltage() read bias_voltage for HED_DET_AGIPD500K2G only.

Description

Start using SPB_DET_AGIPD1M/PSC/HV/channels.U0.measurementSenseVoltage.value and MID_DET_AGIPD1M/PSC/HV/channels.U0.measurementSenseVoltage.value to ready bias voltage for SPB_DET_AGIPD1M-1 and MID_DET_AGIPD1M-1.

How Has This Been Tested?

Relevant Documents (optional)

Types of changes

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

Checklist:

• I have updated the documentation accordingly.
• I added tests where appropriate. A part of a different MR: https://git.xfel.eu/detectors/pycalibration/-/merge_requests/646

Reviewers

@calibration

changed the description

added 1 commit

• ac3dff5c - only go to default value for SPB and MID and correct PSC source

Compare with previous version

changed the description

changed the description

changed title from Read bias_voltage for AGIPD at SPB and MID from RUN source to [AGIPD] Read bias_voltage for AGIPD at SPB and MID from RUN source

added Waiting for review label

changed milestone to %3.5.1

added 30 commits

• ac3dff5c...ea1b0f06 - 28 commits from branch master
• 632a1228 - Read bias_voltage for AGIPD at SPB and MID from RUN source
• ecfbb724 - only go to default value for SPB and MID and correct PSC source

Compare with previous version

added 1 commit

• 2de24b3d - add comment on using get_run_value instead of as_single_value

Compare with previous version

OK, this looks fine to me, then. If it's meant as a temporary change, there's no point thinking too hard about the best way to design the check.

Thank you @kluyvert for the review :)

resolved all threads

merged

removed Waiting for review label

mentioned in commit 477189d2