Skip to content

[Common] [WIP] merge centrality and multiplicity tasks - #10783

Closed
njacazio wants to merge 2 commits into
masterfrom
nj-mergedcent
Closed

[Common] [WIP] merge centrality and multiplicity tasks#10783
njacazio wants to merge 2 commits into
masterfrom
nj-mergedcent

Conversation

@njacazio

@njacazionjacazio commented Apr 7, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actionsgithub-actionsBot changed the title [WIP] merge centrality and multiplicity tasks[Common] [WIP] merge centrality and multiplicity tasksApr 7, 2025
@alibuildalibuild mentioned this pull request Apr 7, 2025
Merged
njacazio pushed a commit that referenced this pull request Apr 10, 2025
@alibuild
Please consider the following formatting changes to#10783(#10784)
7108359
@alibuildalibuild mentioned this pull request Apr 10, 2025
Merged
njacazio pushed a commit that referenced this pull request Apr 10, 2025
@alibuild
Please consider the following formatting changes to#10783(#10806)
95fecc1
@njacazio
njacazio marked this pull request as ready for review April 10, 2025 10:19
@alibuildalibuild mentioned this pull request Apr 10, 2025
Merged
njacazio pushed a commit that referenced this pull request Apr 10, 2025
@alibuild
Please consider the following formatting changes to#10783(#10809)
341f0f4
@alibuildalibuild mentioned this pull request Apr 10, 2025
Closed
@alibuild

alibuild commented Apr 10, 2025

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for e9e0d81 at 2025-04-10 13:08:

## sw/BUILD/O2Physics-latest/log
CMake Error at cmake/O2PhysicsAddExecutable.cmake:113 (add_executable):
CMake Error at cmake/O2PhysicsAddExecutable.cmake:113 (add_executable):

Full log here.

njacazio pushed a commit that referenced this pull request May 3, 2025
@alibuild@njacazio
Please consider the following formatting changes to#10783(#10784)
b503a6a
njacazio pushed a commit that referenced this pull request May 3, 2025
@alibuild@njacazio
Please consider the following formatting changes to#10783(#10806)
8801a29
njacazio pushed a commit that referenced this pull request May 3, 2025
@alibuild@njacazio
Please consider the following formatting changes to#10783(#10809)
12122cf
@github-actions

github-actionsBot commented May 3, 2025•
edited
Loading

Copy link
Copy Markdown

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

@alibuild

alibuild commented May 3, 2025•
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 5d4f65d at 2025-05-10 09:17:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10783-slc9_x86-64/0/Common/TableProducer/multiplicityTable.cxx:316:57: error: unused parameter 'bcs' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented May 13, 2025•
edited
Loading

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for ee05f74 at 2025-05-14 21:32:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10783-slc9_x86-64/0/Common/TableProducer/multiplicityTable.cxx:316:57: error: unused parameter 'bcs' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

njacazioand others added 2 commits May 16, 2025 15:56
@njacazio
Update match-mft-mch-data.cxx
4e200c4
Clean CMakeFile
Clean MC task
Clean CMAKEfile
Please consider the following formatting changes to #11066 (#11067)
Please consider the following formatting changes to #11066 (#11068)
Update CMakeLists.txt
Prototype of merged centrality
Update
Update
update
update
Update
Update
Please consider the following formatting changes to #10783 (#10784)
Update
Please consider the following formatting changes to #10783 (#10806)
Update
Update
Please consider the following formatting changes to #10783 (#10809)
Update
Update
Update
UniformCENT
Update
Update
@njacazio
Update
62dcf52
@njacazionjacazio closed this Jun 7, 2025
@njacazio
njacazio deleted the nj-mergedcent branch June 7, 2025 12:41
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuildalibuildAwaiting requested review from alibuildalibuild is a code owner
@jgrosseojgrosseoAwaiting requested review from jgrosseojgrosseo is a code owner
@iarseneiarseneAwaiting requested review from iarseneiarsene is a code owner
@ktfktfAwaiting requested review from ktfktf is a code owner
@ddobrigkddobrigkAwaiting requested review from ddobrigkddobrigk is a code owner

Assignees

No one assigned

Labels

common

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@njacazio@alibuild