You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 31, 2022. It is now read-only.
Last week, issues related to incorrectly setting the IREF values were discovered at QC7. After a more detailed study (thanks Nimantha!) and a code review different issues were found.
This issue aim at improving the correctness, robustness and efficiency of testConnectivity.py by redefining its purpose.
Types of issue
Bug report (report an issue with the code)
Feature request (request for change which adds functionality)
Issues found
Incorrect IREF configuration for the Scurves taken during testConnectivity.py
confChamber.py is not guaranteed to configure the chamber in the same way as testConnectivity.py
DB access mandatory; if VFAT without Reed-Muller encoded chipID are used, the calibration constants can be retrieved for the wrong VFAT in case of bitflip
VFATs missing in DB seems not to be properly reported
The analysis of the Scurves taken during testConnectivity.py does not produce any output; they must be analyzed a second time to get the plots
Expected Behavior/Proposal
Historically, testConnectivity.py was designed to make the pass/failed decision about on the quality control of a chamber during QC7. However, the QC7 tests were improved and complexed over time leading to other scripts doQC7.sh and doQC7_complete.sh, both using testConnectivity.py.
Considering this, below is a list of changes we agreed on with Nimantha. They would fulfill all the requirements of QC7 and remote tests stands without VFAT3 hybrid v3 and/or DB access.
In summary, testConnectivity.py must be able to completely prepare a new chamber on every test stands while ensuring that confChamber.py correctly configure the chamber. After the chamber configuration, all acquisitions must run in the best conditions.
Whenever the DAC scans are successful, all DAC values - including IREF so - must be written to the CTP7.
Instead of modifying the chamber configuration on the fly, testConnectivity.py should warn that the VFAT configuration does not use the recommended values. This should ensure that confChamber.py will configure the chamber in the same way as testConnectivity.py while allowing for experimentation with different parameters.
Provide a DB (default) and non-DB mode (--noDB flag):
In non-DB mode, zero DB access is made. The the information must have been retrieved from the DB before launching (through a first call to testConnectivity.py or through getCalInfoFromDB.py) or manually (for experimentation and/or VFAT3 hybrid v2).
In DB mode, all the information is always retrieved from the DB. No missing VFAT is allowed. If it was to happen, the user can use getCalInfoFromDB.py first.
Remove the Scurve acquisition from the testConnectivity.py script. The acquisition and analysis can be added to doQC7.sh or to doQC7_complete.sh (only on the the QC7 DAQ machine) for easy QC7 operation.
Before anyone starts putting significant effort into this, let me stress that this change should be made post migration of framework.
If there is effort floating around wanting to work on the task in this issue, it should be redirected to ensuring that the migration is completed first.
AndrewLevin
pushed a commit
to AndrewLevin/vfatqc-python-scripts
that referenced
this issue
May 19, 2020
Brief summary of issue
Last week, issues related to incorrectly setting the
IREF
values were discovered at QC7. After a more detailed study (thanks Nimantha!) and a code review different issues were found.This issue aim at improving the correctness, robustness and efficiency of
testConnectivity.py
by redefining its purpose.Types of issue
Issues found
IREF
configuration for the Scurves taken duringtestConnectivity.py
confChamber.py
is not guaranteed to configure the chamber in the same way astestConnectivity.py
testConnectivity.py
does not produce any output; they must be analyzed a second time to get the plotsExpected Behavior/Proposal
Historically,
testConnectivity.py
was designed to make the pass/failed decision about on the quality control of a chamber during QC7. However, the QC7 tests were improved and complexed over time leading to other scriptsdoQC7.sh
anddoQC7_complete.sh
, both usingtestConnectivity.py
.Considering this, below is a list of changes we agreed on with Nimantha. They would fulfill all the requirements of QC7 and remote tests stands without VFAT3 hybrid v3 and/or DB access.
In summary,
testConnectivity.py
must be able to completely prepare a new chamber on every test stands while ensuring thatconfChamber.py
correctly configure the chamber. After the chamber configuration, all acquisitions must run in the best conditions.IREF
so - must be written to the CTP7.testConnectivity.py
should warn that the VFAT configuration does not use the recommended values. This should ensure thatconfChamber.py
will configure the chamber in the same way astestConnectivity.py
while allowing for experimentation with different parameters.vfatqc-python-scripts/testConnectivity.py
Lines 964 to 975 in 49d1b6a
--noDB
flag):testConnectivity.py
or throughgetCalInfoFromDB.py
) or manually (for experimentation and/or VFAT3 hybrid v2).getCalInfoFromDB.py
first.testConnectivity.py
script. The acquisition and analysis can be added todoQC7.sh
or todoQC7_complete.sh
(only on the the QC7 DAQ machine) for easy QC7 operation.Your Environment
The text was updated successfully, but these errors were encountered: