Updated anaXDAQLatency.py to be mutli link and include more distributions - Githubissues

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

Updated anaXDAQLatency.py to be mutli link and include more distributions #177

Closed bdorney closed 5 years ago

bdorney commented 5 years ago

Description

Options menu now uses argparse instead of OptionParser
Now returns sensible exit codes when error occurs
Distributions written to $ELOG_PATH/runXXXXXX where run number is determined from input filename
Now adds the possibility to apply an external mapping file to have the output int terms of readout strip instead of ASIC channel number.
Additional output distributions are created
Summary distributions have their own canvas and creates 3-by-8 grid of per VFAT distributions for a given observable.
Can process multipe links per call instead of just one link now.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context

Quality of life improvements and additional observable desired.

How Has This Been Tested?

Repeatedly by analyzing new data produced by ldqm.

Screenshots (if appropriate):

See mattermost.

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

bdorney commented 5 years ago

I did a rebase off of develop.