Page MenuHomeHEPForge

Modernise EvtIdSet and other improvements

Authored by tlatham on Aug 31 2023, 2:42 PM.



Modernise and greatly simplify EvtIdSet implementation
Fixes in EvtPropSLPole to avoid unnecessary dynamic allocations
Other minor fixes and tidy-ups

Credit to Heather Ratcliffe and Chris Brady for providing
and/or inspiring several of these improvements

Test Plan

CI tests give identical results wrt master branch

Diff Detail

rEVTGEN evtgen
Lint Not Applicable
Tests Not Applicable

Event Timeline

tlatham published this revision for review.Aug 31 2023, 2:47 PM
tlatham changed the visibility from "All Users" to "Public (No Login Required)".
tlatham changed the edit policy from "All Users" to "Restricted Project (Project)".
tlatham added a project: Restricted Project.
tlatham added reviewers: jback, kreps, abudinen.
tlatham added a subscriber: Restricted Project.

All changes look plausible to me.

This revision is now accepted and ready to land.Oct 6 2023, 5:55 PM

Thanks @abudinen! @jback, @kreps, any comments? Or should I go ahead with this?

Changes look OK. This can be merged.