Skip to content

Add ITS stuck-pixel CCDB object output#15468

Open
JaehyeokRyu wants to merge 6 commits into
AliceO2Group:devfrom
JaehyeokRyu:feature/its-stuck-pixels
Open

Add ITS stuck-pixel CCDB object output#15468
JaehyeokRyu wants to merge 6 commits into
AliceO2Group:devfrom
JaehyeokRyu:feature/its-stuck-pixels

Conversation

@JaehyeokRyu

Copy link
Copy Markdown

This PR adds an ITS-only StuckPixels CCDB payload to the ITSMFT dead-map builder workflow.

Main changes:

  • collect RepeatingPixel errors from the ErrorInfo input into a dedicated StuckPixelData object;
  • publish the StuckPixelData object as a new StuckPixels CCDB payload;
  • optionally write the same information as a local ROOT TTree for inspection/debugging;
  • keep the new input/output path ITS-only, leaving the MFT workflow unaffected.

The existing TimeDeadMap behavior is kept unchanged.

Comment thread Detectors/ITSMFT/common/workflow/include/ITSMFTWorkflow/DeadMapBuilderSpec.h Outdated
Comment thread DataFormats/Detectors/ITSMFT/common/CMakeLists.txt
@shahor02

shahor02 commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Thanks, could you also fix the formatting problems?

Comment thread DataFormats/Detectors/ITSMFT/common/src/ITSMFTDataFormatsLinkDef.h
f3sch
f3sch previously approved these changes Jun 5, 2026

@f3sch f3sch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

approving to start CI

@alibuild

alibuild commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 406d853 at 2026-06-05 12:37:

## sw/BUILD/O2-latest/log
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/ITSMFT/common/workflow/src/DeadMapBuilderSpec.cxx:486:22: error: no matching function for call to 'o2::itsmft::ITSMFTDeadMapBuilder::PrepareOutputCcdb(o2::framework::EndOfStreamContext*)'
ninja: build stopped: subcommand failed.

Full log here.

@f3sch

f3sch commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

@JaehyeokRyu CI build failures are genuine and would need to be fixed

@shahor02 shahor02 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving for CI

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants