add occupancy bookkeeping class and HIPO banks - #1378
Conversation
7df1707 to
bf9ad64
Compare
baltzell
changed the title
|
Negative indices definitely doesn't work: Probably this means the |
|
Nope, IndexedTables in decoding always have positive indices (they're defined only by CCDB tables), and Note, the swaps service will throw an exception for RICH's |
|
Some info about (CC @raffaelladevita) |
| table.addEntry(index); | ||
| table.setDoubleValueByHash(0.0d, 0, hash); | ||
| } | ||
| table.setDoubleValueByHash(table.getDoubleValueByHash(0, hash) + weight, 0, hash); |
There was a problem hiding this comment.
should this be *weight?
| "entries": [ | ||
| { "name":"sector" , "type":"B", "info":"sector"}, | ||
| { "name":"layer" , "type":"B", "info":"layer"}, | ||
| { "name":"component" , "type":"S", "info":"component"}, |
There was a problem hiding this comment.
do we want to keep order for FTOF, CTOF, CND, ...?
| { "name":"hits" , "type":"F", "info":"number of hits"} | ||
| ] | ||
| } | ||
| ] |
| @Override | ||
| public boolean processDataEventUser(DataEvent event) { | ||
| for (OccupanceTable t : tables) { | ||
| RawDataBank b = new RawDataBank(t.getHitBank(), 1000, OrderGroups.NOMINAL); |
There was a problem hiding this comment.
I think we may want NODENOISE instead of NOMINAL
| b.read(event); | ||
| t.fill(b, false); | ||
| } | ||
| if (++events % prescale == 0) { |
There was a problem hiding this comment.
is this thread safe?
| * @param event | ||
| * @return | ||
| */ | ||
| public DataBank create(long events, DataEvent event) { |
There was a problem hiding this comment.
in multithreading, is it possible the table gets modified by thread X while thread Y is creating the bank?
3 participants
To reduce the calibration data volume from detector hit banks, this adds new occupancy bank schemas, thread-safe
OccupanceTablebook-keeping class with I/O helpers for standard indexed HIPO banks, andOccupanceEngineready for all standard clas12 detectors.The new occupancy banks currently look like this:
FilteredBankfor order-hijacking by denoisingoccupancyPrescaleThe main consumer of these new occupancy banks is clas12-timeline.
Depends on
#1380, #1381, #1382, #1383.