search

JeffersonLab / hcana

Hall C++ Analyzer
7 stars 118 forks source link

Tdc conv fix #411

Closed pooser closed 5 years ago

pooser commented 5 years ago

Remove multiple instances of the CAEN 1990 tdc to ns conversion factor and instead utilize single global parameter. Also remove instances of hard coded conversion factors.

This PR is dependent on the JeffersonLab/hallc_replay commit f78d9d2

pooser commented 5 years ago

@Yero1990 Because the parameter was defined as negative in hallc_replay.

MarkKJones commented 5 years ago

Talked with Eric and decided that it would be better to have the module determine the tdc conversion factor. When calling GetTime it would give the time in ns. But the hardcoding of the tdc channel to time in the THcCoinTime.cxx needs to be fixed. I will put in an modified version in a new commit.