From 832348571ab360ae386e2939d1ab8bda65153258 Mon Sep 17 00:00:00 2001
From: Florian Lorkowski <florian.lorkowski@desy.de>
Date: Wed, 27 Sep 2023 16:54:14 +0200
Subject: [PATCH] Implement suggested changes for DataFormats/RecoParticleFlow
 subpackage

1) Merge files src/alpaka/classes_serial{.h,_def.xml} into src/classes{.h,_def.xml} and update BuildFile.xml
2) Split class definitions into two files, undoing commit 42fd4a73bb03b04983328ba453595f9bcc3045b6. This is possible due to PR cms-sw/cmsdist/pull/8705. Switching to this version required a small change to the Python configuration.
These two changes were suggested for PR cms-sw/cmssw#42807.
---
 DataFormats/ParticleFlowReco/BuildFile.xml    |   4 +-
 .../src/alpaka/classes_serial.h               |  10 --
 .../src/alpaka/classes_serial_def.xml         |  57 ---------
 DataFormats/ParticleFlowReco/src/classes_1.h  |  54 ++++++++
 .../src/{classes.h => classes_2.h}            |   5 +
 .../ParticleFlowReco/src/classes_def_1.xml    |  64 ++++++++++
 .../{classes_def.xml => classes_def_2.xml}    | 118 +++++++++---------
 .../test/pfRecHitProducerAlpakaValidation.py  |   2 -
 8 files changed, 184 insertions(+), 130 deletions(-)
 delete mode 100644 DataFormats/ParticleFlowReco/src/alpaka/classes_serial.h
 delete mode 100644 DataFormats/ParticleFlowReco/src/alpaka/classes_serial_def.xml
 create mode 100644 DataFormats/ParticleFlowReco/src/classes_1.h
 rename DataFormats/ParticleFlowReco/src/{classes.h => classes_2.h} (91%)
 create mode 100644 DataFormats/ParticleFlowReco/src/classes_def_1.xml
 rename DataFormats/ParticleFlowReco/src/{classes_def.xml => classes_def_2.xml} (77%)

diff --git a/DataFormats/ParticleFlowReco/BuildFile.xml b/DataFormats/ParticleFlowReco/BuildFile.xml
index 6149a54d0c6ef..4ea0bae8b4916 100644
--- a/DataFormats/ParticleFlowReco/BuildFile.xml
+++ b/DataFormats/ParticleFlowReco/BuildFile.xml
@@ -21,7 +21,9 @@
 <use name="DataFormats/Portable"/>
 <use name="DataFormats/SoATemplate"/>
 <use name="HeterogeneousCore/AlpakaInterface"/>
-<flags ALPAKA_BACKENDS="1"/>
+<flags LCG_DICT_HEADER="classes_1.h classes_2.h"/>
+<flags LCG_DICT_XML="classes_def_1.xml classes_def_2.xml"/>
+<flags ALPAKA_BACKENDS="cuda rocm"/>
 <export>
   <lib name="1"/>
 </export>
diff --git a/DataFormats/ParticleFlowReco/src/alpaka/classes_serial.h b/DataFormats/ParticleFlowReco/src/alpaka/classes_serial.h
deleted file mode 100644
index 2e8cc69a11c11..0000000000000
--- a/DataFormats/ParticleFlowReco/src/alpaka/classes_serial.h
+++ /dev/null
@@ -1,10 +0,0 @@
-#include "DataFormats/Common/interface/Wrapper.h"
-
-#include "DataFormats/ParticleFlowReco/interface/CaloRecHitSoA.h"
-#include "DataFormats/ParticleFlowReco/interface/CaloRecHitHostCollection.h"
-#include "DataFormats/ParticleFlowReco/interface/PFRecHitSoA.h"
-#include "DataFormats/ParticleFlowReco/interface/PFRecHitHostCollection.h"
-#include "DataFormats/ParticleFlowReco/interface/PFClusterSoA.h"
-#include "DataFormats/ParticleFlowReco/interface/PFClusterHostCollection.h"
-#include "DataFormats/ParticleFlowReco/interface/PFRecHitFractionSoA.h"
-#include "DataFormats/ParticleFlowReco/interface/PFRecHitFractionHostCollection.h"
diff --git a/DataFormats/ParticleFlowReco/src/alpaka/classes_serial_def.xml b/DataFormats/ParticleFlowReco/src/alpaka/classes_serial_def.xml
deleted file mode 100644
index 4ac11f2018f00..0000000000000
--- a/DataFormats/ParticleFlowReco/src/alpaka/classes_serial_def.xml
+++ /dev/null
@@ -1,57 +0,0 @@
-<lcgdict>
-  <class name="reco::CaloRecHitHostCollection"/>
-  <read
-    sourceClass="reco::CaloRecHitHostCollection"
-    targetClass="reco::CaloRecHitHostCollection"
-    version="[1-]"
-    source="reco::CaloRecHitSoA layout_;"
-    target="buffer_,layout_,view_"
-    embed="false">
-  <![CDATA[
-    reco::CaloRecHitHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
-  ]]>
-  </read>
-  <class name="edm::Wrapper<reco::CaloRecHitHostCollection>" splitLevel="0"/>
-
-  <class name="reco::PFRecHitHostCollection"/>
-  <read
-    sourceClass="reco::PFRecHitHostCollection"
-    targetClass="reco::PFRecHitHostCollection"
-    version="[1-]"
-    source="reco::PFRecHitSoA layout_;"
-    target="buffer_,layout_,view_"
-    embed="false">
-  <![CDATA[
-    reco::PFRecHitHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
-  ]]>
-  </read>
-  <class name="edm::Wrapper<reco::PFRecHitHostCollection>" splitLevel="0"/>
-
-  <class name="reco::PFClusterHostCollection"/>
-  <read
-    sourceClass="reco::PFClusterHostCollection"
-    targetClass="reco::PFClusterHostCollection"
-    version="[1-]"
-    source="reco::PFClusterSoA layout_;"
-    target="buffer_,layout_,view_"
-    embed="false">
-  <![CDATA[
-    reco::PFClusterHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
-  ]]>
-  </read>
-  <class name="edm::Wrapper<reco::PFClusterHostCollection>" splitLevel="0"/>
-
-  <class name="reco::PFRecHitFractionHostCollection"/>
-  <read
-    sourceClass="reco::PFRecHitFractionHostCollection"
-    targetClass="reco::PFRecHitFractionHostCollection"
-    version="[1-]"
-    source="reco::PFRecHitFractionSoA layout_;"
-    target="buffer_,layout_,view_"
-    embed="false">
-  <![CDATA[
-    reco::PFRecHitFractionHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
-  ]]>
-  </read>
-  <class name="edm::Wrapper<reco::PFRecHitFractionHostCollection>" splitLevel="0"/>
-</lcgdict>
diff --git a/DataFormats/ParticleFlowReco/src/classes_1.h b/DataFormats/ParticleFlowReco/src/classes_1.h
new file mode 100644
index 0000000000000..874750a6bf83f
--- /dev/null
+++ b/DataFormats/ParticleFlowReco/src/classes_1.h
@@ -0,0 +1,54 @@
+#include "DataFormats/Common/interface/Wrapper.h"
+#include "DataFormats/Common/interface/Ref.h"
+#include <DataFormats/Common/interface/OwnVector.h>
+#include <DataFormats/Common/interface/ClonePolicy.h>
+#include "DataFormats/Common/interface/ValueMap.h"
+
+#include "DataFormats/ParticleFlowReco/interface/PFCluster.h"
+#include "Math/Cartesian3D.h"
+#include "DataFormats/ParticleFlowReco/interface/PFRecHit.h"
+#include "Math/Polar3D.h"
+#include "Math/CylindricalEta3D.h"
+#include "DataFormats/ParticleFlowReco/interface/PFRecTrack.h"
+#include "DataFormats/ParticleFlowReco/interface/GsfPFRecTrack.h"
+#include "Math/GenVector/PositionVector3D.h"
+#include "DataFormats/Math/interface/Point3D.h"
+#include "Rtypes.h"
+#include "DataFormats/Math/interface/Vector3D.h"
+#include "Math/PxPyPzE4D.h"
+#include "DataFormats/DetId/interface/DetId.h"
+#include "DataFormats/TrackReco/interface/Track.h"
+#include "DataFormats/TrackReco/interface/TrackFwd.h"
+#include "DataFormats/GsfTrackReco/interface/GsfTrack.h"
+#include "DataFormats/GsfTrackReco/interface/GsfTrackExtra.h"
+#include "DataFormats/ParticleFlowReco/interface/PFSimParticle.h"
+#include "DataFormats/ParticleFlowReco/interface/PFTrajectoryPoint.h"
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElement.h"
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElementTrack.h"
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElementGsfTrack.h"      //Daniele
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElementBrem.h"          //Daniele
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElementSuperCluster.h"  //Florian
+#include "DataFormats/ParticleFlowReco/interface/PFBlockElementCluster.h"
+#include "DataFormats/ParticleFlowReco/interface/PFBlock.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedTrackerVertex.h"
+#include "DataFormats/ParticleFlowReco/interface/PFNuclearInteraction.h"
+#include "DataFormats/ParticleFlowReco/interface/PFConversion.h"
+#include "DataFormats/ParticleFlowReco/interface/PFConversionFwd.h"
+#include "DataFormats/ParticleFlowReco/interface/PFV0.h"
+#include "DataFormats/ParticleFlowReco/interface/ConvBremSeed.h"
+#include "DataFormats/ParticleFlowReco/interface/ConvBremSeedFwd.h"
+#include "DataFormats/ParticleFlowReco/interface/ParticleFiltrationDecision.h"
+
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertexCandidate.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertexSeed.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertex.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertexCandidateFwd.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertexSeedFwd.h"
+#include "DataFormats/ParticleFlowReco/interface/PFDisplacedVertexFwd.h"
+
+#include "DataFormats/ParticleFlowReco/interface/PreId.h"
+#include "DataFormats/ParticleFlowReco/interface/PreIdFwd.h"
+#include "DataFormats/ParticleFlowReco/interface/RecoPFClusterRefCandidate.h"
+#include "DataFormats/ParticleFlowReco/interface/RecoPFClusterRefCandidateFwd.h"
+
+#include <map>
diff --git a/DataFormats/ParticleFlowReco/src/classes.h b/DataFormats/ParticleFlowReco/src/classes_2.h
similarity index 91%
rename from DataFormats/ParticleFlowReco/src/classes.h
rename to DataFormats/ParticleFlowReco/src/classes_2.h
index 4e81f96938cce..43d5f52cec886 100644
--- a/DataFormats/ParticleFlowReco/src/classes.h
+++ b/DataFormats/ParticleFlowReco/src/classes_2.h
@@ -56,6 +56,11 @@
 #include <map>
 
 #include "DataFormats/ParticleFlowReco/interface/CaloRecHitSoA.h"
+#include "DataFormats/ParticleFlowReco/interface/CaloRecHitHostCollection.h"
 #include "DataFormats/ParticleFlowReco/interface/PFRecHitSoA.h"
+#include "DataFormats/ParticleFlowReco/interface/PFRecHitHostCollection.h"
+
 #include "DataFormats/ParticleFlowReco/interface/PFClusterSoA.h"
+#include "DataFormats/ParticleFlowReco/interface/PFClusterHostCollection.h"
 #include "DataFormats/ParticleFlowReco/interface/PFRecHitFractionSoA.h"
+#include "DataFormats/ParticleFlowReco/interface/PFRecHitFractionHostCollection.h"
diff --git a/DataFormats/ParticleFlowReco/src/classes_def_1.xml b/DataFormats/ParticleFlowReco/src/classes_def_1.xml
new file mode 100644
index 0000000000000..58b21e59e3c35
--- /dev/null
+++ b/DataFormats/ParticleFlowReco/src/classes_def_1.xml
@@ -0,0 +1,64 @@
+<lcgdict>
+<selection>
+
+
+  <class name="reco::PFNuclearInteraction"  ClassVersion="10">
+   <version ClassVersion="10" checksum="1573561360"/>
+  </class>
+  <class name="std::vector<reco::PFNuclearInteraction>" />
+  <class name="edm::Wrapper<std::vector<reco::PFNuclearInteraction> >" />
+  <class name="edm::Ref<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction, edm::refhelper::FindUsingAdvance<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction> >" />
+  <class name="edm::RefProd<std::vector<reco::PFNuclearInteraction> >" />
+  <class name="edm::RefVector<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction, edm::refhelper::FindUsingAdvance<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction> >" />
+
+
+  <class name="reco::PFDisplacedTrackerVertex"  ClassVersion="10">
+   <version ClassVersion="10" checksum="2059737151"/>
+  </class>
+  <class name="std::vector<reco::PFDisplacedTrackerVertex>" />
+  <class name="edm::Wrapper<std::vector<reco::PFDisplacedTrackerVertex> >" />
+  <class name="edm::Ref<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex, edm::refhelper::FindUsingAdvance<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex> >" />
+  <class name="edm::RefProd<std::vector<reco::PFDisplacedTrackerVertex> >" />
+  <class name="edm::RefVector<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex, edm::refhelper::FindUsingAdvance<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex> >" />
+
+  <class name="reco::PFConversion" ClassVersion="10">
+   <version ClassVersion="10" checksum="2417207933"/>
+  </class>
+  <class name="std::vector<reco::PFConversion>"/>
+  <class name="edm::Wrapper<std::vector<reco::PFConversion> >"/>
+  <class name="edm::Ref<std::vector<reco::PFConversion>,reco::PFConversion,edm::refhelper::FindUsingAdvance<std::vector<reco::PFConversion>,reco::PFConversion> >"/>
+  <class name="edm::RefProd<std::vector<reco::PFConversion> >"/>
+  <class name="edm::RefVector<std::vector<reco::PFConversion>,reco::PFConversion,edm::refhelper::FindUsingAdvance<std::vector<reco::PFConversion>,reco::PFConversion> >"/>
+  <class name="std::vector<edm::Ref<std::vector<reco::PFRecTrack>,reco::PFRecTrack,edm::refhelper::FindUsingAdvance<std::vector<reco::PFRecTrack>,reco::PFRecTrack> > >"/>
+
+ <class name="reco::PFV0"  ClassVersion="10">
+  <version ClassVersion="10" checksum="40832700"/>
+ </class>
+  <class name="std::vector<reco::PFV0>" />
+  <class name="edm::Wrapper<std::vector<reco::PFV0> >" />
+  <class name="edm::Ref<std::vector<reco::PFV0>, reco::PFV0, edm::refhelper::FindUsingAdvance<std::vector<reco::PFV0>, reco::PFV0> >" />
+  <class name="edm::RefProd<std::vector<reco::PFV0> >" />
+  <class name="edm::RefVector<std::vector<reco::PFV0>, reco::PFV0, edm::refhelper::FindUsingAdvance<std::vector<reco::PFV0>, reco::PFV0> >" />
+<class name="edm::Ref<std::vector<reco::VertexCompositeCandidate>,reco::VertexCompositeCandidate,edm::refhelper::FindUsingAdvance<std::vector<reco::VertexCompositeCandidate>,reco::VertexCompositeCandidate> >" />
+
+
+  <class name="reco::ConvBremSeed" ClassVersion="12">
+   <version ClassVersion="10" checksum="2980747190"/>
+   <version ClassVersion="11" checksum="637662266"/>
+   <version ClassVersion="12" checksum="2604169839"/>
+  </class>
+  <class name="std::vector<reco::ConvBremSeed>"/>
+  <class name="edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> >"/>
+  <class name="edm::RefProd<std::vector<reco::ConvBremSeed> >"/>
+  <class name="edm::Wrapper<std::vector<reco::ConvBremSeed> >"/>
+  <class name="edm::RefVector<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> >"/>
+  <class name="edm::Wrapper<edm::RefVector<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
+  <class name="edm::RefToBase<reco::ConvBremSeed>"/>
+  <class name="edm::reftobase::BaseHolder<reco::ConvBremSeed>" />
+  <class name="edm::reftobase::RefHolder<edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
+  <class name="edm::reftobase::Holder<reco::ConvBremSeed,edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
+
+
+</selection>
+</lcgdict>
+
diff --git a/DataFormats/ParticleFlowReco/src/classes_def.xml b/DataFormats/ParticleFlowReco/src/classes_def_2.xml
similarity index 77%
rename from DataFormats/ParticleFlowReco/src/classes_def.xml
rename to DataFormats/ParticleFlowReco/src/classes_def_2.xml
index e4ceffd2c9417..badc708629e8c 100644
--- a/DataFormats/ParticleFlowReco/src/classes_def.xml
+++ b/DataFormats/ParticleFlowReco/src/classes_def_2.xml
@@ -10,7 +10,64 @@
   <class name="reco::PFRecHitFractionSoA"/>
   <class name="reco::PFRecHitFractionSoA::View"/>
 
-  <!-- from classes_def_2.xml -->
+  <class name="reco::CaloRecHitHostCollection"/>
+  <read
+    sourceClass="reco::CaloRecHitHostCollection"
+    targetClass="reco::CaloRecHitHostCollection"
+    version="[1-]"
+    source="reco::CaloRecHitSoA layout_;"
+    target="buffer_,layout_,view_"
+    embed="false">
+  <![CDATA[
+    reco::CaloRecHitHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
+  ]]>
+  </read>
+  <class name="edm::Wrapper<reco::CaloRecHitHostCollection>" splitLevel="0"/>
+
+  <class name="reco::PFRecHitHostCollection"/>
+  <read
+    sourceClass="reco::PFRecHitHostCollection"
+    targetClass="reco::PFRecHitHostCollection"
+    version="[1-]"
+    source="reco::PFRecHitSoA layout_;"
+    target="buffer_,layout_,view_"
+    embed="false">
+  <![CDATA[
+    reco::PFRecHitHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
+  ]]>
+  </read>
+  <class name="edm::Wrapper<reco::PFRecHitHostCollection>" splitLevel="0"/>
+
+
+  <class name="reco::PFClusterHostCollection"/>
+  <read
+    sourceClass="reco::PFClusterHostCollection"
+    targetClass="reco::PFClusterHostCollection"
+    version="[1-]"
+    source="reco::PFClusterSoA layout_;"
+    target="buffer_,layout_,view_"
+    embed="false">
+  <![CDATA[
+    reco::PFClusterHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
+  ]]>
+  </read>
+  <class name="edm::Wrapper<reco::PFClusterHostCollection>" splitLevel="0"/>
+
+  <class name="reco::PFRecHitFractionHostCollection"/>
+  <read
+    sourceClass="reco::PFRecHitFractionHostCollection"
+    targetClass="reco::PFRecHitFractionHostCollection"
+    version="[1-]"
+    source="reco::PFRecHitFractionSoA layout_;"
+    target="buffer_,layout_,view_"
+    embed="false">
+  <![CDATA[
+    reco::PFRecHitFractionHostCollection::ROOTReadStreamer(newObj, onfile.layout_);
+  ]]>
+  </read>
+  <class name="edm::Wrapper<reco::PFRecHitFractionHostCollection>" splitLevel="0"/>
+
+
   <class name="reco::PFCluster" ClassVersion="17">
    <version ClassVersion="17" checksum="3021111509"/>
    <version ClassVersion="16" checksum="1786894710"/>
@@ -312,68 +369,9 @@
   <class name="edm::reftobase::RefVectorHolder<reco::RecoPFClusterRefCandidateRefVector>" />
 
   <class name="edm::Wrapper<edm::RefVector<std::vector<reco::RecoPFClusterRefCandidate>,reco::RecoPFClusterRefCandidate,edm::refhelper::FindUsingAdvance<std::vector<reco::RecoPFClusterRefCandidate>,reco::RecoPFClusterRefCandidate> > >" />
-
-  <!-- from classes_def_1.xml -->
-  <class name="reco::PFNuclearInteraction"  ClassVersion="10">
-   <version ClassVersion="10" checksum="1573561360"/>
-  </class>
-  <class name="std::vector<reco::PFNuclearInteraction>" />
-  <class name="edm::Wrapper<std::vector<reco::PFNuclearInteraction> >" />
-  <class name="edm::Ref<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction, edm::refhelper::FindUsingAdvance<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction> >" />
-  <class name="edm::RefProd<std::vector<reco::PFNuclearInteraction> >" />
-  <class name="edm::RefVector<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction, edm::refhelper::FindUsingAdvance<std::vector<reco::PFNuclearInteraction>, reco::PFNuclearInteraction> >" />
-
-
-  <class name="reco::PFDisplacedTrackerVertex"  ClassVersion="10">
-   <version ClassVersion="10" checksum="2059737151"/>
-  </class>
-  <class name="std::vector<reco::PFDisplacedTrackerVertex>" />
-  <class name="edm::Wrapper<std::vector<reco::PFDisplacedTrackerVertex> >" />
-  <class name="edm::Ref<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex, edm::refhelper::FindUsingAdvance<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex> >" />
-  <class name="edm::RefProd<std::vector<reco::PFDisplacedTrackerVertex> >" />
-  <class name="edm::RefVector<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex, edm::refhelper::FindUsingAdvance<std::vector<reco::PFDisplacedTrackerVertex>, reco::PFDisplacedTrackerVertex> >" />
-
-  <class name="reco::PFConversion" ClassVersion="10">
-   <version ClassVersion="10" checksum="2417207933"/>
-  </class>
-  <class name="std::vector<reco::PFConversion>"/>
-  <class name="edm::Wrapper<std::vector<reco::PFConversion> >"/>
-  <class name="edm::Ref<std::vector<reco::PFConversion>,reco::PFConversion,edm::refhelper::FindUsingAdvance<std::vector<reco::PFConversion>,reco::PFConversion> >"/>
-  <class name="edm::RefProd<std::vector<reco::PFConversion> >"/>
-  <class name="edm::RefVector<std::vector<reco::PFConversion>,reco::PFConversion,edm::refhelper::FindUsingAdvance<std::vector<reco::PFConversion>,reco::PFConversion> >"/>
-  <class name="std::vector<edm::Ref<std::vector<reco::PFRecTrack>,reco::PFRecTrack,edm::refhelper::FindUsingAdvance<std::vector<reco::PFRecTrack>,reco::PFRecTrack> > >"/>
-
- <class name="reco::PFV0"  ClassVersion="10">
-  <version ClassVersion="10" checksum="40832700"/>
- </class>
-  <class name="std::vector<reco::PFV0>" />
-  <class name="edm::Wrapper<std::vector<reco::PFV0> >" />
-  <class name="edm::Ref<std::vector<reco::PFV0>, reco::PFV0, edm::refhelper::FindUsingAdvance<std::vector<reco::PFV0>, reco::PFV0> >" />
-  <class name="edm::RefProd<std::vector<reco::PFV0> >" />
-  <class name="edm::RefVector<std::vector<reco::PFV0>, reco::PFV0, edm::refhelper::FindUsingAdvance<std::vector<reco::PFV0>, reco::PFV0> >" />
-<class name="edm::Ref<std::vector<reco::VertexCompositeCandidate>,reco::VertexCompositeCandidate,edm::refhelper::FindUsingAdvance<std::vector<reco::VertexCompositeCandidate>,reco::VertexCompositeCandidate> >" />
-
-
-  <class name="reco::ConvBremSeed" ClassVersion="12">
-   <version ClassVersion="10" checksum="2980747190"/>
-   <version ClassVersion="11" checksum="637662266"/>
-   <version ClassVersion="12" checksum="2604169839"/>
-  </class>
-  <class name="std::vector<reco::ConvBremSeed>"/>
-  <class name="edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> >"/>
-  <class name="edm::RefProd<std::vector<reco::ConvBremSeed> >"/>
-  <class name="edm::Wrapper<std::vector<reco::ConvBremSeed> >"/>
-  <class name="edm::RefVector<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> >"/>
-  <class name="edm::Wrapper<edm::RefVector<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
-  <class name="edm::RefToBase<reco::ConvBremSeed>"/>
-  <class name="edm::reftobase::BaseHolder<reco::ConvBremSeed>" />
-  <class name="edm::reftobase::RefHolder<edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
-  <class name="edm::reftobase::Holder<reco::ConvBremSeed,edm::Ref<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed,edm::refhelper::FindUsingAdvance<std::vector<reco::ConvBremSeed>,reco::ConvBremSeed> > >"/>
-
 </selection>
 
 <exclusion>
-  <!-- from classes_def_2.xml -->
   <class name="edm::OwnVector<reco::PFBlockElement, edm::ClonePolicy<reco::PFBlockElement> >">
     <method name="sort"/>
   </class>
diff --git a/RecoParticleFlow/PFRecHitProducer/test/pfRecHitProducerAlpakaValidation.py b/RecoParticleFlow/PFRecHitProducer/test/pfRecHitProducerAlpakaValidation.py
index a70f1ca162ec8..dd752ec948e52 100644
--- a/RecoParticleFlow/PFRecHitProducer/test/pfRecHitProducerAlpakaValidation.py
+++ b/RecoParticleFlow/PFRecHitProducer/test/pfRecHitProducerAlpakaValidation.py
@@ -40,10 +40,8 @@
 )
 
 process.options = cms.untracked.PSet(
-    #FailPath = cms.untracked.vstring(),
     IgnoreCompletely = cms.untracked.vstring(),
     Rethrow = cms.untracked.vstring(),
-    #SkipEvent = cms.untracked.vstring(),
     allowUnscheduled = cms.obsolete.untracked.bool,
     canDeleteEarly = cms.untracked.vstring(),
     deleteNonConsumedUnscheduledModules = cms.untracked.bool(True),