DUNE / dune-tms

DUNE ND Temporary Muon Spectrometer
0 stars 2 forks source link

Merge feature/kalman, TMS_Bar and TMS_Kalman small changes #167

Open LiamOS opened 1 month ago

LiamOS commented 1 month ago

config/TMS_Default_Config.toml:

TMS_Bar:

TMS_Kalman:

TMS_Reco

TMS_Track

TMS_TreeWriter

LiamOS commented 1 month ago

Should be behaving substantially better with this, but not perfect. Still leaves the detector sometimes. Want to wait for some feedback for Asa before tackling this fully.

LiamOS commented 1 month ago

Validation plots from one file located in /exp/dune/data/users/losulliv/dune-tms/Validation/Reco_Tracks/MicroProdN1p1_NDLAr_1E18_RHC.spill.nu.0000120.EDEPSIM_SPILLS_TMS_RecoCandidates_Hough_Cluster1_images/

For future PRs, should we define some minimal set of required validation plots that should be presented, or do it depending on the content of the PRs?

AsaNehm commented 1 week ago

Solved merging conflict by removing the sanity check in TMS_Bar that was still in in this branch but removed on main