Merge pull request #18 from hatakeyamak/PFRecHitAndCluster_GPU_12_5_dev

Fine tunes.
jsamudio · Oct 11, 2022 · dcf25f2 · dcf25f2
2 parents 989035a + d3628bd
commit dcf25f2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/RecoParticleFlow/PFClusterProducer/plugins/PFClusterProducerCudaHCAL.cc b/RecoParticleFlow/PFClusterProducer/plugins/PFClusterProducerCudaHCAL.cc
@@ -265,7 +265,8 @@ void PFClusterProducerCudaHCAL::fillDescriptions(edm::ConfigurationDescriptions&
 }
 
 void PFClusterProducerCudaHCAL::beginLuminosityBlock(const edm::LuminosityBlock& lumi, const edm::EventSetup& es) {
-  initCuda_ = true;  // (Re)initialize cuda arrays
+  //initCuda_ = true;  // (Re)initialize cuda arrays
+  //KenH: for now comment this out, as we know we don't change the channel status on lumisection basis
 }
 
 void PFClusterProducerCudaHCAL::acquire(edm::Event const& event,
@@ -341,7 +342,7 @@ void PFClusterProducerCudaHCAL::acquire(edm::Event const& event,
                             cudaStream));
 
   cudaCheck(cudaMemcpyAsync(
-      outputCPU.pcrh_frac.get(), outputGPU.pcrh_frac.get(), sizeof(int) * nFracs, cudaMemcpyDeviceToHost, cudaStream));
+      outputCPU.pcrh_frac.get(), outputGPU.pcrh_frac.get(), sizeof(float) * nFracs, cudaMemcpyDeviceToHost, cudaStream));
   cudaCheck(cudaMemcpyAsync(
       outputCPU.pfrh_isSeed.get(), outputGPU.pfrh_isSeed.get(), numbytes_int, cudaMemcpyDeviceToHost, cudaStream));
   cudaCheck(cudaMemcpyAsync(
@@ -354,7 +355,7 @@ void PFClusterProducerCudaHCAL::acquire(edm::Event const& event,
 void PFClusterProducerCudaHCAL::produce(edm::Event& event, const edm::EventSetup& setup) {
   // cms::cuda::ScopedContextProduce ctx{cudaState_};
   // if (_produceSoA)
-  //   ctx.emplace(event, OutputPFRecHitSoA_Token_, std::move(outputGPU.PFClusters)); // SoA "PFClusters" still need to be defined.
+  //   ctx.emplace(event, OutputPFClusterSoA_Token_, std::move(outputGPU.PFClusters)); // SoA "PFClusters" still need to be defined.
 
   if (_produceLegacy) {
 

diff --git a/RecoParticleFlow/PFClusterProducer/plugins/PFHBHERecHitProducerGPU.cc b/RecoParticleFlow/PFClusterProducer/plugins/PFHBHERecHitProducerGPU.cc
@@ -235,7 +235,8 @@ void PFHBHERecHitProducerGPU::beginLuminosityBlock(edm::LuminosityBlock const& l
   }
   // -> vDenseIdHcal, validDetIdPositions
 
-  initCuda = true;  // (Re)initialize cuda arrays
+  //initCuda = true;  // (Re)initialize cuda arrays
+  //KenH: for now comment this out, as we know we don't change the channel status on lumisection basis
 }
 
 void PFHBHERecHitProducerGPU::acquire(edm::Event const& event,