ATTPC/Spyral

Trim interpolated trajectories based on boundaries

Closed this issue · 1 comments

When solving the ODE, we stop upon hitting boundaries in the detector size. However, when getting the interpolated trajectory, we translate the vertex, which could cause an the trajectory to again leave the valid bounds of the detector. This is a performance issue mostly, as we need to loop over all points in the interpolated for each point in the data. Getting rid of this out-of-bounds junk would probably help a lot.

Turns out it doesn't help. My guess is that too few events acutally benefit enough to outweigh the cost of an extra loop. You could probably tune this enough that it has no impact, but then whats the point. Closing.