Skip to content

inital commit of calostatusskimmer#4182

Draft
nk7252 wants to merge 5 commits intosPHENIX-Collaboration:masterfrom
nk7252:master
Draft

inital commit of calostatusskimmer#4182
nk7252 wants to merge 5 commits intosPHENIX-Collaboration:masterfrom
nk7252:master

Conversation

@nk7252
Copy link
Contributor

@nk7252 nk7252 commented Feb 18, 2026

This module will be used to skim on the status of Towers. We are re-using the not instrumented status bit to flag empty and missing packets. The number of these status bits are counted, and if greater than a packet's worth of towers are "not instrumented", the event will be aborted.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [x ] New feature (non-breaking change which adds functionality)
  • [x ] Breaking change (fix or feature that would cause existing functionality to not work for users)
  • Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

New skimmer module

TODOs (if applicable)

  1. Commit changing calotowerbuilder to set waveform samples for missing and empty packets to -1. This is to flag these towers as not instrumented.
  2. The bit name will be changed to PacketIssue. (Breaking change)
  3. Testing

This module will be used to skim on the status of packets. Specifically we are re-using the not instrumented status bit to flag empty and missing packets.
The number of these status bits are counted, and if they pass a certain number of towers, the event will be aborted.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit ccc849875701d1dbb6a9869763296a5a3babdfcc:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit c0b2905148d13e0e26c15597d6afb6844405251d:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 9d02849111da5374214e5d150025c8aceb06cd49:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 42a15a95d59ece20f48cbfb224b359fda7d28285:
Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

@sphenix-jenkins-ci
Copy link

Build & test report

Report for commit 8a18f615325156044d66275cc3d47aa45b594d23:
Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration
sPHENIX             jenkins.io

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments