RHEL4 SCons versus CMT redhat4 builds are validated.
RHEL5 SCons builds versus RHEL4 SCons validation is well underway.
GlastRelease moved to Gaudi v21r7-gl1 allowing for RHEL5 builds starting with tag GlastRelease 20-00-00. Validation started with tag 20-03-04, where first the CMT rhel4_gcc34opt build was used as the baseline for the SCons redhat4-i686-32bit-gcc34 Optimized build system tests, where event based random number seeding was used.
http://glast-ground.slac.stanford.edu/SystemTests/?releaseVersionId=13441
Looking at the plots, the main thing I noticed is that the quantity CALRMSTRANS seems to be different in most tests, AG, Muons, Bkg. For other variables in the BackGndMixDC2, the differences seem really small to me, small enough to be ignored... unless someone is willing to dig into the code obviously. Cheers, the CALRMSTRANS variable in the systests comes from the recon ROOT files, it is contains the value of getRmsTrans() for every CalCluster, see line 82 in CalCluster.h: http://www-glast.stanford.edu/cgi-bin/viewcvs/reconRootData/reconRootData/CalCluster.h?annotate=1.32
Where if I'm reading the code correctly, getRmsTrans is associated with an old pre-P8 variable that is left in to allow reading of older ROOT files. But in today's world - that variable isn't initialized ... not even to zero..and appears to be completely unused. One might modify the code to at least init those old variables to zero to avoid future confusion and this particular plot could be eliminated and replaced in the system tests..with something else such as the CalMomParams values - see next paragraph. Meanwhile the merit variable Cal1TransRms is filled in AnaTup via: Heather
|
Next the redhat5-i686-32bit-gcc41 Optimized build was run against the redhat4-i686-32bit-gcc34 Optimized build of GlastRelease-20-03-04:
http://glast-ground.slac.stanford.edu/SystemTests/?releaseVersionId=13461
some information about the RHEL5 vs RHEL4 system test: http://glast-ground.slac.stanford.edu/SystemTests/plots.jsp?releaseVersionId=13461&selectedReferenceReleaseVersionId=13441&testName=BackGndMixDC2 + allGamma: + VerticalProton1GeV + VerticalGamma100MeV has a number of interesting features + BackGndMixDC2 So, overall, to my eyes, there is nothing really wrong: However, I don't have much experience in looking at these system tests, and in understanding how to deal with small differences, so I'll let other more expert than I am chime in ! Cheers, |