Version 7 of the FT decoding

Works with lhcb/LHCb!2942 (merged)

Reformulates the link<->SiPM correspondency in the FT. Not backwards-compatible, but adds a versioning for the ReadoutTool to send proper ERROR in case of mismatch.

do not delete the branch as it is used in Boole tests. @rmatev will do this once the tag used in Boole is updated.

changed target branch from master to upgrade/master

mentioned in merge request lhcb/LHCb!2942 (merged)

Is it possible to rename this tag upgrade/sim-20211006-vc-md100-ft-v7 ? @cattanem

This is not a tag, it is a branch. The easier thing is to make a temporary copy. We can decide if we merged this or close it and delete this branch and make a new MR.

The new branch upgrade/sim-20211006-vc-md100-ft-v7 is there.

This is being tested with the new temporary branch. It is kept for now as a placeholder.

@lohenry, when ready, please put a note here and we will decide if merging from here or make a new MR.

cc: @gcavalle, @edelucia, @tlatham

assigned to @rmatev

mentioned in issue lhcb/LHCb#202

A version of this also need adding to the yaml based DB, the one to be used by dd4hep

https://gitlab.cern.ch/lhcb-conddb/lhcb-conditions-database

SIMCOND isn't going to be used, eventually, for real data taking.

mentioned in merge request lhcb/LHCb!3510 (merged)

mentioned in issue lhcb/LHCb#213 (closed)

changed the description

resolved all threads

approved this merge request

@seaso this is ready and has been tested in the nighlies. Just make sure not to delete the branch (as per the description).

Edit: since this is not backward-compatible, there is a possible issue if tests are using upgrade/master (they shouldn't outside Boole). We can fix that after the MR is merged if needed.

assigned to @seaso and unassigned @rmatev

ping @seaso it would be useful to merge this soon as otherwise it gets harder to follow up with the changes for the other subdetectors