Fix for L1KeyList migration
Add ME0 to tgeo reco. Organise TGeoManager colors.

Conflicts:
	Fireworks/Geometry/src/FWTGeoRecoGeometryESProducer.cc
Temporary comment out ME0.
Temporary comment out ME0.
fix uninitialised var timetype to avoid error from checker
fix out of bound array index in HCalResponse
Fix run crashing issues
Cleanup
remove extra semicolon
Update HCALResponse.cc
Resolve undefined behavior with VLAs length

RecoEcal/EgammaClusterAlgos/src/Multi5x5BremRecoveryClusterAlgo.cc:47:34:
runtime error: variable length array bound evaluates to non-positive
value 0
RecoEcal/EgammaClusterAlgos/src/Multi5x5BremRecoveryClusterAlgo.cc:50:30:
runtime error: variable length array bound evaluates to non-positive
value 0

Multi5x5BremRecoveryClusterAlgo::makeIslandSuperClusters(..) is called
with empty clusters_v. There is nothing to be done if clusters_v is
empty, thus bail out early.

This happens 36 times while running short matrix.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Enabling import of HCAL OOT related types
TMatrix and TVector replacement with SMatrix and SVector in default MET code
Update QGLikelihoodDBWriter to read bins from file
Remove unsued seedsRefs from DuplicateListMerger.cc

RecoTracker/FinalTrackSelectors/plugins/DuplicateListMerger.cc:196:49
was reported by UBSan 25 times during full matrix.

DuplicateListMerger.cc:196:49: runtime error: variable length array
bound evaluates to non-positive value 0

Example workflows:
- 33.0 / step3
- 1319.0 / step3
- 1001.0 / step2
- 1002.0 / step2

seedsRefs is not used and removing it does not trigger any side effects.
Otherwise this is a potential spot for out-of-bounds writes to stack if
a possible code path if found.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Change Word64 type from long long to unsigned long long

The following change is required to solve undefined behavior detected by
UBSan.

    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:17:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:18:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:19:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:20:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:21:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:22:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:23:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:24:22
    405 EventFilter/ESRawToDigi/src/ESUnpacker.cc:25:22

~Word64(0) is -1 and left shifting a negative number is undefined
behavior by C++.

These produce 3645 runtime errors from UBSan on full matrix. That's ~15%
of all detected runtime errors.

Word32 is defined as unsigned type. Keep it consistent and define
Word64 as unsigned long long, thus resolved undefined behavior.

Looking at the code these 64-bit masks are not used outside this package
and they are only used with bitwise and operators. The change should not
have any side effects.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Remove undefined behavior from PixelTrackCleanerBySharedHits.cc

According to UBSan

    RecoPixelVertexing/PixelTrackFitting/src/
    PixelTrackCleanerBySharedHits.cc:38:20

is triggered 151 times on full matrix.

Input vector for the function is empty, which attempts to create
0-length VLA. There is nothig to be done if size is 0, bail out early.

Additionaly make sure that all indexes are unsigned.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
increase splitting for PU samples; using GEN-SIM from 720_pre6
Merge pull request #5459 from davidlt/pixel-track-ubsan-1

Remove undefined behavior from PixelTrackCleanerBySharedHits.cc
Merge pull request #5458 from davidlt/eventfilter-ubsan-1

Change Word64 type from long long to unsigned long long