|
[Rivet] ATLAS analysis to be included into RivetHendrik Hoeth hendrik.hoeth at cern.chMon Apr 15 15:58:51 BST 2013
Any opinions? This affects 14 out of 27 histograms. I really don't feel comfortable taking the analysis as it is, but I also don't feel like taking the heat on this list after rejecting the analysis (hey, this one doesn't even crash! I can't even imaging how strong a fire protection I'd need!) -- been there, done that way too often. So what do you guys think? Thus spake Roman Lysak (lysak at fzu.cz): > On 04/12/2013 11:41 AM, Hendrik Hoeth wrote: > >Hi Roman, hi Kiran, > > > >I'm having a brief look at the code right now. Have you tested it with > >weighted events? You do fill a weight vector _vecWeight, but never use > >it. The regression seems to assume equal weights for all events. Is that > >correct? > That is correct. > > The azimuthal part of the analysis includes the weights. > The fwd-bkwd part of the analysis does not include the weights. > > We tested this only with unweighted events (Herwig++ & Pythia6) and > compared to the results in the paper (there are no plots with MC > having weighted events, so we would not be able to validate the code > on weighted events anyway). > > I'm not sure what is the preference here, but it seems to me that > including the weights would require quite some changes to the code > (I'm not the author of the analysis who wrote the analyzing part of > the code and so it may take some time to do it).
More information about the Rivet mailing list |