Skip to content

Fix CC/NC Extended flags in EBSaver - #399

Merged
marc1uk merged 1 commit into
ANNIEsoft:Applicationfrom
anuj-guptta:extended
Oct 21, 2026
Merged

marc1uk merged 1 commit into
ANNIEsoft:Applicationfrom
anuj-guptta:extended

Conversation

@anuj-guptta

Copy link
Copy Markdown
Contributor

Describe your changes

Updated the flags to represent the correct Extended window.

Triggerword = 40: BE_status.minbias_start --> Non-CC extended readout
Triggerword = 41: BE_status.extend_start --> CC-extended readout

Reference: Michael's thesis

  • This change does not affect anything downstream as the CTCWord is unchanged; the flags were not used downstream.

Checklist before submitting your PR

  • This PR implements a single change (one new/modified Tool, or a set of changes to implement one new/modified feature)
  • This PR alters the minimum number of files to affect this change
  • If this PR includes a new Tool, a README and minimal demonstration ToolChain is provided
  • If a new Tool/ToolChain requires model or configuration files, their paths are not hard-coded, and means of generating those files is described in the readme, with examples provided on /pnfs/annie/persistent
  • For every new usage, there is a reason the data must be on the heap
  • For every new there is a delete, unless I explicitly know why (e.g. ROOT or a BoostStore takes ownership)

Additional Material

Attach any validation or demonstration files here. You may also link to relavant docdb articles.

@S81D

S81D commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

This is the correct implementation, thanks for fixing this!

CCExtended = true;
ExtendedTriggerTime = trigger.first;
CTCWordExtended = 1; <--- as you said, this is what we actually use in the downstream analysis, which is left unchanged. The adjustment applies the correct mapping

@marc1uk
marc1uk merged commit 4673831 into ANNIEsoft:Application Oct 21, 2026
1 check passed
@anuj-guptta
anuj-guptta deleted the extended branch October 23, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants