[PWGJE] eventWeight fix for fake collisions

Author: aimeric-landou

PWGJE/TableProducer/derivedDataProducer.cxx

@@ -241,11 +241,10 @@ struct JetDerivedDataProducerTask {
 
   void processMcCollisionLabels(soa::Join<aod::Collisions, aod::McCollisionLabels>::iterator const& collision, aod::McCollisions const&)
   {
+    products.jMcCollisionsLabelTable(collision.mcCollisionId()); // collision.mcCollisionId() returns -1 if collision has no associated mcCollision
     if (collision.has_mcCollision()) {
-      products.jMcCollisionsLabelTable(collision.mcCollisionId());
       products.jCollisionMcInfosTable(collision.mcCollision().weight(), collision.mcCollision().getSubGeneratorId());
     } else {
-      products.jMcCollisionsLabelTable(-1);
       products.jCollisionMcInfosTable(-1.0, jetderiveddatautilities::JCollisionSubGeneratorId::none);
     }
   }

PWGJE/TableProducer/jetEventWeightMCD.cxx

@@ -40,14 +40,22 @@ struct JetEventWeightMCDTask {
   void processMCDetectorLevelEventWeight(MCDetectorLevelJetTable const& jet, soa::Join<aod::JetCollisions, aod::JMcCollisionLbs> const&, aod::JetMcCollisions const&)
   {
     auto collision = jet.template collision_as<soa::Join<aod::JetCollisions, aod::JMcCollisionLbs>>();
-    mcDetectorLevelWeightsTable(jet.globalIndex(), collision.mcCollision().weight());
+    float eventWeight = 0.0;
+    if (collision.has_mcCollision()) {
+      eventWeight = (float)(collision.mcCollision()).weight();
+    }
+    mcDetectorLevelWeightsTable(jet.globalIndex(), eventWeight);
   }
   PROCESS_SWITCH(JetEventWeightMCDTask, processMCDetectorLevelEventWeight, "Fill event weight tables for detector level MC jets", false);
 
   void processMCDetectorLevelEventWiseSubtractedEventWeight(MCDetectorLevelEventWiseSubtractedJetTable const& jet, soa::Join<aod::JetCollisions, aod::JMcCollisionLbs> const&, aod::JetMcCollisions const&)
   {
     auto collision = jet.template collision_as<soa::Join<aod::JetCollisions, aod::JMcCollisionLbs>>();
-    mcDetectorLevelEventWiseSubtractedWeightsTable(jet.globalIndex(), collision.mcCollision().weight());
+    float eventWeight = 0.0;
+    if (collision.has_mcCollision()) {
+      eventWeight = (float)(collision.mcCollision()).weight();
+    }
+    mcDetectorLevelEventWiseSubtractedWeightsTable(jet.globalIndex(), eventWeight);
   }
   PROCESS_SWITCH(JetEventWeightMCDTask, processMCDetectorLevelEventWiseSubtractedEventWeight, "Fill event weight tables for detector level MC jets", false);
 };

PWGJE/Tasks/jetFinderQA.cxx

@@ -1180,7 +1180,10 @@ struct JetFinderQATask {
                              aod::JetMcCollisions const&,
                              soa::Filtered<soa::Join<aod::JetTracks, aod::JTrackExtras>> const& tracks)
   {
-    float eventWeight = collision.mcCollision().weight();
+    float eventWeight = 0.0;
+    if (collision.has_mcCollision()) {
+      eventWeight = (float)(collision.mcCollision()).weight();
+    }
     if (skipMBGapEvents && collision.subGeneratorId() == jetderiveddatautilities::JCollisionSubGeneratorId::mbGap) {
       return;
     }