cms-gem-daq-project / gem-plotting-tools

Repository for GEM commissioning plotting tools
GNU General Public License v3.0
1 stars 26 forks source link

Fix the vfatID array in the DAC scan analysis #263

Closed AndrewLevin closed 4 years ago

AndrewLevin commented 4 years ago

The VFAT ID array is now passed through np.unique such it will only have 24 entries, one for each VFAT.

Description

Previously the VFAT ID array was thousands of entries long, because there is one entry in the DAC scan tree for each DAC value.

Types of changes

Motivation and Context

The commissioning team found that VFATs on different detectors appeared to have the same vfat ID.

How Has This Been Tested?

Yes, I ran over the DAC scan root file provided by @bregnery and got the expected output.

Screenshots (if appropriate):

Checklist: