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

refactor: Reduce abuse of auto in mbf smoother #3630

Merged
merged 11 commits into from
Sep 23, 2024

Conversation

CarloVarni
Copy link
Collaborator

Reduce abuse of auto

@CarloVarni CarloVarni added this to the v37.0.0 milestone Sep 19, 2024
@github-actions github-actions bot added Component - Core Affects the Core module Track Fitting labels Sep 19, 2024
@CarloVarni CarloVarni changed the title refactor: Reduce use of auto in mbf smoother refactor: Reduce abuse of auto in mbf smoother Sep 19, 2024
Copy link

github-actions bot commented Sep 19, 2024

📊: Physics performance monitoring for d90274c

Full contents

physmon summary

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.

I feel like we lose a bit of readability by explicitly writing out the types. But I don't feel strongly about it. Potentially this can be improved a bit with typedefs. Also note that the Projector typedef of track state is proposed to be removed

Core/src/TrackFitting/MbfSmoother.cpp Show resolved Hide resolved
Core/src/TrackFitting/MbfSmoother.cpp Outdated Show resolved Hide resolved
Core/src/TrackFitting/MbfSmoother.cpp Outdated Show resolved Hide resolved
Core/src/TrackFitting/MbfSmoother.cpp Outdated Show resolved Hide resolved
Core/src/TrackFitting/MbfSmoother.cpp Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Sep 23, 2024

@kodiakhq kodiakhq bot merged commit ebde4f7 into acts-project:main Sep 23, 2024
42 checks passed
@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Sep 23, 2024
@CarloVarni CarloVarni deleted the ReduceAutoInMsbSmoother branch September 24, 2024 06:44
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.

5 participants