Skip to content

Conversation

@BongHwi
Copy link
Collaborator

@BongHwi BongHwi commented Dec 30, 2025

Summary

This PR introduces a new CollisionCutsGroup class that follows the v0SelectionGroup pattern to enable automatic configurable registration of collision event selection parameters in O2 analysis tasks.

Key Changes

  • New utility class: CollisionCutsGroup that auto-registers all collision selection parameters via ROOT serialization
  • Automatic event selection import: Includes all event selection flags from EventSelectionParams.h using macro-based generation
  • Reduced boilerplate: Eliminates ~25 lines of manual configurable declarations per task
  • Comprehensive documentation: Added tutorial task and README with usage examples

Files Added:

  • PWGLF/Utils/collisionCutsGroup.h - Main class definition
  • PWGLF/Utils/collisionCutsGroup.cxx - Implementation with ROOT dictionary
  • PWGLF/Utils/collisionCutsGroupLinkDef.h - ROOT dictionary linkage
  • PWGLF/Utils/EventSelectionFlagsMapping.def - Event selection flag mappings
  • Tutorials/PWGLF/Utils/collisionCuts.cxx - Tutorial task demonstrating usage
  • Tutorials/PWGLF/Utils/README.md - Comprehensive documentation

Files Modified:

  • PWGLF/Utils/collisionCuts.h - Refactored to support the new group pattern
  • PWGLF/Utils/CMakeLists.txt - Added library build for collisionCutsGroup
  • Tutorials/PWGLF/CMakeLists.txt - Registered new Utils tutorial subdirectory

@github-actions
Copy link

github-actions bot commented Dec 30, 2025

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

@BongHwi BongHwi changed the title [PWGLF] Add CollisionCutsGroup utility class with automatic configurable registration [PWGLF,Tutorial] Add CollisionCutsGroup utility class with automatic configurable registration Dec 30, 2025
Please consider the following formatting changes to AliceO2Group#14362
BongHwi and others added 2 commits December 30, 2025 12:12
@BongHwi BongHwi enabled auto-merge (squash) December 30, 2025 10:49
@BongHwi
Copy link
Collaborator Author

BongHwi commented Dec 30, 2025

Ready to merge now. Please review it when you're free.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants