Skip to content

fix: Fix GeoModel version compile switch#5266

Merged
kodiakhq[bot] merged 2 commits intoacts-project:mainfrom
andiwand:fix-geomodel-version-compile-switch
Mar 20, 2026
Merged

fix: Fix GeoModel version compile switch#5266
kodiakhq[bot] merged 2 commits intoacts-project:mainfrom
andiwand:fix-geomodel-version-compile-switch

Conversation

@andiwand
Copy link
Contributor

@andiwand andiwand commented Mar 20, 2026

After #5223 I cannot locally compile Examples/Detectors/MuonSpectrometerMockupDetector/src/GeoMuonMockupExperiment.cpp anymore. I suspect that the captured lambda does not trigger the type switch correctly while using auto does because the lambda will be effectively templated.

--- END COMMIT MESSAGE ---

@andiwand andiwand added this to the next milestone Mar 20, 2026
@github-actions github-actions bot added the Component - Examples Affects the Examples module label Mar 20, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 20, 2026

📊: Physics performance monitoring for f13a4be

Full contents

physmon summary

@andiwand andiwand requested a review from paulgessinger March 20, 2026 13:10
@andiwand andiwand marked this pull request as ready for review March 20, 2026 13:10
@asalzburger
Copy link
Contributor

Can confirm that this was an issue, probably specific to Mac/clang ?

@andiwand
Copy link
Contributor Author

Can confirm that this was an issue, probably specific to Mac/clang ?

I think it depends on the GeoModel version but why it does not happen the other way around not sure

@sonarqubecloud
Copy link

@kodiakhq kodiakhq bot merged commit 8df6689 into acts-project:main Mar 20, 2026
42 checks passed
@andiwand andiwand deleted the fix-geomodel-version-compile-switch branch March 21, 2026 05:41
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants