Skip to content

add occupancy bookkeeping class and HIPO banks - #1378

Open
baltzell wants to merge 47 commits into
developmentfrom
occupancy
Open

add occupancy bookkeeping class and HIPO banks#1378
baltzell wants to merge 47 commits into
developmentfrom
occupancy

Conversation

@baltzell

@baltzell baltzell commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

To reduce data volume, this adds an occupancy bookkeeping class and engine, with I/O helpers for indexed banks ready for all standard clas12 detectors. The new occupancy banks currently look like this:

        "name": "OCC::FTOF::adc",
        "group": 500,
        "item": 14,
        "info": "Occupancy Bank",
        "entries": [
            { "name":"sector"    , "type":"B", "info":"sector"},
            { "name":"layer"     , "type":"B", "info":"layer"},
            { "name":"component" , "type":"S", "info":"component"},
            { "name":"hits"      , "type":"F", "info":"number of hits"}
  • Choose occupancy bank naming.
  • Need to use FilteredBank for order-hijacking by denoising
  • Check the #-index assumption for all detectors
  • Missing detectors
  • Note, this ignore negative indices, e.g. BMT/BST
  • YAML variable: "prescale"?

The main consumer of these new occupancy banks is clas12-timeline.

Depends on #1380, #1381, #1382, #1383.

@baltzell
baltzell force-pushed the occupancy branch 4 times, most recently from 7df1707 to bf9ad64 Compare August 12, 2026 21:00
@baltzell baltzell changed the title add occupancy class add occupancy bookkeeping class and HIPO banks Aug 13, 2026
@baltzell
baltzell marked this pull request as ready for review August 13, 2026 22:21
@baltzell
baltzell marked this pull request as draft August 14, 2026 21:28
@baltzell
baltzell marked this pull request as ready for review August 18, 2026 18:00
Comment thread etc/bankdefs/hipo4/occupancy.json
Comment on lines +195 to +214
new OccupanceTable(schema,"DC::tot"),
new OccupanceTable(schema,"DC::tdc"),
new OccupanceTable(schema,"ECAL::adc"),
new OccupanceTable(schema,"ECAL::tdc"),
new OccupanceTable(schema,"FTOF::adc"),
new OccupanceTable(schema,"FTOF::tdc"),
new OccupanceTable(schema,"CTOF::adc"),
new OccupanceTable(schema,"CTOF::tdc"),
new OccupanceTable(schema,"HTCC::adc"),
new OccupanceTable(schema,"HTCC::tdc"),
new OccupanceTable(schema,"LTCC::adc"),
new OccupanceTable(schema,"LTCC::tdc"),
new OccupanceTable(schema,"BST::adc"),
new OccupanceTable(schema,"BMT::adc"),
new OccupanceTable(schema,"FTCAL::adc"),
new OccupanceTable(schema,"FTHODO::adc"),
new OccupanceTable(schema,"FTTRK::adc"),
new OccupanceTable(schema,"RICH::tdc"),
new OccupanceTable(schema,"BAND::adc"),
new OccupanceTable(schema,"BAND::tdc"),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there any way to avoid the DRY violation here, and read occupancy.json somehow?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

That could surely be done.

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.

3 participants