<div>Thanks. Do you remember which one? This should only be visible to the DressedLeptons class, which was validated for the original form... so this update may be double-counting the bare lepton for all other analyses at the moment. I'd like to take a look and see if this analysis is doing something special with DressedLepton.</div><br><div>Cheers,</div><div>Andy</div><br><div><signature id="initial"><div><table cellpadding="0" cellspacing="0"><tbody><tr><td colspan="2"><div style="padding-bottom:15px;"><div><strong>Dr Andy Buckley, Lecturer / Royal Society University Research Fellow</strong></div><div>Particle Physics Experiment Group, University of Glasgow</div></div></td></tr><tr><td style="vertical-align:top;"></td><td><div style="font-size:0.9em;white-space:nowrap;border-left:2px solid gray;margin-left:20px;padding-left:20px;"><div><div></div><div></div></div></div></td></tr></tbody></table></div></signature></div><div class="gmail_quote_attribution">On Mar 4 2018, at 12:25 pm, Jonathan Butterworth <J.Butterworth@ucl.ac.uk> wrote:</div><blockquote><br><div><div>Ok. I fixed it because it was causing an uncontrolled crash in one of</div><div>the contrib CMS routines...</div><br><div>On 02/03/2018 17:18, Andy Buckley wrote:</div><blockquote><div>I have a funny feeling that I didn't do that on purpose, but can't</div><div>remember the exact motivation now... should have put it in a comment.</div><div>Hmm, one to look out for in the next pre-release tests.</div><br><div>More generally, I would like to eliminate the DressedLepton class and</div><div>just use Particle for that purpose, now that it can have constituents.</div><div>And also unify the Particle and Jet a bit better, since both will have</div><div>constituents... and it could be useful to use jets as inputs to FastJets</div><div>for ATLAS-style reclustering.</div><br><div>Andy</div><br><div>*Dr Andy Buckley, Lecturer / Royal Society University Research Fellow*</div><div>Particle Physics Experiment Group, University of Glasgow</div><br><br><div>On Feb 27 2018, at 11:15 pm, Rivet Mercurial</div><div><rivet@projects.hepforge.org> wrote:</div><br><br><div>details: https://rivet.hepforge.org/hg/rivet/rev/b4200fb4a998</div><div>branches: release-2-6-x</div><div>changeset: 6245:b4200fb4a998</div><div>user: Jon Butterworth <j.butterworth@cern.ch></div><div>date: Tue Feb 27 22:34:33 2018 +0000</div><div>description:</div><div>fix constructor - always needs to have itself as a constituent</div><br><div>diffs (13 lines):</div><br><div>--- a/src/Projections/DressedLeptons.cc Thu Feb 22 16:09:31 2018 +0000</div><div>+++ b/src/Projections/DressedLeptons.cc Tue Feb 27 22:34:33 2018 +0000</div><div>@@ -9,7 +9,9 @@</div><br><div>DressedLepton::DressedLepton(const Particle& dlepton)</div><div>: Particle(dlepton)</div><div>- { }</div><div>+ {</div><div>+ setConstituents({{dlepton}}); //< bare lepton is first constituent</div><div>+ }</div><br><div>DressedLepton::DressedLepton(const Particle& lepton, const</div><div>Particles& photons, bool momsum)</div><div>: Particle(lepton.pid(), lepton.momentum())</div><div>_______________________________________________</div><div>Rivet-svn mailing list</div><div>Rivet-svn@projects.hepforge.org</div><div>https://www.hepforge.org/lists/listinfo/rivet-svn</div><br></blockquote><br><div>--</div><div>~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</div><div>Prof. Jonathan Butterworth, http://www.hep.ucl.ac.uk/~jmb/</div><div>Head, Physics and Astronomy Department Tel: +44 20 7679 3444</div><div>University College London Gower St, London WC1E 6BT, UK</div><div>ATLAS, CERN Tel: +41 22 76 72340</div><div>~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</div></div></blockquote>