Work around: avoid assert in Phase 2 workflows (#438)

SiPixelRecHitConverter needs to produce the very same products as SiPixelRecHitSoAFromLegacy, even if they are not used.

These changes limit the size of the product to gpuClustering::MaxNumModules, avoid an assert or an overflow for Phase 2 workflows.

RecoLocalTracker/SiPixelRecHits/plugins/SiPixelRecHitConverter.cc

@@ -120,7 +120,9 @@ namespace cms {
       DetId detIdObject(detid);
       const GeomDetUnit* genericDet = geom->idToDetUnit(detIdObject);
       auto gind = genericDet->index();
-      assert(gind < 2000);
+      // FIXME to be changed to support Phase2
+      if (gind >= int(gpuClustering::MaxNumModules))
+        continue;
       auto const nclus = DSViter->size();
       assert(nclus > 0);
       clusInModule[gind] = nclus;