Skip to content

[PWGCF] Fix MC processing, remove tmp histograms from registry#16106

Merged
wenyaCern merged 2 commits intoAliceO2Group:masterfrom
EmilGorm:mcfix
May 7, 2026
Merged

[PWGCF] Fix MC processing, remove tmp histograms from registry#16106
wenyaCern merged 2 commits intoAliceO2Group:masterfrom
EmilGorm:mcfix

Conversation

@EmilGorm
Copy link
Copy Markdown
Contributor

@EmilGorm EmilGorm commented May 4, 2026

  • Fix to MC reco and MC gen processing for radial and elliptic flow-spectra correlations
  • Temporary histograms for tracking EbyE spectra have been removed from the histogram registry to remove clutter in the output
  • Some changes to enum names to conform to coding guidellines
  • Proper use of subevent-averaged mean pt for v0

@github-actions github-actions Bot added the pwgcf label May 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

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

@github-actions github-actions Bot changed the title Fix MC processing, remove tmp histograms from registry [PWGCF] Fix MC processing, remove tmp histograms from registry May 4, 2026
@EmilGorm EmilGorm marked this pull request as ready for review May 4, 2026 19:54
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 4, 2026

Why don't you fix the magic numbers and names?

Comment thread PWGCF/GenericFramework/Tasks/flowGenericFramework.cxx Outdated
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented May 6, 2026

Error while checking build/O2Physics/staging for 05b98e2 at 2026-05-07 10:17:

## sw/BUILD/O2Physics-latest/log
CMake Error at /sw/slc9_x86-64/CMake/v4.1.4-2/share/cmake-4.1/Modules/CMakeTestCCompiler.cmake:67 (message):
    2026-05-07T10:17:50.992+0200 [70:140066366488192] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T10:17:50.992+0200 [70:140066366488192] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    2026-05-07T10:17:50.993+0200 [70:140066366488192] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T10:17:50.993+0200 [70:140066366488192] [executioncontext.cpp:908] [ERROR] Error while uploading resources to CAS at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    ninja: build stopped: subcommand failed.

Full log here.

@wenyaCern wenyaCern merged commit 6b2f79c into AliceO2Group:master May 7, 2026
13 of 15 checks passed
@EmilGorm EmilGorm deleted the mcfix branch May 7, 2026 12:26
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.

4 participants