[d2n-analysis-talk] d2n BB replay (BBDecData problem)

[Brad Sawatzky]

Hi all,

It looks like the most important historical difference between
THaDecData and BBDecData was that the latter had been modified to
support reference-time subtraction for F1 TDCs -- which we don't use.
The problem with the 'stock' THaDecData is that it spews copies of the
HRS VDC efficiency histograms whether you want them or not.

I compile a THaDecData class with the VDC stuff removed.  (That stuff
should really be moved to a macro anyway...)  

Diana and Matt:  Please test out the patched analyzer by pointing your
  env variables at my test analyzer.  If you followed the recommended
  approach in '/usr/local/d2n_analysis/root-setup.sh', then just change
  the line at the top to
    export d2n_root="/home/brads/d2n_analysis_copy"

Let me know if it works (or not).  If it's all good, then I'll make it
our standard analyzer and disable the BBDecData class.

-- Brad

On Tue, 09 Mar 2010, Brad Sawatzky wrote:

> Yikes.  Good catch.
> 
> Unfortunately I don't have time to debug this today...  If someone wants
> to dig into it, I would start with the diff between the BBDecData.cxx
> and THaDecData.C source:
> 
>   diff -burN  /usr/local/d2n_analysis/d2n/bigbitelib/BBDecData.cxx /usr/local/d2n_analysis/analyzer/src/THaDecData.C
> 
> At one point BBDecData.cxx was copy-paste duplicate of THaDecData.C, but
> they have diverged over the years.  Most of the changes in the diff are
> due to my recent removal of the VDC efficiency histo generation stuff
> from BBDecData.  That was useless for BigBite and was causing a ton of
> duplicate histos to generated due to a long standing bug.
> 
> There are other differences too though -- those aren't mine.  Perhaps
> folks fixed a bug in THaDecData but didn't port the patch to the
> BBDecData class.  Might be worth doing a 'resync' between the two files.
> 
> In the meantime, everyone should be very suspicious of the BBDecData
> variables.
> 
> -- Brad
> 
-- 
Brad Sawatzky, PhD <brads at jlab.org>  -<>-  Jefferson Lab / Hall C / C111
  Ph: 757-269-5947 -<>- Pager: 757-584-5947 -<>- Fax: 757-269-5235
The most exciting phrase to hear in science, the one that heralds new
  discoveries, is not "Eureka!" but "That's funny..."   -- Isaac Asimov