Skip to content

Add an overload for Pandora for the member function propagateToLayer#42

Merged
jmcarcell merged 4 commits into
mainfrom
add-overload-pandora
Sep 30, 2025
Merged

Add an overload for Pandora for the member function propagateToLayer#42
jmcarcell merged 4 commits into
mainfrom
add-overload-pandora

Conversation

@jmcarcell

Copy link
Copy Markdown
Member

BEGINRELEASENOTES

  • Add an overload for Pandora for the member function propagateToLayer that doesn't take any hits and uses the last hit added to the track to get the track measurement and propagate

ENDRELEASENOTES

This overload is used by Pandora for propagating tracks without fitting. There exists another overload that takes an EDM4hep hit and we could pass the last hit that was added to the track to it. However, that doesn't work because that overload will try to get the TKalTrackSite (track measurement) that is only built after fitting, which Pandora does not need since it uses the refitted tracks (for CLD). Already used in key4hep/k4GaudiPandora#16

This overload is used by Pandora for propagating tracks without fitting. There
exists another overload that takes an EDM4hep hit and we could pass the last hit
that was added to the track to it. However, that doesn't work because that
overload will try to get the TKalTrackSite (track measurement) that is only
built after fitting, which Pandora does not need since it uses the refitted
tracks (for CLD).
Comment thread k4Reco/GaudiTrkUtils/src/GaudiDDKalTestTrack.cpp Outdated
Co-authored-by: Thomas Madlener <thomas.madlener@desy.de>
@jmcarcell jmcarcell enabled auto-merge (squash) September 30, 2025 19:12
@jmcarcell jmcarcell merged commit ac42a0a into main Sep 30, 2025
6 checks passed
@jmcarcell jmcarcell deleted the add-overload-pandora branch September 30, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants