Adding option to save multiple MFT-MUON matching candidates #1872
Conversation
mcoquet642
requested review from
benedikt-voelkel,
chiarazampolli and
sawenzel
as code owners
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
|
||
| # Matching training for machine learning | ||
| parser.add_argument('--fwdmatching-save-trainingdata', action='store_true', help='enables saving parameters at plane for matching training with machine learning') | ||
| parser.add_argument('--fwdmatchingNcandidates', type=int, help='enables saving provided number of matching candidates', default=0) |
There was a problem hiding this comment.
I believe this change is no longer necessary with #1869
You can now simply pass configurables from an external file. Could you please check if this works for you?
I really would favour to avoid adding more and more detector specific options to o2dpg_sim_workflow (when they merely set a config-key).
sawenzel
left a comment
sawenzel
left a comment
There was a problem hiding this comment.
see inline comment. Potentially this PR can be closed.
|
Thanks Sandro, I did not know about this new feature. Closing. |
2 participants
This PR allows to save multiple MFT-MUON matching candidates, which will be used to build machine learning training samples.