Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

@ddobrigk ddobrigk commented Jun 13, 2025

  • modularizes bc selection, event selection and luminosity task into subclasses. Further defines named ConfigurableGroup structs for simplicity / modularity. Classes and declarations are stored in EventSelectionTools.h.
  • utilizes class modularity to create separate task structs for run 2 and run 3 with minimal code duplication and autodetect proper path using defineDataProcessing directives, effectively obsoleting processRun2 and processRun3 switches. Coupled with MC autodetect from metadata, no user selection is necessary, potentially simplifying hyperloop dependencies.
  • as a consequence of the single-device rewrite, memory consumption went from ~350-400 MB to ~120 MB, potentially improving the probability of using fewer-core queues in grid processing.

Put here as a draft PR for a first discussion in the WP4+14 meeting on Monday the 16th June.

@github-actions
Copy link

github-actions bot commented Jun 13, 2025

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

@ddobrigk
Copy link
Collaborator Author

Following the discussion in the WP4+14 meeting, this will now be merged for testing of the Run 2 / 3 autodetect at defineDataProcessing time in Hyperloop. @ekryshen has also suggested that the time stamp task could be merged with the event selection as it is essential in our preparation of events for analysis: this will be left for a future step (local code already exists in draft form).

@ddobrigk ddobrigk enabled auto-merge (squash) June 22, 2025 18:05
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk merged commit 0e37ae3 into AliceO2Group:master Jun 22, 2025
11 of 14 checks passed
fgrosa pushed a commit that referenced this pull request Jul 15, 2025
#11613)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jpxrk pushed a commit to jpxrk/O2Physics that referenced this pull request Jul 16, 2025
AliceO2Group#11613)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jul 18, 2025
AliceO2Group#11613)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
vojmach pushed a commit to vojmach/O2Physics that referenced this pull request Jul 23, 2025
AliceO2Group#11613)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
AliceO2Group#11613)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants