Skip to content

ActsEventCnv: handle annulus surfaces correctly

Edward Moyse requested to merge emoyse/athena:main-improve-acts-event-cnv into main

I added some sanity checks to try to understand the differences between Trk and ACTS, and (with the help of @pagessin) realised that Annulus bound surfaces were not handled correctly.

These changes seem to fix these differences (many still remain, but hopefuly #65239 will help here).

BTW I will remove the sanity checks (or make them optional) once this is all working, but for the moment I think it's helpful to have them in.

Edit: I should have said that this does NOT correct the annulus cov matrices yet.

cc @ncalace

Edited by Edward Moyse

Merge request reports