Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(gx2f): new error UsedUnreachableMeasurements #3653

Merged
merged 5 commits into from
Oct 2, 2024

Conversation

AJPfleger
Copy link
Contributor

@AJPfleger AJPfleger commented Sep 26, 2024

This fixes a super rare bug. 1 in 17k segfaulted at calculateTrackQuantities(track). This was, because the tipIndex was set outside the actual track. The wrong tipIndex comes from the last propagation where we didn't try to hit all inputMeasurements on insertExternalSurfaces.

The created track contains measurement information, that is missing in the final track. We could think about having this as an outlier or refit ignoring the missed measurements. For now, I would just return an error, because:

  • We have no proper outlier definition in the GX2F yet
  • Refitting should be maybe a user-task. The whole refitting logic with a dropped measurement might make the whole fitter quite complicated
  • The error is so rare, that it is fine, to just avoid crashing

We discussed an would refrain from creating a unit test, since it is quite difficult to reproduce this edge case in an isolated environment. Due to its rarity, it might be not worth the time.

@AJPfleger AJPfleger added this to the v37.0.0 milestone Sep 26, 2024
@github-actions github-actions bot added Component - Core Affects the Core module Track Fitting labels Sep 26, 2024
@AJPfleger AJPfleger changed the title fix(gx2f): missed surface in final propagation fix(gx2f): new error UsedUnreachableMeasurements Sep 27, 2024
@AJPfleger AJPfleger marked this pull request as ready for review September 27, 2024 08:11
Copy link

github-actions bot commented Sep 27, 2024

📊: Physics performance monitoring for 2d1fde0

Full contents

physmon summary

@AJPfleger AJPfleger mentioned this pull request Sep 30, 2024
Copy link

sonarcloud bot commented Oct 2, 2024

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@kodiakhq kodiakhq bot merged commit 661ab12 into acts-project:main Oct 2, 2024
44 checks passed
@github-actions github-actions bot removed the automerge label Oct 2, 2024
@AJPfleger AJPfleger deleted the gx2f-segfault17k branch October 2, 2024 07:58
@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Fails Athena tests This PR causes a failure in the Athena tests Track Fitting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants