Hi, Sorry for holding things up here. I have a first try of the new energy calibration--the one used to produce the dN/deta figure that Hiro posted yesterday-- and am now just testing to see that it works correctely in brat. There was a change in the PTQ pulser calibration function, so it requires a little more work than just adding a new calibration file. Christian is being kind-- I actually thought I could get this in last week, but I am confident that it will be done this week. Also, the calibration that I have is apparently based on an earlier BB vertex (the one in the current brat) which is apparently off by about 1.8 cm. I'll need to redo the calibration with the new vertex, although I don't expect this to have a large effect. --Steve Christian Holm Christensen wrote: > Hi Jens Ivar et al, > > On Wed, 3 Oct 2001 11:04:16 -0100 (GMT+1) > Jens Ivar Jordre <jens@fi.uib.no> wrote > concerning "Re: current BR_MULT_CAL_TMP setting": > >> On Thu, 27 Sep 2001, Christian Holm Christensen wrote: >> >> [....skip skip.....] >> >>> The one in BrMultCentCalibration was defined, the one in >>> BrMultCalibration was undefined. >> >> After several attempts to undefine BR_MULT_CAL_TMP in BrMultCalibration.h >> I'd like to know the file should be set. >> >> As of right now BrMultCalibration.h is set up with the following defs and >> undefs: >> #ifdef BR_MULT_CAL_TMP >> #undef BR_MULT_CAL_TMP >> #endif >> #define BR_MULT_CAL_TMP >> >> Trying to comment out the last line leads to error when generating ROOT >> dictionary in <brat top dir>/packages. The complaint is that in >> BrSiDigModule.h and BrTileDigModule.h the BrMultTmpCalibration is not >> defined. But this is just the way it should be when BR_MULT_CAL_TMP is >> undefined. Does anyone know why this "feature" should lead to error here? > > > The problem is, that rootcint, that is CINT, only expands preprocessor > macros _up_to_one_level_, and so in headers included at second or > higher level, the processor macros are not properly expanded. To me, > that's a limitation of CINT that is so annoying. The -P and +P flags > does not seem to work properly. > > >> Commenting out all four lines leads to even more trouble. Then the linker >> complains about undefined reference for Br<Si|Tile>TmpCalibration members >> and member functions. > > > Which I again believe is due to the limitation of CINT. > >> So my main question is therefore: What is it tha has to be commented out >> in order to use the database for the mult. single detector calibration and >> ASCII file(s) for the mult. centrality calibration? > > > What I did, and Steve later reversed, was to _remove_ all references > to Br[Si|Tile]TmpCalibration [1] in the TMA and SMA modules. That is > really what should be done. On a shorter time scale, you can do is to > move the lines > > #ifndef BRAT_BrTileRdoModule > #include "BrTileRdoModule.h" > #endif > #ifndef BRAT_BrSiRdoModule > #include "BrSiRdoModule.h" > #endif > > from BrGlbPackage.h to BrGlbPackage.cxx, and replace them in > BrGlbPackage.h with the lines > > class BrTileRdoModule; > class BrSiRdoModule; > > That should do the trick. Please note, that I believe the > calibrations from last year, those that are in the database, are not > valid for this years run. So if you want to analyse data from this > year, you'd better wait for Steve and Hiro to finish the > calibrations. I believe Steve promised us last week that we'll get > them this week, or was that just wishful thinking? > > As soon as Steve has finished with Br[Tile|Si]TmpCalibration, we will > remove all references, definitions, implmentations, ascii files, and > so on pertaining to those classes. It was a temporary hack introduced > by me to ease the transistion, not a solution. > > 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 > > [1] I generally use brackets ('[' and ']') to denote alternatives, and > angles ('<' and '>') to denote metaidentifiers. It's ofcourse a > matter of convention. Mine is taken from Donald E. Knuth.
This archive was generated by hypermail 2b30 : Wed Oct 03 2001 - 12:35:09 EDT