algo: Change include scheme.
Change algo include scheme, so detector-name is prefixed in the include path.
For discussion, see e.g.: !1185 (comment 30533)
CC: @d.smith
Merge request reports
Activity
requested review from @v.friese
assigned to @v.friese
Looks ok, although it seems to be an example of "conservation of complexity": We gain the ability to drop detector-suffixes from filenames but pay with longer include statements. We then replace detector-suffixes in class names with sub-namespaces.
I have no strong opinions either way on this, but lean slightly towards giving this variant a try. It seems more "modern".
Dear @f.uhlig, @v.friese, @p.-a.loizeau,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
- Resolved by Volker Friese
It would seem consequent, when following the namespace scheme, to drop the detector suffixes in the class names, e.g. tof/TofConfig to tof/Config. @fweig, what is your opinion to that?
changed milestone to %OCT23
added Online label