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: Copy whole trackstate did not copy calibrated #3693

Merged

Conversation

benjaminhuth
Copy link
Member

Know your defaults...

@benjaminhuth benjaminhuth added this to the next milestone Oct 4, 2024
@benjaminhuth benjaminhuth requested a review from andiwand October 4, 2024 13:03
@github-actions github-actions bot added Component - Core Affects the Core module Track Fitting labels Oct 4, 2024
andiwand
andiwand previously approved these changes Oct 4, 2024
@andiwand
Copy link
Contributor

andiwand commented Oct 4, 2024

I wonder if this should really be a parameter over a new function since it seems to change the behavior quite substantially

Copy link

github-actions bot commented Oct 4, 2024

📊: Physics performance monitoring for 96c79c4

Full contents

physmon summary

@benjaminhuth
Copy link
Member Author

benjaminhuth commented Oct 4, 2024

Yeah I would agree... Any thoughts @paulgessinger ?

@benjaminhuth
Copy link
Member Author

@paulgessinger I added a test case that only checks the copy of the calibrated field of two track states, that also confirmed the bug we found. The test is very specific to this error, is it okay like that?

@paulgessinger paulgessinger changed the title fix: Copy whole trackstate in GSF combined state creation fix: Copy whole trackstate did not copy calibratedd Oct 7, 2024
paulgessinger
paulgessinger previously approved these changes Oct 7, 2024
@andiwand andiwand changed the title fix: Copy whole trackstate did not copy calibratedd fix: Copy whole trackstate did not copy calibrated Oct 8, 2024
Copy link

sonarqubecloud bot commented Oct 9, 2024

@kodiakhq kodiakhq bot merged commit 94698b3 into acts-project:main Oct 9, 2024
42 checks passed
@github-actions github-actions bot removed the automerge label Oct 9, 2024
@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Oct 9, 2024
@paulgessinger paulgessinger modified the milestones: next, v37.1.0 Oct 16, 2024
Rosie-Hasan pushed a commit to Rosie-Hasan/acts that referenced this pull request Nov 13, 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 Event Data Model Fails Athena tests This PR causes a failure in the Athena tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants