Scott Snyder authored 3 months ago and Jean-Baptiste De Vivie De Regie committed 3 months ago

DerivationFrameworkPhys: Avoid overwriting decorations.

LLP1 was configuring IsoCloseByCorrectionAlg with non-LRT containers
as well as LRT ones.  But PHYS also configures this algorithm with
non-LRT containers, so if they were used together, then decorations
would get overwritten.  This causes problems then when we want to ensure
that the decorations get locked.

Split the configuration here into two algorithms, the first being identical
to PHYS and the second with the LRT containers.  When PHYS and LLP1 are
used together, then then first algorithm will be merged with the one
from PHYS and we avoid overwrites.

This can result in changes in the decorations for non-LRT containers
in the case that PHYS and LLP1 are used together, due to the differing
track selectors that are used.