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: Correct initial q/p covariance term in TrackParamsEstimationAlgorithm in Examples #3665

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

andiwand
Copy link
Contributor

I seem to have made a mistake calculating the covariance term on the first try in #3422. This is corrected here

@andiwand andiwand added this to the next milestone Sep 30, 2024
@github-actions github-actions bot added Component - Examples Affects the Examples module Track Finding labels Sep 30, 2024
@asalzburger asalzburger self-requested a review September 30, 2024 15:23
asalzburger
asalzburger previously approved these changes Sep 30, 2024
@asalzburger
Copy link
Contributor

Changes in output are expected, I approve the change (as it seems correct), so let's wait for the physmon results.

Copy link

github-actions bot commented Sep 30, 2024

📊: Physics performance monitoring for 3d672ab

Full contents

physmon summary

Copy link

sonarcloud bot commented Sep 30, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Examples Affects the Examples module Track Finding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants