ID |
Date |
Author |
Category |
Subject |
50
|
Wed Jan 22 11:50:20 2020 |
Dinko Ferencek | Software | Module configuration files for interactive tests |
There are two module configuration folders set up for elComandante
/home/l_tester/L1_SW/pxar/data/tbm10d_procv3/
/home/l_tester/L1_SW/pxar/data/tbm10d_procv4/
These folder can also be used for interactive tests with pXar. However, in that case these folders get filled with pxar.log and pxar.root files. For some reason, elComandante when copying the module configuration files also copies all pxar.log files. To prevent unnecessary duplication of junk files, two new module configuration folders were set up for interactive tests with pXar
/home/l_tester/L1_SW/pxar/data/tbm10d_procv3_pxar/
/home/l_tester/L1_SW/pxar/data/tbm10d_procv4_pxar/
The configurations are identical to those used by elComandante. However, one needs to remember to keep the tests folders in sync with the elComandante folders once they get updated. This can be done as follows
rsync -avPSh /home/l_tester/L1_SW/pxar/data/tbm10d_procv3/ /home/l_tester/L1_SW/pxar/data/tbm10d_procv3_pxar/
rsync -avPSh /home/l_tester/L1_SW/pxar/data/tbm10d_procv4/ /home/l_tester/L1_SW/pxar/data/tbm10d_procv4_pxar/
To check if there are any extraneous pXar files present in the configuration folders, run
find /home/l_tester/L1_SW/pxar/data/tbm10d_procv?/ -type f \( -name 'pxar*.log' -o -name 'pxar*.root' \)
If any, they can be deleted by running
find /home/l_tester/L1_SW/pxar/data/tbm10d_procv?/ -type f -name \( -name 'pxar*.log' -o -name 'pxar*.root' \) -delete |
52
|
Thu Jan 23 15:15:45 2020 |
Dinko Ferencek | Software | Trimming Vcal value changed from 35 to 40 |
Trimming Vcal value changed from 35 to 40 in the testParameters.dat files stored in
/home/l_tester/L1_SW/pxar/data/tbm10d_procv3/
/home/l_tester/L1_SW/pxar/data/tbm10d_procv3_test/
/home/l_tester/L1_SW/pxar/data/tbm10d_procv4/
/home/l_tester/L1_SW/pxar/data/tbm10d_procv4_test/
This is the value we will use for the FullQualification. |
53
|
Fri Jan 24 18:12:00 2020 |
Dinko Ferencek | Software | MoReWeb code updates |
MoReWeb code updated to include the ROC wafer info in the XrayCalibration and XRayHRQualification pages:
https://gitlab.cern.ch/CMS-IRB/MoReWeb/commit/4de8ea39050600367ae0b8e959fcc00f29be45d8
In addition, two BB2 plots with wrong axis labels were fixed:
https://gitlab.cern.ch/CMS-IRB/MoReWeb/commit/3cfa18fc3aab1a29859ba9f0f81aa0ed4c59d5c8
https://gitlab.cern.ch/CMS-IRB/MoReWeb/commit/bb6627e6ab3f4d08eb54970a7e3d3b751983e15a |
108
|
Mon Mar 16 10:05:23 2020 |
Matej Roguljic | Software | PhQualification change |
Urs made a change in pXar, in the PhOptimization algorithm. One of the changes is in the testParameters.dat where vcalhigh is set to 100 instead of 255. This was implemented on the PC used to run full qualification. A separate procedure for elComandante was created, "PhQualification.ini", which runs pretest, pixelalive, trimming, ph and gainpedestal. This procedure will need to be run on all the modules qualified before this change was made and later merged with previous full qualification results. |
112
|
Wed Mar 18 15:16:47 2020 |
Andrey Starodumov | Software | Change in trimmig algorithm |
Urs yesterday modified the algorithm and tested it. From today we are using it. Main change that threshold for trimming is not any more fixed to VthrComp=79 but calculated based on the bottom tornado value + 20. CalDel is also optimised. This allows us to avoid failures in the trimbit tests due to too high threshold, eg if the bottom of tornado close to 70 a fraction of pixels in a chip could have threshold above 70 and hence fail in the test.
|
198
|
Wed Apr 8 15:02:37 2020 |
Andrey Starodumov | Software | Change of CrtlReg for RT |
So far in a blue box for RT tbm10d_procv3 parameters were used.
This is wrong, since CtrlReg=9 is better for -20C while for +10 or higher T
CtrlReg=17 is better.
From now on for RT tbm10d_procv4 will be used. |
243
|
Thu Apr 30 16:47:00 2020 |
Matej Roguljic | Software | MoReWeb empty DAC plots |
Some of the DAC parameters plots were empty in the total production overview page. All the empty plots had the number "35" in them (e.g. DAC distribution m20_1 vana 35). The problem was tracked down to the trimming configuration. Moreweb was expecting us to trim to Vcal 35, while we decided to trim to Vcal 50. I "grepped" where this was hardcoded and changed 35->50.
The places where I made changes:
- Analyse/AbstractClasses/TestResultEnvironment.py
'trimThr':35
- Analyse/Configuration/GradingParameters.cfg.default
trimThr = 35
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
TrimThresholds = ['', '35']
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
self.SubPages.append({"InitialAttributes" : {"Anchor": "DACDSpread35", "Title": "DAC parameter spread per module - 35"}, "Key": "Section","Module": "Section"})
It's interesting to note that someone had already made the change in "Analyse/Configuration/GradingParameters.cfg" |
244
|
Thu Apr 30 17:24:57 2020 |
Dinko Ferencek | Software | MoReWeb empty DAC plots |
Matej Roguljic wrote: | Some of the DAC parameters plots were empty in the production overview page. All the empty plots had the number "35" in them (e.g. DAC distribution m20_1 vana 35). The problem was tracked down to the trimming configuration. Moreweb was expecting us to trim to Vcal 35, while we decided to trim to Vcal 50. I "grepped" where this was hardcoded and changed 35->50.
The places where I made changes:
- Analyse/AbstractClasses/TestResultEnvironment.py
'trimThr':35
- Analyse/Configuration/GradingParameters.cfg.default
trimThr = 35
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
TrimThresholds = ['', '35']
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
self.SubPages.append({"InitialAttributes" : {"Anchor": "DACDSpread35", "Title": "DAC parameter spread per module - 35"}, "Key": "Section","Module": "Section"})
It's interesting to note that someone had already made the change in "Analyse/Configuration/GradingParameters.cfg" |
As far as I can remember, the changes in Analyse/AbstractClasses/TestResultEnvironment.py, Analyse/Configuration/GradingParameters.cfg.default and Analyse/Configuration/GradingParameters.cfg were there from before, probably made by Andrey. It is possible that you looked at the files when I was preparing logically separate commits affecting the same files which required temporarily undoing and later reapplying some of the changes to be able to separate the commits. The commits are now on GitLab https://gitlab.cern.ch/CMS-IRB/MoReWeb/-/commits/L1replacement, specifically:
435ffb98: grading parameters related to the trimming threshold updated from 35 to 50 VCal units
1987ff18: updates in the production overview page related to a change in the trimming threshold |
245
|
Thu Apr 30 17:33:04 2020 |
Andrey Starodumov | Software | MoReWeb empty DAC plots |
Matej Roguljic wrote: | Some of the DAC parameters plots were empty in the total production overview page. All the empty plots had the number "35" in them (e.g. DAC distribution m20_1 vana 35). The problem was tracked down to the trimming configuration. Moreweb was expecting us to trim to Vcal 35, while we decided to trim to Vcal 50. I "grepped" where this was hardcoded and changed 35->50.
The places where I made changes:
- Analyse/AbstractClasses/TestResultEnvironment.py
'trimThr':35
- Analyse/Configuration/GradingParameters.cfg.default
trimThr = 35
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
TrimThresholds = ['', '35']
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
self.SubPages.append({"InitialAttributes" : {"Anchor": "DACDSpread35", "Title": "DAC parameter spread per module - 35"}, "Key": "Section","Module": "Section"})
It's interesting to note that someone had already made the change in "Analyse/Configuration/GradingParameters.cfg" |
I have changed
1)StandardVcal2ElectronConversionFactorfrom 50 to 44 for VCal calibration of PROC600V4 is 44el/VCal
2)TrimBitDifference from 2 to -2 for not to take into account failed trim bit test that is an artifact from trimbit test SW. |
253
|
Thu May 7 00:10:15 2020 |
Dinko Ferencek | Software | MoReWeb empty DAC plots |
Andrey Starodumov wrote: |
Matej Roguljic wrote: | Some of the DAC parameters plots were empty in the total production overview page. All the empty plots had the number "35" in them (e.g. DAC distribution m20_1 vana 35). The problem was tracked down to the trimming configuration. Moreweb was expecting us to trim to Vcal 35, while we decided to trim to Vcal 50. I "grepped" where this was hardcoded and changed 35->50.
The places where I made changes:
- Analyse/AbstractClasses/TestResultEnvironment.py
'trimThr':35
- Analyse/Configuration/GradingParameters.cfg.default
trimThr = 35
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
TrimThresholds = ['', '35']
- Analyse/OverviewClasses/CMSPixel/ProductionOverview/ProductionOverviewPage/ProductionOverviewPage.py
self.SubPages.append({"InitialAttributes" : {"Anchor": "DACDSpread35", "Title": "DAC parameter spread per module - 35"}, "Key": "Section","Module": "Section"})
It's interesting to note that someone had already made the change in "Analyse/Configuration/GradingParameters.cfg" |
I have changed
1)StandardVcal2ElectronConversionFactorfrom 50 to 44 for VCal calibration of PROC600V4 is 44el/VCal
2)TrimBitDifference from 2 to -2 for not to take into account failed trim bit test that is an artifact from trimbit test SW. |
1) is committed in 74b1038e.
2) was made on Mar. 24 (for more details, see this elog) and is currently left in Analyse/Configuration/GradingParameters.cfg and might be committed in the future depending on what is decided about the usage of the Trim Bit Test in module grading
$ diff Analyse/Configuration/GradingParameters.cfg.default Analyse/Configuration/GradingParameters.cfg
45c45
< TrimBitDifference = 2.
---
> TrimBitDifference = -2.
There were a few other code updates related to a change of the warm test temperature from 17 to 10 C. Those were committed in 3a98fef8. |
255
|
Thu May 7 00:56:50 2020 |
Dinko Ferencek | Software | MoReWeb updates related to the BB2 test |
Andrey noticed that results of the BB2 test (here example for ROC 12 in M1675)
were not properly propagated to the ROC Summary
This was fixed in d9a1258a. However, looking at the summary for ROC 5 in the same module after the fix
it became apparent that dead pixels were double-counted under the dead bumps despite the fact they were supposed to be subtracted here. From the following debugging printout
Chip 5 Pixel Defects Grade A
total: 5
dead: 2
inef: 0
mask: 0
addr: 0
bump: 2
trim: 1
tbit: 0
nois: 0
gain: 0
par1: 0
total: set([(5, 4, 69), (5, 3, 68), (5, 37, 30), (5, 38, 31), (5, 4, 6)])
dead: set([(5, 37, 30), (5, 3, 68)])
inef: set([])
mask: set([])
addr: set([])
bump: set([(5, 4, 69), (5, 38, 31)])
trim: set([(5, 4, 6)])
tbit: set([])
nois: set([])
gain: set([])
par1: set([])
it became apparent that the column and row addresses for pixels with bump defects were shifted by one. This was fixed in 415eae00
However, there was still a problem with the pixel defects info in the production overview page which was still using using the BB test results
After switching to the BB2 tests results in ac9e8844, the pixel defects info looked better
but it was still not in a complete sync with the info presented in the FullQualification Summary 1
This is due to double-counting of dead pixels which still needs to be fixed for the Production Overview. |
256
|
Thu May 7 01:51:03 2020 |
Dinko Ferencek | Software | Strange bug/feature affecting Pixel Defects info in the Production Overview page |
It was observed that sometimes the Pixel Defects info in the Production Overview page is missing
It turned out this was happening for those modules for which the MoReWeb analysis was run more than once. The solution is to remove all info from the database for the affected modules
python Controller.py -d
type in, e.g. M1668, and when prompted, type in 'all' and press ENTER and confirm you want to delete all entries. After that, run
python Controller.py -m M1668
followed by
python Controller.py -p
The missing info should now be visible. |
260
|
Mon May 11 21:32:20 2020 |
Dinko Ferencek | Software | Fixed double-counting of pixel defects in the production overview page |
As a follow-up to this elog, double-counting of pixel defects in the production overview page was fixed in 3a2c6772. |
261
|
Mon May 11 21:37:43 2020 |
Dinko Ferencek | Software | Fixed the BB defects plots in the production overview page |
0407e04c: attempting to fix the BB defects plots in the production overview page (seems mostly related to the 17 to 10 C change)
f2d554c5: it appears that BB2 defect maps were not processed correctly |
262
|
Mon May 11 21:41:15 2020 |
Dinko Ferencek | Software | 17 to 10 C changes in the production overview page |
0c513ab8: a few more updates on the main production overview page related to the 17 to 10 C change |
265
|
Wed May 13 23:16:37 2020 |
Dinko Ferencek | Software | Fixed double-counting of pixel defects in the production overview page |
Dinko Ferencek wrote: | As a follow-up to this elog, double-counting of pixel defects in the production overview page was fixed in 3a2c6772. |
A few extra adjustments were made in:
38eaa5d6: also removed double-counting of pixel defects in module maps in the production overview page
51aadbd7: adjusted the trimmed threshold selection to the L1 replacement conditions |
270
|
Fri May 22 16:19:01 2020 |
Andrey Starodumov | Software | Change in MoreWeb GradingParameters.cfg |
Xray noise:
grade B moved from 300e to 400e
grade C moved from 400e to 500e |
273
|
Mon May 25 17:24:05 2020 |
Andrey Starodumov | Software | Change in MoreWeb ColumnUniformityPerColumn.py |
In file
~/L1_SW/MoReWeb/Analyse/TestResultClasses/CMSPixel/QualificationGroup/XRayHRQualification$ emacs Chips/Chip/ColumnUniformityPerColumn/ColumnUniformityPerColumn.py
the high and low rates at which double column uniformity is checked are hard coded. Rates for L2 was there. Now correction is added:
# Layer2 settings
# HitRateHigh = 150
# HitRateLow = 50
# Layer1 settings
HitRateHigh = 250
HitRateLow = 150 |
60
|
Mon Feb 3 15:53:32 2020 |
Andrey Starodumov | Reception test | 5 modules RT: 1545, 1547, 1548, 1549, 1550 |
Feb 03
- RT is done and OK for all modules. All 5 modules graded A. |
61
|
Mon Feb 3 17:09:36 2020 |
Andrey Starodumov | Reception test | 2 modules RT failed: 1544, 1546 |
Feb 03
- RT failed
-- 1544:
--- all ROCs are programmable
--- ROC14 no reliable Threshold found, ROC15 no threshold at all
-- 1546
--- all ROCs are programmable
--- no threshold found for all ROCs |