Moved STS system time offset parameter from sts::ReadoutConfig to sts::WalkMap.

Changes requested by @a.toia.

Requires updated YAML files, to be submitted separately.

requested review from @p.-a.loizeau

assigned to @p.-a.loizeau

@a.toia @p.-a.loizeau @a.toia FYI: The time offset is stored in hex format. This might be confusing for someone reading the YAML file.

mentioned in merge request CbmSoft/cbmroot_parameter!194 (merged)

are you sure it's an hex? I always used it as int

in algo/detectors/sts/ReadoutConfig.h

i32 timeOffset;

CBM_YAML_PROPERTIES(yaml::Property(&ReadoutSetup::timeOffset, "timeOffset", "Time offset for STS", YAML::Hex),

added 5 commits

• bb43464a...823c0976 - 4 commits from branch computing:master
• 034a0d33 - Moved STS system time offset parameter from sts::ReadoutConfig to sts::WalkMap.

Compare with previous version

added Online label

@d.smith Did you check with a file that the changes are transparent?

I would expect then to have 1:1 digis in the output, is this assumption right?
When I tried it locally on the mFLES node with the first spill of run 3105, the files generated by the master version and by your two MRs combined did not have identical entries, at least according to the tool made by Florian. I will try to have a look at the Digis themselves one by one tomorrow morning.

@p.-a.loizeau So nothing more is to be done in this MR? The problem is in the parameter file?

resolved all threads

@d.smith CbmSoft/cbmroot_parameter!194 (merged) is now merged, you can rebase and add the commit with the parameters repo hash update

added 39 commits

• 034a0d33...fa388282 - 37 commits from branch computing:master
• e8c28004 - Moved STS system time offset parameter from sts::ReadoutConfig to sts::WalkMap.
• 3780c675 - Updated parameter hash.

Compare with previous version

resolved all threads

enabled an automatic merge when the pipeline for 3780c675 succeeds

Dear @v.friese, @f.uhlig,

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

merged