WIP: fix MCLinking in TDR branch for LHCb::Tracks

@sponce , @raaij , @sstahl The solution I adopted to restore MCLinking for the LHCb::Tracks -> vector conversion, is to append in the MCSequence an algorithm reading all the tracks in the Track container and make a copy of them in /Copy/Track. This allow to run the PrChecker without any issue. Maybe this can be extended to other algorithm which do not allow to merge TDR to master. @gligorov , you may need this hack to setup a timing & efficiency test for CPU using the exact same software stack. To - do -list

• Make the TrackConverter (move to functional) a 1-to-1 conversion One path In , One path Out (KeyedTrack) chained by the PrTrackAssociator using single containers. The final MCChecking sequence will be Per track type produced, i.e. MCCHeckingVelo, MCCheckingUpstream, MCCheckingForward, where optionally, whether no KeyedContainer is produced, a converter is placed in the sequence.
• Let TrackSys and UpgradeChecking sequence to know where converted Keyed tracks are located and which tracks have to be converted (this depends on the signature of the HLT algorithms, i.e. if they produce LHCb::Tracks or vectorLHCb::Track

Now, you can pass TrackSys().TracksToConvert = ["Velo","Upstream","ForwardFast"] to convert from vector<Track> to LHCb::Tracks when doing MC association. Also TrackSys().VeloUpgradeOnly = True will execute VeloOnly ( and Decoding detectors) in the sequence. The last can be useful concerning @cattanem suggestions in Brunel!344 (closed)

Calo/CaloMoniDst/src/CaloAlignmentNtp.cpp

@@ -156,7 +156,7 @@ void CaloAlignmentNtp::operator()(const Vertices& verts, const ODIN& odin,
     if( m_counterStat->isQuiet()) counter("2-selected protos")+=1;
 
     // 3: Validate track
-    const auto track = proto->track();
+    auto track = proto->track();
     if( !acceptTrack( track )) continue;
     if( m_counterStat->isQuiet()) counter("3-selected tracks")+=1;
 
@@ -294,9 +294,9 @@ void CaloAlignmentNtp::operator()(const Vertices& verts, const ODIN& odin,
       double bStCY=0;
 
       if( hasBrem ){
-        auto bState = (LHCb::State*) track->stateAt(LHCb::State::Location::BegRich1);
+        auto bState = track->stateAt(LHCb::State::Location::BegRich1);
         if( bState == nullptr ){
-          bState = track->states()[0];
+          bState = &track->firstState();
           debug() << track->states();
           Warning("BegRich1 state does not exists - used first state", StatusCode::SUCCESS).ignore();
         }