Index: trunk/MagicSoft/Mars/Changelog =================================================================== --- trunk/MagicSoft/Mars/Changelog (revision 7183) +++ trunk/MagicSoft/Mars/Changelog (revision 7188) @@ -19,6 +19,362 @@ While an underscore is a placeholder for a white-space or an empty line. - -*-*- END OF LINE -*-*- + + 2005/07/13 Abelardo Moralejo (2005/07/12) + + * manalysis/MGeomApply.cc + - Now, if the MGeomApply task has a serial number >0 (indicating + most likely a specific telescope in a multi-telescope file), then + apply the geometry only to objects with the same serial number. + This is important for stereo setups in which the telescopes have + cameras with different numbers of pixels. If the MGeomApply task + has serial number 0 (default), it will apply the geometry to all + found objects as it used to do. + + * manalysis/MMcCalibrationUpdate.cc, mpedestal/MMcPedestalCopy.cc + - Now the PreProcess adds the serial number also to MRawRunHeader + and MMcRunHeader, since in stereo MC files produced with the + current camera simulation at CVS there is one such container per + telescope. There is no difference in behaviour for single + telescope files. Old MC stereo files will no longer be readable. + + * mimage/MNewImagePar.cc + - Made calculation of "Inner size" valid also for camera geometries + other than MAGIC standard one. The calculation of "inner leakage" + is now made only for MAGIC standard geometry (there is no easy + fix for that, but having fInnerSize and MHillas.fSize, the inner + leakage is a completely unnecessary parameter). Added a comment on + a possible fix for the calculation of fConc. + + * mcalib/MMcCalibrationCalc.[h,cc] + - Added member fMinSize and setter for it. It is the minimum + (uncalibrated) size of events to be used in the calibration. + Previously it was fixed at 1000 ADC counts, but the value depends + on the extractor used, so it must be possible to change it. + - Add the serial number to MRawRunHeader and MMcConfigRunHeader + (now there is one per telescope in stereo files). + - Moved the creation of histograms from the constructor to the + PreProcess, otherwise the serial number is not added to the + histogram name. + - In PostProcess, set the average QE for area 0 (inner pixels) in + MCalibrationQECam object, to be used later by MCalibrateData. + + * macros/starmcstereo.C + - Big update. Adapted to all changes above. Now not only relative, but + also absolute calibration (=SIZE in phes) is made. + + + + 2005/07/13 Raquel de los Reyes (2005/07/12) + + * merpp.cc + - Added the container MCameraActiveLoad to the rootified central + control files + + * mreport/MReportCC.[h,cc] + - Changes in the InterpreteBody to write the variable UPS status and + the Time difference between GPS and rubidium clock. + + * mreport/MReportCamera.cc + - Changes in the InterpreteCamera and InterpreteBody functions to allow + the data merpping from 2005_06_30. + + + + 2005/07/13 Abelardo Moralejo (2005/07/12) + + * macros/starmcstereo.C + - Fixed error which made some objects to be written twice in the + _test_ output file. + + + + 2005/07/13 Patricia Liebing (2005/07/06) + + * mhcalib/MHCalibrationChargeCam.cc: + - include new flag (kLoGainBlackout) into the "uncalibrated" + pixel classification (see docu from 28/06/2005 Markus Gaug) + + * mbadpixels/MBadPixelsPix.[cc.h], mcalib/MCalibrationChargeCalc.cc + - included flag kLoGainBlackout for unsuitable pixels w/ + corresponding docu/printout (see MHCalibrationChargeCam.cc) + + * mhcalib/MHCalibrationPulseTimeCam.cc, MHCalibrationCam.cc + - add Pixel ID to Debug printout statements + + * mjobs/MJCalibration.cc + - include new display for kLoGainBlackout unsuitable pixels + + + + 2005/07/13 Antonio Stamerra (2005/07/04) + + * manalysis/MMcTriggerLvl2Calc.cc + - the checks written on ReInit for MC have been moved to PreProcess() + - kFALSE substituted with kSKIP in the check for the standard MAGIC + geometry + + + + 2005/07/13 Abelardo Moralejo (2005/07/01) + + * macros/starmc.C + - Set two different cleaning tasks for the calibration loop and + for the second (analysis) loop. Otherwise, some pixels were + cleaned out in the first loop (we do not want this) if the + absolute method was chosen. + + + + 2005/07/13 Abelardo Moralejo (2005/06/30) + + * mmc/MTriggerDefine.h + - Added TRIGGER_PIXELS_4, a provisional number of trigger pixels + for the MGeomCamMagic1183 geometry. + + * mgeom/MGeomCamMagic1183.[h,cc] + - added. Possible design for a MAGIC-II camera. Preliminary + version, not yet ready to use. + + + + 2005/07/13 Abelardo Moralejo (2005/06/29) + + * mgeom/MGeomCamMagic1183.[h,cc] + - added. Possible design for a MAGIC-II camera + + * mgeom/Makefile, GeomLinkDef.h + - added new class above. + + + + 2005/07/13 Markus Gaug (2005/06/28) + + * mhcalib/MHCalibrationChargeCam.[h,cc] + - set the fPickupLimit and fBlackoutLimit for the low-gain arrays + to a 3.5 instead of the default used in MHCalibrationPix (5.0). + - use a new criterium to exclude bad pixels: If the high-gain was + saturated and the blackout-events in the low-gain exceed the + fNumLoGainBlackoutLimit, the pixel is declared unsuitable. + This excludes those pixels which have a saturating high-gain + channel, but the low-gain switch does not switch often enough + to make the distribution reliable. + + + + 2005/07/13 Markus Gaug (2005/06/23) + + * mhcalib/MHCalibrationChargeCam.cc + - fix a bug counting the number of saturated events. Up to now, the + number of saturated slices was counted (which is one for a not too + high number), but for some (pathological) pixels, many more slices + saturated and produced wrong limits. + - make the PickupLimit und BlackoutLimit for the low-gain explicitely + smaller. The too large limits caused a failed fit for some channels + which resulted in wrong calibration constants. (M. Gaug) + + + + 2005/07/13 Markus Gaug (2005/06/22) + + * mjobs/MJPedestal.cc + - change order of reading local environmnet variables in lines 1044 + to 1068. Reason: The lines: + if (fExtractor==tenv.GetTask()) + return kTRUE; + yielded sometimes a return before the rest of the env-variables + could be read in. This affected ONLY the reading of the pedestal + reference file and the bad pixels file name. (M. Gaug) + - Added "decode" task (MTriggerPatternDecode) task to the list also + for MC runs. + + * mhcalib/MHCalibrationChargeCam.cc + - fixed the setting of unreliability for saturated events where the + high-gain signal was not stable over time. Now, saturation is + asked for before (like in the high-gain fitted-case). (M. Gaug) + - fixed a small bug in the precise setting of the histogram ranges. + May have introduced a bias of typically 1 FADC cnt in the mean + fitted signal (< 0.5%). + + * mbadpixels/MBadPixelsPix.[h,cc] + - took out the un-used bad pixels information in function: + GetUnsuitableCalLevel(). + - debugged the class-documentation + + * mcalib/MCalibrationIntensityChargeCam.cc + - a very small fix in the axis title from N_phe to N_{phe} + - fixed one bug in the draw-functions of this class. Not (yet) used by + standard calibration. + - +some undocumented changes + + * mjobs/MJCalibration.cc + - changed order in tasklist between MCalibColorSet and + MPedCalcFromPedRun. Caused crashes in the intensity calibrations. + Does not affect ordinary calibration since MPedCalcFromPedRun is + then not in the tasklist anyhow. + - changed order of MHCalibrationChargeCam and MCalibrationRelTimeCalc. + This wrong order caused MCalibrationRelTimeCalc to print out + unreliable pixels which were not set by the rel. time calibration. + This caused some confusion in the reading of the output because of + the un-logical order of flag settings and posterior output. + - set the error of the hi-logain intercalibration constants + as the real error on the mean instead of the sigma. + - took out the those bad pixel-informations which are not used + any more in the "Defect" - display. (M. Gaug) + + + + 2005/07/13 Markus Gaug (2005/06/21) + + * mhcalib/MHCalibrationChargePix.h + - take out one un-necessary 'virtual'-declaration for the Draw- + function + + * mhcalib/MHGausEvents.cc + - fixed a small bug in the non-standard display with options. + + + + 2005/07/13 Markus Gaug (2005/06/19) + + * msignal/MExtractTimeAndChargeSpline.cc + - set fgOffsetLoGain from 1.7 to 1.39. This is the correct time + shift between high-gain and low-gain. Affects only very detailed + timing studies. + + * msignal/MExtractTimeAndChargeDigitalFilter.cc + - set fOffsetLoGain from 1.7 to 1.4 which corrects now exactly for + the arrival time difference between low-gain and high-gain pulses. + This wrong number could have had some effect on image cleanings + using the time information. (M. Gaug) + + * mcalib/MCalibrationBlindCam.[h,cc] + - remove obsolete Copy-function + + * mcalib/MCalibConstPix.h, mcalib/MCalibConstCam.h + - introduce Copy-functions + - set class index to 1 (was 0) + + * mcalib/MCalibrationChargePix.[h,cc] + - introduce Copy() function. + + + + 2005/07/13 Markus Gaug (2005/06/18) + + * mpedestal/MPedestalCam.cc + - modify Print() function to print out also the event-by-event + averaged results. + + * mpedestal/MExtractPedestal.[h,cc] + - new flag fUseSpecialPixels (set to kFALSE per default) + - new data member: fNameRawEvtData which can be set from outside + (e.g. to "MRawEvtData2"). + - with these two changes, the special pixels can be extracted. + Difference to previous version: if flag fUseSpecialPixels is set, + * the area and sector averages are not calculated. + * the MPedestalCam will get initialized to + MRawRunHeader->GetNumSpecialPixels(). + - fix a bug in the calculation of the event-by-event averaged + pedestal RMS (one total number of areas or sectors was not taken + into account). + - make local variables in the calculation of rms and mean doubles + instead of floats. + + * mpedestal/MPedCalcPedRun.cc + - do not calculate the area and sector averages in case flag + fUseSpecialPixels is set. + - check for existence of pointer fRawEvt in PostProcess. + + + + 2005/07/13 Markus Gaug (2005/06/15) + + * mcalib/MCalibrationIntensityConstCam.[h,cc] + - new class to store the actual applied conversion factors (now with + averaged photo-electrons not in the MCalibrationIntensityChargeCam + any more). + + * mjobs/MJCalibration.cc + - set the correct (statistical) error of the inter-calibraiton factor + into MCalibrationChargePix::SetConversionHiLoErr() and the + sigma of the distribution + into MCalibrationChargePix::SetConversionHiLoSigma(). + - In MCalibrationChargePix, the mean conversion error is used for the + statistical error of the conversion factor. + - In CalibrateData, the sigma is used for the statistical error of the + converted signal to equiv. high-gain charges. + + + 2005/07/13 Markus Gaug (2005/06/14) + + * mcalib/MCalibrationHiLoPix.[h,cc] + - store also results from profile fits. + - increades class version + + * mhcalib/MHCalibrationHiLoPix.[h,cc], mhcalib/MHCalibrationHiLoCam.[h,cc] + - added TProfiles for every pixel filled with the high-gain signal + vs. low-gain signal which can be fitted to extract gain and + offset. + + * mhcalib/Makefile, mhcalib/HCalibLinkDef.h + - added MHCalibrationHiLoPix + + + + 2005/07/13 Markus Gaug (2005/06/10) + + * mcalib/MCalibrationRelTimeCalc.cc + - print out forgotten successfully calibrated pixels + - small change in spacing of one output + + * mhcalib/MHCalibrationCam.[h,cc] + - introduce max. number of events, the rest gets skipped before the + next ReInit() is called. Default: 4096 + Reason: The calibration causes too many un-reliable pixels if more + than about 5000 events are treated (@500 Hz) because of the + mode hopping of the VCSels. However, in the past, some + calibration runs have been taken (erroneously) with more + than 5000 events, especially the intensity scans where + a good precision is needed. + - increased version number + - introduce a Getter for the fAverageAreaNum array. + - re-normalize the area-averaged sigmas to the equivalent number per + pixel also for the intensity calibration. + + * mcalib/MCalibrationChargePix.[h,cc] + - divide errors in stat. and syst. errors. Both can be retrieved + now, depending on the study, one wants to make. + + + + 2005/07/13 Markus Gaug (2005/06/06) + + * mhcalib/MHGausEvents.[h,cc] + - put small functions into header file to speed up calculations a bit. + + * mcalib/MCalibrationRelTimeCalc.cc + - introduce flags to steer the setting of unreliability like done + in MCalibrationChargeCalc. Can be steered from the conf-file. + + + + 2005/07/13 Thomas Bretz + + * mbase/MTaskInteractive.[h,cc]: + - fixed a bug with calling ReInit... PostProcess was called instead. + - added the missing initialisation of fReInit in constructor + + * sinope.cc: + - removed plotting the tasklist statistics twice + - replaced DrawClone by DrawCopy (DrawClone gave problems) + + * mhflux/MAlphaFitter.[h,cc]: + - fixed the ThetaSq function. It was wrongly defined as + exp(-((x-c)/s)^2) instead of exp(-1/2*((x-c)/s)) + - Set start value for corresponding fit back to same value + as for a standard gauss-fit. + + 2005/07/12 Daniela Dorner