Skip to content

Cleanup track decorators

Samuel Van Stroud requested to merge svanstro/training-dataset-dumper:svanstro/cleanup-track-decorators into r22
  • Cleaned up TrackVertexDecorator in the hope of improving performance
  • Remove TrackAmiDecorator since downstream tagger performance impact was found to be negligible (and I promised to remove this if it was not useful)

The TrackVertexDecorator had a negligible impact on performance, I must have got my wires crossed when writing #36 (closed). It's actually the track truth decoration which has the massive overhead. But I kept the changes anyway as they didn't hurt, and I think the code is a little more clear and consistent with the other decorators. I check that the SV1VertexIndex and JFVertexIndex variables were identical between old and new outputs.

Merge request reports