Follow up on MR !120
This is a merge-request addressing @simonspa's comments on MR !120 (merged).
Here's the list:
Simon:
This merge might have been a bit early, there are several things I would like to see addressed:
-
Hardcoding TLUis not nice, especially with a type-sensitive comparison -
For adjust_event_timeswhy can't the default not just be empty? Having a default that doesn't mean anything isn't good style imho -
For require_detectorI remember discussing to implement this asgetArray<std::string>to directly allow requiring multiple detectors. -
I don't like the name detector_to_set_track_timestampand also not thatTimepix3_0is a hard-coded value as default. -
Track::hasDetectorhas to be aconstfunction.
Apart from that - nice work!
Edited by Jens Kroeger