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

Bug Fix: Addressing #37 #38

Closed bdorney closed 7 years ago

bdorney commented 7 years ago

Description

Addressing https://github.com/cms-gem-daq-project/gem-plotting-tools/issues/37

Now the overlay_fit() function in anaUltraScurve.py will take the event count as an input and use it for the fit.

Types of changes

Motivation and Context

If someone used the -b option the fit defined in overlay_fit() would not be correct for scurves taken with event counts different from the default.

How Has This Been Tested?

I re-ran an old anaysis:

anaUltraScurve.py -i$DATA_PATH/GE11-VI-L-CERN-0001/scurve/current/SCurveData.root --ztrim=4 -b -f

Screenshots (if appropriate):

screen shot 2017-09-14 at 15 23 53 screen shot 2017-09-14 at 11 36 35

Checklist:

cbravo135 commented 7 years ago

See my comment on issue #37 about this.