diff --git a/reco/detectors/sts/unpack/CbmStsUnpackAlgo.cxx b/reco/detectors/sts/unpack/CbmStsUnpackAlgo.cxx index 279e6f0deceee1522c5484b3b245f426726edd95..5af3f3d1f052d8dbc8e89bccf03474ad3955226c 100644 --- a/reco/detectors/sts/unpack/CbmStsUnpackAlgo.cxx +++ b/reco/detectors/sts/unpack/CbmStsUnpackAlgo.cxx @@ -457,7 +457,6 @@ void CbmStsUnpackAlgo::processHitInfo(const stsxyter::Message& mess) // Get the asix index uint32_t uAsicIdx = getAsicIndex(fuCurrDpbIdx, uCrobIdx, usElinkIdx); - const uint16_t usChan = mess.GetHitChannel(); const uint16_t usRawAdc = mess.GetHitAdc(); const uint16_t usRawTs = mess.GetHitTimeBinning(); @@ -510,7 +509,6 @@ void CbmStsUnpackAlgo::processHitInfo(const stsxyter::Message& mess) fviFebSide[uFebIdx]); } - fOutputVec.emplace_back(CbmStsDigi(fviFebAddress[uFebIdx], uChanInMod, ulTimeInNs, dCalAdc)); // REMARK This seems to be double looping, I do not see any reason here, to first store the stsxyter hits in a vector over which we loop afterwards again. May be I miss something @sts-experts? // fvmHitsInMs.push_back(stsxyter::FinalHit(ulHitTime, usRawAdc, uAsicIdx, usChan, fuCurrDpbIdx, uCrobIdx)); @@ -617,11 +615,6 @@ void CbmStsUnpackAlgo::processTsMsbInfo(const stsxyter::Message& mess, uint32_t fMonitor->FillStsDpbRawTsMsb(fuCurrDpbIdx, fvulCurrentTsMsb[fuCurrDpbIdx]); fMonitor->FillStsDpbRawTsMsbSx(fuCurrDpbIdx, fvulCurrentTsMsb[fuCurrDpbIdx]); fMonitor->FillStsDpbRawTsMsbDpb(fuCurrDpbIdx, fvulCurrentTsMsb[fuCurrDpbIdx]); - const uint16_t usElinkIdx = mess.GetStatusLink(); - const uint32_t uCrobIdx = usElinkIdx / fNrElinksPerCrob; - const uint32_t uAsicIdx = getAsicIndex(fuCurrDpbIdx, uCrobIdx, usElinkIdx); - - fMonitor->FillStsAsicTsMsb(fvulCurrentTsMsb[fuCurrDpbIdx], uAsicIdx); } } diff --git a/reco/detectors/sts/unpack/CbmStsUnpackMonitor.cxx b/reco/detectors/sts/unpack/CbmStsUnpackMonitor.cxx index 07acc65f5a9508314f9f8e0e0576783eb054b43b..d2052233a995b82111684c1583cd88b7d44ef58e 100644 --- a/reco/detectors/sts/unpack/CbmStsUnpackMonitor.cxx +++ b/reco/detectors/sts/unpack/CbmStsUnpackMonitor.cxx @@ -441,12 +441,6 @@ Bool_t CbmStsUnpackMonitor::CreateDebugHistograms(CbmMcbm2018StsPar* pUnpackPar) fhStsDpbRawTsMsbDpb->SetCanExtend(TH2::kAllAxes); AddHistoToVector(fhStsDpbRawTsMsbDpb, ""); - sHistName = "fhStsAsicTsMsb"; - title = "MSB messages for each Asic; TsMsb; Asic; Count []"; - fhStsAsicTsMsb = new TH2I(sHistName, title, 10, 0, 10, uNbAsics, 0, uNbAsics); - fhStsAsicTsMsb->SetCanExtend(TH2::kAllAxes); - AddHistoToVector(fhStsAsicTsMsb, ""); - /// Asic plots /// All histos per Asic: with channels or ASIC as axis!! for (UInt_t uAsicIdx = 0; uAsicIdx < uNbAsics; ++uAsicIdx) { @@ -502,7 +496,6 @@ Bool_t CbmStsUnpackMonitor::ResetDebugHistograms() fhStsDpbRawTsMsb->Reset(); fhStsDpbRawTsMsbSx->Reset(); fhStsDpbRawTsMsbDpb->Reset(); - fhStsAsicTsMsb->Reset(); for (UInt_t uAsicIdx = 0; uAsicIdx < fvhStsChanCntRaw.size(); ++uAsicIdx) { fvhStsChanCntRaw[uAsicIdx]->Reset(); diff --git a/reco/detectors/sts/unpack/CbmStsUnpackMonitor.h b/reco/detectors/sts/unpack/CbmStsUnpackMonitor.h index ab1536a39ab31bf3a6f295101f936efa9509502e..0811c58b81d8c48f6218eb60b9a88b2f35873d03 100644 --- a/reco/detectors/sts/unpack/CbmStsUnpackMonitor.h +++ b/reco/detectors/sts/unpack/CbmStsUnpackMonitor.h @@ -164,7 +164,6 @@ public: { fhStsDpbRawTsMsbDpb->Fill(uCurrDpbIdx, (ulCurrentTsMsb >> 5)); } - void FillStsAsicTsMsb(ULong_t ulCurrentTsMsb, UInt_t uAsicIdx) { fhStsAsicTsMsb->Fill(ulCurrentTsMsb, uAsicIdx); } ///Fill debugging "per Asic" histogram vectors void FillStsChanCntRaw(UInt_t uAsicIdx, UShort_t usChan) { fvhStsChanCntRaw[uAsicIdx]->Fill(usChan); } @@ -260,7 +259,6 @@ private: TH2* fhStsDpbRawTsMsb = nullptr; TH2* fhStsDpbRawTsMsbSx = nullptr; TH2* fhStsDpbRawTsMsbDpb = nullptr; - TH2* fhStsAsicTsMsb = nullptr; static const UInt_t kiMaxNbFlibLinks = 32; TH1* fvhMsSize[kiMaxNbFlibLinks];