algo: Add skeleton for cbmreco app.

Adds the cbmreco binary. This is just a dummy for the moment that doens't do anything. I'll add features from my fork in upcoming MRs to reproduce the results i have there.

There's a lot of overlap with the existing cbmreco_fairrun binary. Once i have reproduced the state from my fork, i'll try to merge those two.

assigned to @fweig

added 1 commit

• ad791056 - algo: Add skeleton for cbmreco app.

Compare with previous version

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

mentioned in merge request !1161 (merged)

requested review from @j.decuveland

added 2 commits

• 9ed6d5c2 - algo: Add config micro-library to read YAML files and map to C++ classes.
• 6535245d - algo: Add skeleton for cbmreco app.

Compare with previous version

@j.decuveland, I've renamed the input-file flag to fles-locator for consistency with the naming in fles::TimesliceAutoSource.

First of all: I have no objections to this MR.

I do have a few questions and comments:

• The Options struct can hardly be reviewed without more context. For std::filesystem::path InputFile() const { return fInputFile; }, could you have a look at TimesliceAutoSource.hpp? I think we could better use something like:

std::string TimesliceSourceLocator
std::string InputLocator

• As mentioned earlier, the FairLogger library has two issues that may make it less than ideal for the online application: First, it does not easily support syslog, and second, it limits console output performance by requiring a sync after every message.
  If we don't want to introduce an additional dependency, we could use the minimalistic library (small wrapper around boost_log) that is included in cbmroot through Flesnet: logging (as cbmreco_fairrun already does). It can easily be replaced by a more evolved solution in the future.

added 1 commit

• 0c1ed027 - algo: Add skeleton for cbmreco app.

Compare with previous version

added 7 commits

• 0c1ed027...64321a41 - 5 commits from branch computing:master
• ce9c360b - algo: Add config micro-library to read YAML files and map to C++ classes.
• b457fa95 - algo: Add skeleton for cbmreco app.

Compare with previous version

mentioned in merge request !1164 (merged)

resolved all threads

approved this merge request

Sorry for the delay. Let's merge this and move on!

added 8 commits

• b457fa95...59b6cd65 - 6 commits from branch computing:master
• b9d061ac - algo: Add config micro-library to read YAML files and map to C++ classes.
• 76c414ed - algo: Add skeleton for cbmreco app.

Compare with previous version

enabled an automatic merge when the pipeline for 76c414ed succeeds

merged

mentioned in merge request !1152 (closed)