Skip to content

[PWGLF] Update on the resonanceModuleInitializer - #17961

Merged
dmallick2 merged 3 commits into
AliceO2Group:masterfrom
mjkim525:codex/module-microtrack-quality
Sep 19, 2026
Merged

[PWGLF] Update on the resonanceModuleInitializer#17961
dmallick2 merged 3 commits into
AliceO2Group:masterfrom
mjkim525:codex/module-microtrack-quality

Conversation

@mjkim525

@mjkim525 mjkim525 commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

This PR extends Micro001 track-quality information to support a broader range of systematic studies and simplifies track processing in resonanceModuleInitializer.

  • Add TPC crossed rows and the ITS cluster map to ResoMicroTracks_001.
  • Provide dynamic getters: itsNCls(), itsNClsInnerBarrel(), and hasITSHitInLayer().
  • Remove processDataHybrid, retaining processData with explicit collision-based slicing.
  • Add optional pT-dependent ITS/TPC QA through cfgQAITS and cfgQATPC, both disabled by default.

@mjkim525
PWGLF: extend Micro001 quality and remove hybrid track processing
8b1d6cf 
Store TPC crossed rows and ITS cluster map in Micro001 and expose ITS quality getters. Keep per-collision processData slicing and remove the redundant hybrid callback and its configuration checks.

Micro001 is intentionally extended in place; older version-1 files require regeneration or conversion. Validated with a target build and identical-input Data/MC smoke tests.
@mjkim525
PWGLF: add configurable Micro001 ITS and TPC QA
5a1d987
@github-actions github-actions Bot changed the title Update on the resonanceModuleInitializer [PWGLF] Update on the resonanceModuleInitializer Sep 18, 2026
@github-actions

github-actions Bot commented Sep 18, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 32 warnings, 🔕 2 disabled

@mjkim525
mjkim525 marked this pull request as ready for review September 18, 2026 08:17
@alibuild

alibuild commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 5a1d987 at 2026-09-18 23:13:

## sw/BUILD/lhapdf-latest/log
collect2: error: ld returned 1 exit status


## sw/BUILD/re2-latest/log
CMake Error at CMakeLists.txt:84 (find_package):

Full log here.

@alibuild

alibuild commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 5a1d987 at 2026-09-18 22:54:

## sw/BUILD/lhapdf-latest/log
collect2: error: ld returned 1 exit status


## sw/BUILD/re2-latest/log
CMake Error at CMakeLists.txt:84 (find_package):

Full log here.

@mjkim525
mjkim525 marked this pull request as draft September 19, 2026 08:50
@mjkim525
mjkim525 marked this pull request as ready for review September 19, 2026 09:11
@dmallick2
dmallick2 merged commit 127f968 into AliceO2Group:master Sep 19, 2026
14 checks passed
@mjkim525
mjkim525 deleted the codex/module-microtrack-quality branch September 19, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmallick2 dmallick2 dmallick2 approved these changes
@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner
@omvazque omvazque Awaiting requested review from omvazque omvazque is a code owner
@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner
@mpuccio mpuccio Awaiting requested review from mpuccio mpuccio is a code owner
@smaff92 smaff92 Awaiting requested review from smaff92 smaff92 is a code owner
@prottayCMT prottayCMT Awaiting requested review from prottayCMT prottayCMT is a code owner
@gbencedi gbencedi Awaiting requested review from gbencedi gbencedi is a code owner
@abmodak abmodak Awaiting requested review from abmodak abmodak is a code owner
@mario-krueger mario-krueger Awaiting requested review from mario-krueger mario-krueger is a code owner
@maciacco maciacco Awaiting requested review from maciacco maciacco is a code owner
@HorstMa HorstMa Awaiting requested review from HorstMa HorstMa is a code owner
@ercolessi ercolessi Awaiting requested review from ercolessi ercolessi is a code owner
@romainschotter romainschotter Awaiting requested review from romainschotter romainschotter is a code owner
@lhusova lhusova Awaiting requested review from lhusova lhusova is a code owner

Assignees

No one assigned

Labels

datamodel pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mjkim525 @alibuild @dmallick2