Skip to content

Commit

Permalink
fix: addAmbiguityResolution writeTrackSummary/writeTrackStates (acts-…
Browse files Browse the repository at this point in the history
…project#3809)

* `addAmbiguityResolution()`'s `writeTrackSummary` and `writeTrackStates` were set the wrong way round.
* update `addCKFTracks()` parameters doc
  • Loading branch information
timadye authored Nov 5, 2024
1 parent a3b81d5 commit 1e2a44b
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions Examples/Python/python/acts/examples/reconstruction.py
Original file line number Diff line number Diff line change
Expand Up @@ -1403,8 +1403,14 @@ def addCKFTracks(
TrackSelector configuration. Each range is specified as a tuple of (min,max).
Specify as a list(TrackSelectorConfig) for eta-dependent cuts, with binning specified by absEta[1].
Defaults of no cuts specified in Examples/Algorithms/TruthTracking/ActsExamples/TruthTracking/TrackSelector.hpp
writeTrajectories : bool, True
write trackstates_ckf.root and tracksummary_ckf.root ntuples? These can be quite large.
writeTrackSummary : bool, True
write tracksummary_ckf.root ntuple?
writeTrackStates : bool, False
write trackstates_ckf.root ntuple? This can be quite large.
writePerformance : bool, True
write performance_fitting_ckf.root and performance_finding_ckf.root ntuples?
writeCovMat : bool, False
write covaraiance matrices to tracksummary_ckf.root ntuple?
"""

customLogLevel = acts.examples.defaultLogging(s, logLevel)
Expand Down Expand Up @@ -1902,8 +1908,8 @@ def addAmbiguityResolution(
tracks=alg.config.outputTracks,
outputDirCsv=outputDirCsv,
outputDirRoot=outputDirRoot,
writeSummary=writeTrackStates,
writeStates=writeTrackSummary,
writeSummary=writeTrackSummary,
writeStates=writeTrackStates,
writeFitterPerformance=writePerformance,
writeFinderPerformance=writePerformance,
writeCovMat=writeCovMat,
Expand Down

0 comments on commit 1e2a44b

Please sign in to comment.