Import mkFit from trackreco/mkFit to cmssw.

This commit is squashed from many commits made during the code review.
Below is a list of changes (in addition to overall cleanup):

- Build subdirs of MkFitCore/src/, put Ice source back in Ice/. Move CMS-2017 geom test to MkFitCMS.

- Move files with stand-alone steering code out of attic/.

- Use std::aligned_alloc in place of _mm_malloc.

- Reenable standalone quality and root-based validation.
  So far ohnly works for standalone runBtpCe_MultiIter() - MIMI
  even though the modifications have also been made in legacy
  best-hit, standard, and clone-engine steering functions.

- Remove standalone externals from MkFitCore.

- Remove RecoTracker/MkFitCore/standalone/dust-chest/ directory that contained nlohmann and
  SMatrix externals.

- Move them to a new github repo: trackreco/mkFit-external

- In standalone configure, clone the mkFit-external repo into buld directory.

- Modify Makefiles to include from this new directory instead of dusty-chest/.

- Make Pool<T> to return unique_ptr directly, and reduce public
  interface, follow CMS' naming convention, aling naming a bit closer
  to edm::ReusableObjectHolder.

- refactor LayerNumberConverter

- Remove std::vector inheritance of CombCandidate - use a std::vector member.

- Switch to regular std::clamp

- Significant cleanup of Config.h/cc

- Split Config.h/cc into standalone/ConfigStandalone.h/cc
  Move as much standalone stuff as possible.

- Move TrackExtra class out of Track.h/cc into standalone/TrackExtra.h/cc

- Move Conformal-fit stuff into standalone as well.

- Makefiles -- distclean removes libs and executables.

- Remove some more Config variables related to standalone sim.

- Remove usage of unnamed union/struct for bitfield initialization.

- LayerNumberConverter.h private data

- Rename most functions to start with a lower-case letter.

- HitStructures: classes LayerOfHits and EventOfHits
  - make all members private
  - proper pub/priv order
  - remove commented code
  - improve comments

- Remove unused Track::clone()

- HitStructures: classes TrackCand, CombCandidate, EventOfCombCandidates
  - make all members private, write accessors as needed
  - proper pub/priv order
  - remove commented code
  - improve comments

- Mostly miscellaneous review followup:
  - commented out code cleanup
  - drop const in returned values
  - enum-> constexpr in Matriplex.h
  - costexpr for 1620

- Remove more standalone vars, typedefs, includes from Config.h.

- include CMS_SA_ALLOW, with support for MKFIT_STANDALONE

- Move Matriplex/test to standalone/test, move define ASSUME_ALIGNED to MatriplexCommon.h

- Fixup CandCloner
  - remove unused code
  - move impl of begin/end funcs to .cc, use dprint
  - make members private

- Remove MkFitCMS/standalone/CMS-2017/. This contained data and code
  to generate CMS-2017.acc geometry description.

- Move plugin loader from TrackerInfo to ConfigStandalone

- Do not dlclose geometry/config plugin library.

- Move standalone auto-generated files of out cmssw repo (to trackreco/mkFit-external).

- Move material constants to class MaterialEffects, make a const global instance of it namespace Config.

- LayerInfo privatization + related changes and removal of some members obsoleted by layer-plan (sibling/next layer).

- TrackerInfo privatization.

- Use unique_ptr for JSON object, and remove release_json() as
  unnecessary

- default LayerInfo::m_is_stereo = false

- Split Config in Const / Config, remove unused useCMSGeom and
  nParams. Config now has to be turned into a class, some variables
  would be better off being put into IterationConfig or similar.

- Remove Config::nTotalLayers, build writeMemoryFile.

- Put propagation-flags into PropagationConfig class. Access through
  const& singleton pattern.

- Move Config::json_verbose to standalone, put json functions into
  JsonConfig class.

- Move Config::silent to standalone and a member of MkBuilder.

- Final cleanup of Config.h
  - add ool argument to PropagationConfig::set_as_default(bool force)
  - move includePCA to ConfigStandalone (for cmd-line option processing) and
    introduce new member bool PropagationConfig::backward_fit_to_pca
  - if-def MKFIT_STANDALONE numThreadsFinder numThreadsEvents numSeedsPerTask

- if-def MKFIT_STANDALONE bool removeDuplicates and useHitsForDuplicates

- use edm::isFinite with non-standalone build

- Fixofy MkBase / MkFitter / MkFinder
  - members use m_ prefix
  - privatize/protectify data members, put them to the end
  - remove some comments

- On Slava's request remove from backward-fit outlier/nan-fit hit
  rejection proto-code and discussion about why it should be
  implemented the way it was and what to do with overlaps when primary
  hit gets rejected.

- drop cmsDeltaRad and related commented-out code for rough
  propagation window from nominal r/z to the actual expected hit (this
  was set up for the CMS geometry and was replaced by the per-layer
  windows; we are not going back from this setup)

- use TrackAlgorithm instead of literals

- Code cleanup:
  - move some aged standalone code out of StdStandaloneSeqs.cc
  - remove unneeded 1.02 alocation scale in LayerOfHits
  - remove unused eta regions from TrackerInfo - superseeded by
    seed-partitioning function, layer-plan and steering-params.

RecoTracker/MkFit/BuildFile.xml

@@ -3,7 +3,8 @@
 <use name="DataFormats/TrackerCommon"/>
 <use name="Geometry/TrackerGeometryBuilder"/>
 <use name="RecoTracker/TkDetLayers"/>
-<use name="mkfit"/>
+<use name="RecoTracker/MkFitCore"/>
+<use name="RecoTracker/MkFitCMS"/>
 <use name="rootcore"/>
 <export>
   <lib name="RecoTrackerMkFit"/>

RecoTracker/MkFit/plugins/BuildFile.xml

@@ -29,7 +29,6 @@
   <use name="TrackingTools/TrackFitters"/>
   <use name="TrackingTools/TrajectoryState"/>
   <use name="TrackingTools/TransientTrackingRecHit"/>
-  <use name="mkfit"/>
   <use name="rootmath"/>
   <flags EDM_PLUGIN="1"/>
 </library>

RecoTracker/MkFit/plugins/MkFitEventOfHitsProducer.cc

@@ -25,9 +25,9 @@
 #include "RecoTracker/Record/interface/TrackerRecoGeometryRecord.h"
 
 // mkFit includes
-#include "mkFit/HitStructures.h"
-#include "mkFit/MkStdSeqs.h"
-#include "LayerNumberConverter.h"
+#include "RecoTracker/MkFitCore/interface/HitStructures.h"
+#include "RecoTracker/MkFitCMS/interface/MkStdSeqs.h"
+#include "RecoTracker/MkFitCMS/interface/LayerNumberConverter.h"
 
 class MkFitEventOfHitsProducer : public edm::global::EDProducer<> {
 public:
@@ -95,7 +95,7 @@ void MkFitEventOfHitsProducer::produce(edm::StreamID iID, edm::Event& iEvent, co
   const auto& mkFitGeom = iSetup.getData(mkFitGeomToken_);
 
   auto eventOfHits = std::make_unique<mkfit::EventOfHits>(mkFitGeom.trackerInfo());
-  mkfit::StdSeq::Cmssw_LoadHits_Begin(*eventOfHits, {&pixelHits.hits(), &stripHits.hits()});
+  mkfit::StdSeq::cmssw_LoadHits_Begin(*eventOfHits, {&pixelHits.hits(), &stripHits.hits()});
 
   if (usePixelQualityDB_ || useStripStripQualityDB_) {
     std::vector<mkfit::DeadVec> deadvectors(mkFitGeom.layerNumberConverter().nLayers());
@@ -168,16 +168,16 @@ void MkFitEventOfHitsProducer::produce(edm::StreamID iID, edm::Event& iEvent, co
         }
       }
     }
-    mkfit::StdSeq::LoadDeads(*eventOfHits, deadvectors);
+    mkfit::StdSeq::loadDeads(*eventOfHits, deadvectors);
   }
 
   fill(iEvent.get(pixelClusterIndexToHitToken_).hits(), *eventOfHits, mkFitGeom);
   fill(iEvent.get(stripClusterIndexToHitToken_).hits(), *eventOfHits, mkFitGeom);
 
-  mkfit::StdSeq::Cmssw_LoadHits_End(*eventOfHits);
+  mkfit::StdSeq::cmssw_LoadHits_End(*eventOfHits);
 
   auto const bs = iEvent.get(beamSpotToken_);
-  eventOfHits->SetBeamSpot(
+  eventOfHits->setBeamSpot(
       mkfit::BeamSpot(bs.x0(), bs.y0(), bs.z0(), bs.sigmaZ(), bs.BeamWidthX(), bs.BeamWidthY(), bs.dxdz(), bs.dydz()));
 
   iEvent.emplace(putToken_, std::move(eventOfHits));
@@ -190,7 +190,7 @@ void MkFitEventOfHitsProducer::fill(const std::vector<const TrackingRecHit*>& hi
     const auto* hit = hits[i];
     if (hit != nullptr) {
       const auto ilay = mkFitGeom.mkFitLayerNumber(hit->geographicalId());
-      eventOfHits[ilay].RegisterHit(i);
+      eventOfHits[ilay].registerHit(i);
     }
   }
 }

RecoTracker/MkFit/plugins/MkFitGeometryESProducer.cc

@@ -13,9 +13,9 @@
 #include "createPhase1TrackerGeometry.h"
 
 // mkFit includes
-#include "ConfigWrapper.h"
-#include "TrackerInfo.h"
-#include "mkFit/IterationConfig.h"
+#include "RecoTracker/MkFitCore/interface/ConfigWrapper.h"
+#include "RecoTracker/MkFitCore/interface/TrackerInfo.h"
+#include "RecoTracker/MkFitCore/interface/IterationConfig.h"
 
 #include <atomic>
 

RecoTracker/MkFit/plugins/MkFitIterationConfigESProducer.cc

@@ -6,10 +6,10 @@
 #include "RecoTracker/MkFit/interface/MkFitGeometry.h"
 
 // mkFit includes
-#include "Track.h"
-#include "TrackerInfo.h"
-#include "mkFit/HitStructures.h"
-#include "mkFit/IterationConfig.h"
+#include "RecoTracker/MkFitCore/interface/Track.h"
+#include "RecoTracker/MkFitCore/interface/TrackerInfo.h"
+#include "RecoTracker/MkFitCore/interface/HitStructures.h"
+#include "RecoTracker/MkFitCore/interface/IterationConfig.h"
 
 namespace {
   using namespace mkfit;
@@ -26,7 +26,7 @@ namespace {
       const bool z_dir_pos = S.pz() > 0;
 
       const auto &hot = S.getLastHitOnTrack();
-      const float eta = eoh[hot.layer].GetHit(hot.index).eta();
+      const float eta = eoh[hot.layer].refHit(hot.index).eta();
 
       // Region to be defined by propagation / intersection tests
       TrackerInfo::EtaRegion reg;
@@ -42,39 +42,39 @@ namespace {
       constexpr int tecp1_id = 27;
       constexpr int tecn1_id = 54;
 
-      const LayerInfo &tib1 = trk_info.m_layers[tib1_id];
-      const LayerInfo &tob1 = trk_info.m_layers[tob1_id];
+      const LayerInfo &tib1 = trk_info.layer(tib1_id);
+      const LayerInfo &tob1 = trk_info.layer(tob1_id);
 
-      const LayerInfo &tecp1 = trk_info.m_layers[tecp1_id];
-      const LayerInfo &tecn1 = trk_info.m_layers[tecn1_id];
+      const LayerInfo &tecp1 = trk_info.layer(tecp1_id);
+      const LayerInfo &tecn1 = trk_info.layer(tecn1_id);
 
       const LayerInfo &tec_first = z_dir_pos ? tecp1 : tecn1;
 
       const float maxR = S.maxReachRadius();
       float z_at_maxr;
 
-      bool can_reach_outer_brl = S.canReachRadius(outer_brl.m_rout);
+      bool can_reach_outer_brl = S.canReachRadius(outer_brl.rout());
       float z_at_outer_brl;
       bool misses_first_tec;
       if (can_reach_outer_brl) {
-        z_at_outer_brl = S.zAtR(outer_brl.m_rout);
+        z_at_outer_brl = S.zAtR(outer_brl.rout());
         if (z_dir_pos)
-          misses_first_tec = z_at_outer_brl < tec_first.m_zmin;
+          misses_first_tec = z_at_outer_brl < tec_first.zmin();
         else
-          misses_first_tec = z_at_outer_brl > tec_first.m_zmax;
+          misses_first_tec = z_at_outer_brl > tec_first.zmax();
       } else {
         z_at_maxr = S.zAtR(maxR);
         if (z_dir_pos)
-          misses_first_tec = z_at_maxr < tec_first.m_zmin;
+          misses_first_tec = z_at_maxr < tec_first.zmin();
         else
-          misses_first_tec = z_at_maxr > tec_first.m_zmax;
+          misses_first_tec = z_at_maxr > tec_first.zmax();
       }
 
       if (misses_first_tec) {
         reg = TrackerInfo::Reg_Barrel;
       } else {
-        if ((S.canReachRadius(tib1.m_rin) && tib1.is_within_z_limits(S.zAtR(tib1.m_rin))) ||
-            (S.canReachRadius(tob1.m_rin) && tob1.is_within_z_limits(S.zAtR(tob1.m_rin)))) {
+        if ((S.canReachRadius(tib1.rin()) && tib1.is_within_z_limits(S.zAtR(tib1.rin()))) ||
+            (S.canReachRadius(tob1.rin()) && tob1.is_within_z_limits(S.zAtR(tob1.rin())))) {
           reg = z_dir_pos ? TrackerInfo::Reg_Transition_Pos : TrackerInfo::Reg_Transition_Neg;
         } else {
           reg = z_dir_pos ? TrackerInfo::Reg_Endcap_Pos : TrackerInfo::Reg_Endcap_Neg;
@@ -101,26 +101,26 @@ namespace {
     constexpr int tecp1_id = 27;
     constexpr int tecn1_id = 54;
 
-    const LayerInfo &tib1 = trk_info.m_layers[tib1_id];
-    const LayerInfo &tob1 = trk_info.m_layers[tob1_id];
+    const LayerInfo &tib1 = trk_info.layer(tib1_id);
+    const LayerInfo &tob1 = trk_info.layer(tob1_id);
 
-    const LayerInfo &tidp1 = trk_info.m_layers[tidp1_id];
-    const LayerInfo &tidn1 = trk_info.m_layers[tidn1_id];
+    const LayerInfo &tidp1 = trk_info.layer(tidp1_id);
+    const LayerInfo &tidn1 = trk_info.layer(tidn1_id);
 
-    const LayerInfo &tecp1 = trk_info.m_layers[tecp1_id];
-    const LayerInfo &tecn1 = trk_info.m_layers[tecn1_id];
+    const LayerInfo &tecp1 = trk_info.layer(tecp1_id);
+    const LayerInfo &tecn1 = trk_info.layer(tecn1_id);
 
     // Merge first two layers to account for mono/stereo coverage.
     // TrackerInfo could hold joint limits for sub-detectors.
-    const auto &L = trk_info.m_layers;
-    const float tidp_rin = std::min(L[tidp1_id].m_rin, L[tidp1_id + 1].m_rin);
-    const float tidp_rout = std::max(L[tidp1_id].m_rout, L[tidp1_id + 1].m_rout);
-    const float tecp_rin = std::min(L[tecp1_id].m_rin, L[tecp1_id + 1].m_rin);
-    const float tecp_rout = std::max(L[tecp1_id].m_rout, L[tecp1_id + 1].m_rout);
-    const float tidn_rin = std::min(L[tidn1_id].m_rin, L[tidn1_id + 1].m_rin);
-    const float tidn_rout = std::max(L[tidn1_id].m_rout, L[tidn1_id + 1].m_rout);
-    const float tecn_rin = std::min(L[tecn1_id].m_rin, L[tecn1_id + 1].m_rin);
-    const float tecn_rout = std::max(L[tecn1_id].m_rout, L[tecn1_id + 1].m_rout);
+    const auto &L = trk_info;
+    const float tidp_rin = std::min(L[tidp1_id].rin(), L[tidp1_id + 1].rin());
+    const float tidp_rout = std::max(L[tidp1_id].rout(), L[tidp1_id + 1].rout());
+    const float tecp_rin = std::min(L[tecp1_id].rin(), L[tecp1_id + 1].rin());
+    const float tecp_rout = std::max(L[tecp1_id].rout(), L[tecp1_id + 1].rout());
+    const float tidn_rin = std::min(L[tidn1_id].rin(), L[tidn1_id + 1].rin());
+    const float tidn_rout = std::max(L[tidn1_id].rout(), L[tidn1_id + 1].rout());
+    const float tecn_rin = std::min(L[tecn1_id].rin(), L[tecn1_id + 1].rin());
+    const float tecn_rout = std::max(L[tecn1_id].rout(), L[tecn1_id + 1].rout());
 
     // Bias towards more aggressive transition-region assignemnts.
     // With current tunning it seems to make things a bit worse.
@@ -153,7 +153,7 @@ namespace {
       const Track &S = in_seeds[i];
 
       const auto &hot = S.getLastHitOnTrack();
-      const float eta = eoh[hot.layer].GetHit(hot.index).eta();
+      const float eta = eoh[hot.layer].refHit(hot.index).eta();
 
       // Region to be defined by propagation / intersection tests
       TrackerInfo::EtaRegion reg;
@@ -165,14 +165,14 @@ namespace {
       const float maxR = S.maxReachRadius();
 
       if (z_dir_pos) {
-        const bool in_tib = barrel_pos_check(S, maxR, tib1.m_rin, tib1.m_zmax);
-        const bool in_tob = barrel_pos_check(S, maxR, tob1.m_rin, tob1.m_zmax);
+        const bool in_tib = barrel_pos_check(S, maxR, tib1.rin(), tib1.zmax());
+        const bool in_tob = barrel_pos_check(S, maxR, tob1.rin(), tob1.zmax());
 
         if (!in_tib && !in_tob) {
           reg = TrackerInfo::Reg_Endcap_Pos;
         } else {
-          const bool in_tid = endcap_pos_check(S, maxR, tidp_rout, tidp_rin, tidp1.m_zmin - tid_z_extra);
-          const bool in_tec = endcap_pos_check(S, maxR, tecp_rout, tecp_rin, tecp1.m_zmin - tec_z_extra);
+          const bool in_tid = endcap_pos_check(S, maxR, tidp_rout, tidp_rin, tidp1.zmin() - tid_z_extra);
+          const bool in_tec = endcap_pos_check(S, maxR, tecp_rout, tecp_rin, tecp1.zmin() - tec_z_extra);
 
           if (!in_tid && !in_tec) {
             reg = TrackerInfo::Reg_Barrel;
@@ -181,14 +181,14 @@ namespace {
           }
         }
       } else {
-        const bool in_tib = barrel_neg_check(S, maxR, tib1.m_rin, tib1.m_zmin);
-        const bool in_tob = barrel_neg_check(S, maxR, tob1.m_rin, tob1.m_zmin);
+        const bool in_tib = barrel_neg_check(S, maxR, tib1.rin(), tib1.zmin());
+        const bool in_tob = barrel_neg_check(S, maxR, tob1.rin(), tob1.zmin());
 
         if (!in_tib && !in_tob) {
           reg = TrackerInfo::Reg_Endcap_Neg;
         } else {
-          const bool in_tid = endcap_neg_check(S, maxR, tidn_rout, tidn_rin, tidn1.m_zmax + tid_z_extra);
-          const bool in_tec = endcap_neg_check(S, maxR, tecn_rout, tecn_rin, tecn1.m_zmax + tec_z_extra);
+          const bool in_tid = endcap_neg_check(S, maxR, tidn_rout, tidn_rin, tidn1.zmax() + tid_z_extra);
+          const bool in_tec = endcap_neg_check(S, maxR, tecn_rout, tecn_rin, tecn1.zmax() + tec_z_extra);
 
           if (!in_tid && !in_tec) {
             reg = TrackerInfo::Reg_Barrel;
@@ -233,7 +233,8 @@ void MkFitIterationConfigESProducer::fillDescriptions(edm::ConfigurationDescript
 
 std::unique_ptr<mkfit::IterationConfig> MkFitIterationConfigESProducer::produce(
     const TrackerRecoGeometryRecord &iRecord) {
-  auto it_conf = mkfit::ConfigJson_Load_File(configFile_);
+  mkfit::ConfigJson cj;
+  auto it_conf = cj.load_File(configFile_);
   it_conf->m_partition_seeds = partitionSeeds1;
   return it_conf;
 }

RecoTracker/MkFit/plugins/MkFitOutputConverter.cc

@@ -37,9 +37,9 @@
 #include "RecoTracker/Record/interface/TrackerRecoGeometryRecord.h"
 
 // mkFit indludes
-#include "LayerNumberConverter.h"
-#include "Track.h"
-#include "mkFit/HitStructures.h"
+#include "RecoTracker/MkFitCMS/interface/LayerNumberConverter.h"
+#include "RecoTracker/MkFitCore/interface/Track.h"
+#include "RecoTracker/MkFitCore/interface/HitStructures.h"
 
 namespace {
   template <typename T>

RecoTracker/MkFit/plugins/MkFitProducer.cc

@@ -18,14 +18,14 @@
 #include "RecoTracker/Record/interface/TrackerRecoGeometryRecord.h"
 
 // mkFit includes
-#include "ConfigWrapper.h"
-#include "LayerNumberConverter.h"
-#include "mkFit/buildtestMPlex.h"
-#include "mkFit/IterationConfig.h"
-#include "mkFit/MkBuilderWrapper.h"
+#include "RecoTracker/MkFitCore/interface/ConfigWrapper.h"
+#include "RecoTracker/MkFitCMS/interface/LayerNumberConverter.h"
+#include "RecoTracker/MkFitCMS/interface/runFunctions.h"
+#include "RecoTracker/MkFitCore/interface/IterationConfig.h"
+#include "RecoTracker/MkFitCore/interface/MkBuilderWrapper.h"
 
 // TBB includes
-#include "tbb/task_arena.h"
+#include "oneapi/tbb/task_arena.h"
 
 // std includes
 #include <functional>
@@ -54,7 +54,6 @@ class MkFitProducer : public edm::global::EDProducer<edm::StreamCache<mkfit::MkB
   const edm::ESGetToken<MkFitGeometry, TrackerRecoGeometryRecord> mkFitGeomToken_;
   const edm::ESGetToken<mkfit::IterationConfig, TrackerRecoGeometryRecord> mkFitIterConfigToken_;
   const edm::EDPutTokenT<MkFitOutputWrapper> putToken_;
-  std::function<double(mkfit::Event&, mkfit::MkBuilder&)> buildFunction_;
   const float minGoodStripCharge_;
   const bool seedCleaning_;
   const bool backwardFitInCMSSW_;
@@ -101,7 +100,7 @@ MkFitProducer::MkFitProducer(edm::ParameterSet const& iConfig)
 
   // TODO: what to do when we have multiple instances of MkFitProducer in a job?
   mkfit::MkBuilderWrapper::populate();
-  mkfit::ConfigWrapper::initializeForCMSSW(mkFitSilent_);
+  mkfit::ConfigWrapper::initializeForCMSSW();
 }
 
 void MkFitProducer::fillDescriptions(edm::ConfigurationDescriptions& descriptions) {
@@ -134,12 +133,9 @@ void MkFitProducer::fillDescriptions(edm::ConfigurationDescriptions& description
 }
 
 std::unique_ptr<mkfit::MkBuilderWrapper> MkFitProducer::beginStream(edm::StreamID iID) const {
-  return std::make_unique<mkfit::MkBuilderWrapper>();
+  return std::make_unique<mkfit::MkBuilderWrapper>(mkFitSilent_);
 }
 
-namespace {
-  std::once_flag geometryFlag;
-}
 void MkFitProducer::produce(edm::StreamID iID, edm::Event& iEvent, const edm::EventSetup& iSetup) const {
   const auto& pixelHits = iEvent.get(pixelHitsToken_);
   const auto& stripHits = iEvent.get(stripHitsToken_);
@@ -180,13 +176,6 @@ void MkFitProducer::produce(edm::StreamID iID, edm::Event& iEvent, const edm::Ev
     stripClusterChargeCut(iEvent.get(stripClusterChargeToken_), stripMask);
   }
 
-  // Initialize the number of layers, has to be done exactly once in
-  // the whole program.
-  // TODO: the mechanism needs to be improved...
-  std::call_once(geometryFlag, [nlayers = mkFitGeom.layerNumberConverter().nLayers()]() {
-    mkfit::ConfigWrapper::setNTotalLayers(nlayers);
-  });
-
   // seeds need to be mutable because of the possible cleaning
   auto seeds_mutable = seeds.seeds();
   mkfit::TrackVec tracks;

RecoTracker/MkFit/plugins/MkFitSeedConverter.cc

@@ -31,8 +31,8 @@
 #include "Math/SMatrix.h"
 
 // mkFit includes
-#include "LayerNumberConverter.h"
-#include "Track.h"
+#include "RecoTracker/MkFitCMS/interface/LayerNumberConverter.h"
+#include "RecoTracker/MkFitCore/interface/Track.h"
 
 class MkFitSeedConverter : public edm::global::EDProducer<> {
 public:

RecoTracker/MkFit/plugins/convertHits.h

@@ -17,8 +17,8 @@
 #include "Math/SMatrix.h"
 
 // mkFit includes
-#include "Hit.h"
-#include "mkFit/HitStructures.h"
+#include "RecoTracker/MkFitCore/interface/Hit.h"
+#include "RecoTracker/MkFitCore/interface/HitStructures.h"
 
 namespace mkfit {
   template <typename Traits, typename HitCollection>