Track Parameter CSV Writing, main branch (2025.02.10.) #849
Conversation
|
There was a problem hiding this comment.
I must say, that I am not completely happy of doing free track param csv IO in detray (https://github.com/acts-project/detray/blob/main/io/include/detray/io/csv/track_parameters.hpp) and bound track param IO in traccc. I think, it would be better to keep the IO implementation of detray types bundled in one place. Sooner or later, somebody decides this is needed for some detray debugging as well, and nobody remembers that traccc has this implemented and then we have two versions...
Besides, detray io is already a traccc dependency, right? So, it should not be too difficult to extend/adapt the detray implementation of the csv IO and then adding an interface for the traccc container types?
|
I was not aware of the CSV capabilities in Detray... Sure, I think it's fair that such "pure Detray collections" would have I/O code in Detray. I'll see how to port this code into that project... |
If that is possible, that would be great! |
While debugging the issue fixed in #848, I wrote this code for dumping the properties of reconstructed bound track parameters.
While
detray::bound_track_parametershas a print operator itself (which I'm very much on board with, and could've technically used for this debugging as well...), I thought this code may still be good to preserve. It could come in handy in some future debugging sessions as well. 🤔