Skip to content

Commit a36119d

Browse files
[PWGDQ] Fix emu prefilter withassoc readers (#16683)
1 parent f985f56 commit a36119d

2 files changed

Lines changed: 10 additions & 12 deletions

File tree

PWGDQ/Tasks/dqEfficiency_withAssoc.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1186,6 +1186,7 @@ struct AnalysisPrefilterSelection {
11861186
if (context.mOptions.get<bool>("processDummy")) {
11871187
return;
11881188
}
1189+
VarManager::SetDefaultVarNames();
11891190

11901191
bool runPrefilter = true;
11911192
// get the list of track cuts to be prefiltered
@@ -1253,7 +1254,6 @@ struct AnalysisPrefilterSelection {
12531254
}
12541255

12551256
VarManager::SetUseVars(AnalysisCut::fgUsedVars); // provide the list of required variables so that VarManager knows what to fill
1256-
VarManager::SetDefaultVarNames();
12571257

12581258
VarManager::SetupTwoProngDCAFitter(5.0f, true, 200.0f, 4.0f, 1.0e-3f, 0.9f, true); // TODO: get these parameters from Configurables
12591259
VarManager::SetupTwoProngFwdDCAFitter(5.0f, true, 200.0f, 1.0e-3f, 0.9f, true);

PWGDQ/Tasks/tableReader_withAssoc.cxx

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1400,7 +1400,6 @@ struct AnalysisSameEventPairing {
14001400
NoBinningPolicy<aod::dqanalysisflags::MixingHash> hashBin;
14011401

14021402
Preslice<soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts, aod::Prefilter>> trackAssocsPerCollision = aod::reducedtrack_association::reducedeventId;
1403-
Preslice<soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts>> trackEmuAssocsPerCollision = aod::reducedtrack_association::reducedeventId;
14041403
Preslice<soa::Join<aod::ReducedMuonsAssoc, aod::MuonTrackCuts>> muonAssocsPerCollision = aod::reducedtrack_association::reducedeventId;
14051404

14061405
void init(o2::framework::InitContext& context)
@@ -2656,9 +2655,8 @@ struct AnalysisSameEventPairing {
26562655

26572656
// Custom combination policy
26582657
for (auto& [a1, a2] : o2::soa::combinations(soa::CombinationsFullIndexPolicy(groupedAssocs1, groupedAssocs2))) {
2659-
if (!(a1.isBarrelSelected_raw() & fTrackFilterMask))
2658+
if (!(a1.isBarrelSelected_raw() & a1.isBarrelSelectedPrefilter_raw() & fTrackFilterMask))
26602659
continue;
2661-
// if (!a1.isBarrelSelectedPrefilter_raw()) continue;
26622660
if (!(a2.isMuonSelected_raw() & fMuonFilterMask))
26632661
continue;
26642662

@@ -2670,7 +2668,7 @@ struct AnalysisSameEventPairing {
26702668
twoTrackFilter = 0;
26712669
int minCuts = std::min(fNCutsBarrel, fNCutsMuon);
26722670
for (int i = 0; i < minCuts; ++i) {
2673-
if ((a1.isBarrelSelected_raw() & (1u << i)) && (a2.isMuonSelected_raw() & (1u << i))) {
2671+
if ((a1.isBarrelSelected_raw() & a1.isBarrelSelectedPrefilter_raw() & (1u << i)) && (a2.isMuonSelected_raw() & (1u << i))) {
26742672
twoTrackFilter |= (1u << i);
26752673
}
26762674
}
@@ -2699,7 +2697,7 @@ struct AnalysisSameEventPairing {
26992697
t1.sign() + t2.sign(), twoTrackFilter, 0);
27002698

27012699
for (int iTrack = 0; iTrack < fNCutsBarrel; ++iTrack) {
2702-
if (!(a1.isBarrelSelected_raw() & (1u << iTrack)))
2700+
if (!(a1.isBarrelSelected_raw() & a1.isBarrelSelectedPrefilter_raw() & (1u << iTrack)))
27032701
continue;
27042702

27052703
for (int iMuon = 0; iMuon < fNCutsMuon; ++iMuon) {
@@ -2744,7 +2742,7 @@ struct AnalysisSameEventPairing {
27442742
constexpr bool eventHasQvectorCentr = ((TEventFillMap & VarManager::ObjTypes::CollisionQvect) > 0);
27452743

27462744
for (auto& a1 : assocs1) {
2747-
if (!(a1.isBarrelSelected_raw() & fTrackFilterMask)) {
2745+
if (!(a1.isBarrelSelected_raw() & a1.isBarrelSelectedPrefilter_raw() & fTrackFilterMask)) {
27482746
continue;
27492747
}
27502748
for (auto& a2 : assocs2) {
@@ -2766,7 +2764,7 @@ struct AnalysisSameEventPairing {
27662764
}
27672765

27682766
for (int iTrack = 0; iTrack < fNCutsBarrel; ++iTrack) {
2769-
if (!(a1.isBarrelSelected_raw() & (1u << iTrack))) {
2767+
if (!(a1.isBarrelSelected_raw() & a1.isBarrelSelectedPrefilter_raw() & (1u << iTrack))) {
27702768
continue;
27712769
}
27722770
for (int iMuon = 0; iMuon < fNCutsMuon; ++iMuon) {
@@ -2905,10 +2903,10 @@ struct AnalysisSameEventPairing {
29052903
}
29062904

29072905
void processElectronMuonSkimmed(MyEventsVtxCovSelected const& events,
2908-
soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts> const& barrelAssocs, MyBarrelTracksWithCovWithAmbiguities const& barrelTracks,
2906+
soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts, aod::Prefilter> const& barrelAssocs, MyBarrelTracksWithCovWithAmbiguities const& barrelTracks,
29092907
soa::Join<aod::ReducedMuonsAssoc, aod::MuonTrackCuts> const& muonAssocs, MyMuonTracksWithCovWithAmbiguities const& muons)
29102908
{
2911-
runEmuSameEventPairing<true, VarManager::kElectronMuon, gkEventFillMapWithCov, gkTrackFillMapWithCov, gkMuonFillMapWithCov>(events, trackEmuAssocsPerCollision, barrelAssocs, barrelTracks, muonAssocsPerCollision, muonAssocs, muons);
2909+
runEmuSameEventPairing<true, VarManager::kElectronMuon, gkEventFillMapWithCov, gkTrackFillMapWithCov, gkMuonFillMapWithCov>(events, trackAssocsPerCollision, barrelAssocs, barrelTracks, muonAssocsPerCollision, muonAssocs, muons);
29122910
}
29132911

29142912
void processMixingAllSkimmed(soa::Filtered<MyEventsHashSelected>& events,
@@ -2950,10 +2948,10 @@ struct AnalysisSameEventPairing {
29502948
}
29512949

29522950
void processMixingElectronMuonSkimmed(soa::Filtered<MyEventsHashSelected>& events,
2953-
soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts> const& barrelAssocs, aod::ReducedTracks const& barrelTracks,
2951+
soa::Join<aod::ReducedTracksAssoc, aod::BarrelTrackCuts, aod::Prefilter> const& barrelAssocs, aod::ReducedTracks const& barrelTracks,
29542952
soa::Join<aod::ReducedMuonsAssoc, aod::MuonTrackCuts> const& muonAssocs, MyMuonTracksWithCovWithAmbiguities const& muons)
29552953
{
2956-
runEmuSameSideMixing<gkEventFillMap>(events, trackEmuAssocsPerCollision, barrelAssocs, barrelTracks, muonAssocsPerCollision, muonAssocs, muons);
2954+
runEmuSameSideMixing<gkEventFillMap>(events, trackAssocsPerCollision, barrelAssocs, barrelTracks, muonAssocsPerCollision, muonAssocs, muons);
29572955
}
29582956

29592957
void processDummy(MyEventsBasic&)

0 commit comments

Comments
 (0)