|
[Rivet] CMS rivet analysesHendrik Hoeth hendrik.hoeth at cern.chThu Jun 16 16:36:21 BST 2011
Dear Albert, thanks a lot for the analyses! I still need to take a closer look, but I do have a few questions: - Did you validate the Rivet analyses against your own analysis, i.e. can you reproduce the MC plots in the publication? You are naturally in a much better situation to do this than we are, so please set the status in the .info files to "VALIDATED" if they are validated or fix them if they need to be fixed. - The info file for CMS_2011_S8968497 is fairly ... empty. Also the plot titles in the .plot file only list cuts, not what observable you are plotting. Do you plan to update this analysis? - The .plot file for CMS_2011_S8884919 is empty. Well, technically it's not, but it doesn't contain any information. The .info file for that analysis causes two parse errors that are easily fixed by indenting the Description text and removing the whitespace around the colons in the References. - In CMS_2011_S8884919 you fill the _h_cq_eta05 and _h_cq_eta24 histograms in the finalize method. Are you sure you are handling the uncertainties correctly? Don't you have to use a DataPointSet and handle the errors manually? - The plot titles of CMS_2010_S8656010 talk about |eta|=0.1, |eta|=0.2, etc. I assume you actually mean <= and not =. Is that correct? Cheers, Hendrik -- The minimum number of planes one should own is one. The correct number is n+1, where n is the number of planes currently owned. This equation may also be re-written as s-1, where s is the number of planes owned that would result in separation from your partner. -- Veluminati
More information about the Rivet mailing list |