Skip to content

[PWGLF] Add event loss, event splitting, and cut-flow QA - #17317

Closed
bhattdurgesh921-sys wants to merge 0 commit into
AliceO2Group:masterfrom
bhattdurgesh921-sys:master
Closed

[PWGLF] Add event loss, event splitting, and cut-flow QA#17317
bhattdurgesh921-sys wants to merge 0 commit into
AliceO2Group:masterfrom
bhattdurgesh921-sys:master

Conversation

@bhattdurgesh921-sys

Copy link
Copy Markdown
Contributor

Dear @dmallick2 ,
In this PR we have implemented the event-level correction framework for the Δ analysis by adding the Event Loss and Event Splitting calculations, corresponding QA and cut-flow histograms, and the truth-level INEL > 0 selection.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Add event loss, event splitting, and cut-flow QA [PWGLF] Add event loss, event splitting, and cut-flow QA Aug 4, 2026
@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for ee95278 at 2026-08-05 15:03:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 4 errors and 0 warnings.'
Found 4 errors and 0 warnings.
++ [[ 4 -gt 0 ]]
++ cat /sw/BUILD/8c7e36b716d4d4731c67633408a921091b300dcc/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/deltaAnalysis.cxx:80:8: error: constructor does not initialize these fields: pdgDB [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Tasks/Resonances/deltaAnalysis.cxx:86:3: error: no header providing "o2::framework::Service" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/deltaAnalysis.cxx:458:38: error: no header providing "TH1" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/deltaAnalysis.cxx:1655:59: error: use 'contains' to check for membership [readability-container-contains,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 4 -gt 0 ]]
++ exit 1
--

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for ee95278 at 2026-08-05 08:37:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for ee95278 at 2026-08-05 09:47:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@bhattdurgesh921-sys

Copy link
Copy Markdown
Contributor Author

Dear @vkucera ,

It appears that the build for this PR is failing due to an issue related to the changes in Detectors/Base/CMakeLists.txt. The build reports the following error:

CMake Error in Detectors/Base/CMakeLists.txt:
Imported target "TGeo2VecGeom::TGeo2VecGeom" includes non-existent path

"/local/workspace/DailyBuilds/DailyO2Physics-slc9/daily-tags.qD5m7JNmUl/slc9_x86-64/ROOT/v6-36-10-alice2-2/include"

in its INTERFACE_INCLUDE_DIRECTORIES. Possible reasons include:

  • The path was deleted, renamed, or moved to another location.
  • An install or uninstall procedure did not complete successfully.

Could you please have a look at this issue when you have a chance? I would appreciate your help in identifying the cause.

Thank you!

@vkucera

vkucera commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

@bhattdurgesh921-sys The issue is being addressed. Please see the support channel.

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.

3 participants