From 2de24b3de4cb1e0f41cfeebae29af35453a1f8c6 Mon Sep 17 00:00:00 2001
From: ahmedk <karim.ahmed@xfel.eu>
Date: Tue, 5 Apr 2022 08:46:23 +0200
Subject: [PATCH] add comment on using get_run_value instead of as_single_value

---
 src/cal_tools/agipdlib.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/cal_tools/agipdlib.py b/src/cal_tools/agipdlib.py
index 86e42b145..ea077b5a4 100644
--- a/src/cal_tools/agipdlib.py
+++ b/src/cal_tools/agipdlib.py
@@ -202,11 +202,14 @@ class AgipdCtrl:
                 "highVoltage.actual.value")
             default_voltage = None
 
-
         if (
             voltage_src[0] in self.run_dc.all_sources and
             voltage_src[1] in self.run_dc.keys_for_source(voltage_src[0])
         ):
+            # Use RUN source for reading the bias voltage value.
+            # As HED_DET_AGIPD500K2G has a hardware issue that leads
+            # to storing arbitrary voltage values in the CONTROL source
+            # array. e.g. /gpfs/exfel/exp/HED/202230/p900248/raw
             return int(self.run_dc.get_run_value(*voltage_src))
         else:
             # TODO: Validate if removing this and
-- 
GitLab