Skip to content

fix: Track container resize dynamic columns if the column is created after tracks have been made#5463

Merged
kodiakhq[bot] merged 11 commits into
acts-project:mainfrom
junggjo9:FixDynCol
May 21, 2026
Merged

fix: Track container resize dynamic columns if the column is created after tracks have been made#5463
kodiakhq[bot] merged 11 commits into
acts-project:mainfrom
junggjo9:FixDynCol

Conversation

@junggjo9

@junggjo9 junggjo9 commented May 18, 2026

Copy link
Copy Markdown
Contributor
  • Track container resize dynamic columns if the column is created after tracks have been made
  • Improve documentation of the DynamicColumnBase

--- END COMMIT MESSAGE ---

Tagging @andiwand, @paulgessinger

@github-actions github-actions Bot added Component - Core Affects the Core module Event Data Model labels May 18, 2026
@github-actions github-actions Bot added this to the next milestone May 18, 2026
andiwand
andiwand previously approved these changes May 18, 2026

@andiwand andiwand left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍 thanks @junggjo9 !

I would still wait for @paulgessinger to have a look before getting this in

@github-actions

github-actions Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

📊: Physics performance monitoring for b391007

Full contents

physmon summary

Comment thread Core/include/Acts/EventData/detail/DynamicColumn.hpp Outdated
Comment thread Core/include/Acts/EventData/detail/DynamicColumn.hpp Outdated
Comment thread Core/include/Acts/EventData/VectorMultiTrajectory.hpp
Co-authored-by: Andreas Stefl <stefl.andreas@gmail.com>
Comment thread Core/include/Acts/EventData/VectorMultiTrajectory.hpp
Comment thread Core/include/Acts/EventData/detail/DynamicColumn.hpp Outdated
paulgessinger
paulgessinger previously approved these changes May 20, 2026

@paulgessinger paulgessinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine by me. The only question I have is if we could locate this behavior in the front end somehow so that we could get it consistently also for back ends like and xAOD? (even if the latter isn't really used anymore)

Comment thread Core/include/Acts/EventData/detail/DynamicColumn.hpp Outdated
Co-authored-by: Paul Gessinger <hello@paulgessinger.com>
@junggjo9

Copy link
Copy Markdown
Contributor Author

I am not really sure how we could do this. I've encountered this issue when I tried to decorate the track returned from the global chi2 fitter (https://gitlab.cern.ch/atlas/athena/-/merge_requests/88064)

@sonarqubecloud

Copy link
Copy Markdown

@kodiakhq kodiakhq Bot merged commit 4b4267e into acts-project:main May 21, 2026
46 checks passed
@junggjo9 junggjo9 deleted the FixDynCol branch May 21, 2026 06:16
@andiwand andiwand modified the milestones: next, v46.7.0 May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants