Re: current BR_MULT_CAL_TMP setting

From: Stephen J. Sanders (ssanders@falcon.cc.ukans.edu)
Date: Thu Sep 27 2001 - 12:22:50 EDT

  • Next message: Christian Holm Christensen: "Re: current BR_MULT_CAL_TMP setting"

    Hi Christian,  
    Please hold off a little in removing the ascii files.  I still don't have
    the mysql stuff going and I would like to first get the calibration changes
    going in a familiar setting before switching over.
    
    
    
    ...steve
    
    > From: Christian Holm Christensen <cholm@hehi03.nbi.dk>
    > Date: Thu, 27 Sep 2001 18:13:07 +0200
    > To: brahms-dev-l@bnl.gov, ssanders@KU.EDU
    > Subject: Re: current  BR_MULT_CAL_TMP setting
    > 
    > Hi Steve and others,
    > 
    > On Tue, 25 Sep 2001 22:37:36 -0500
    > "Stephen J. Sanders" <ssanders@ku.edu> wrote
    > concerning "current  BR_MULT_CAL_TMP setting":
    >> Hi again,
    > 
    >> It just occurred to me that in submitting the "small" changes to
    >> correct cases where the fCalibration variable was defined improperly
    >> when using the ascii file calibration, ...
    > 
    > The member was in fact _not_ defined incorrectly.  Since the parameter
    > values has been input to the MySQL database at rcas0005.rcf.bnl.gov, I
    > saw little point in carrying around the _temporary_ ASCII files
    > anymore, and therefor removed the relevant code from the compilation.
    > This _only_ affected the single detector calibrations for TMA and
    > SMA.  The MA Centrality calibration was not affected by this in
    > anyway. 
    > 
    >> I left BR_MULT_CAL_TMP defined in the uploaded files.  I was going
    >> to undefine the variable, but discovered that it is actually defined
    >> in two places: BrMultCalibration.h and BrMultCentCalibration.h.
    > 
    > The reason for the same define in both of these files was probably
    > some mistake (on mine or someone elses part - I can't remember).
    > Anyway, since only the module BrMultCentModule uses
    > BrMultCentCalibration, and no other calibrations, there's right now no
    > reason to have BR_MULT_CAL_TMP defined in BrMultCalibration.  That is,
    > for all single detector calibrations, we should use the MySQL database
    > and not the ASCII files.
    > 
    > For the MA centrality calibration, we still need the ASCII file,
    > mainly because the issue of to which detector to attribute that
    > calibration is not resolved yet.  I trust Steve, Hiro, and Flemming
    > will figure that out at some point.
    > 
    >> In the brat version that I downloaded this afternoon, before
    >> starting to work on the program, one of these defines was active and
    >> the other was commented out.
    > 
    > The reason being stated above.
    > 
    >> Unfortunately, I don't remember which one was which.
    > 
    > The one in BrMultCentCalibration was defined, the one in
    > BrMultCalibration was undefined.
    > 
    >> Rather than guess on the status, I'm leaving both defined as of now.
    > 
    > As soon as this European Graduate Collage is over, I'll make the
    > nessecary changes, including the removal of the ASCII files from
    > asciidata/mult, since they are no longer relevant.
    > 
    >> I have the new energy calibration of the Si and tile detectors and
    >> hope to upload this calibration, as an ascii file, tomorrow.
    > 
    > Looking forward to that.  As soon as you have the pedestal, gap,
    > pulser, and gain calibrations we can do the SDE centrality
    > calibrations. 
    > 
    > May I suggest you take a look at some of the stuff in my CVS area,
    > subdir jobs/calib.  There's a few utils for commiting stuff to the
    > MySQL db. 
    > 
    > Yours, 
    > 
    > Christian Holm Christensen -------------------------------------------
    > Address: Sankt Hansgade 23, 1. th.           Phone:  (+45) 35 35 96 91
    > DK-2200 Copenhagen N                Cell:   (+45) 28 82 16 23
    > Denmark                             Office: (+45) 353  25 305
    > Email:   cholm@nbi.dk                        Web:    www.nbi.dk/~cholm
    > 
    



    This archive was generated by hypermail 2b30 : Thu Sep 27 2001 - 12:23:39 EDT