New nano format

This merge request is to update the nano flat table to new format as described in https://docs.google.com/document/d/1Rg1dlxHXhJ8i-kp6_WYKC9L8VOxT4OwPuIbR3NsO0sc/edit#bookmark=id.8f6f86icjw77

• information from digi and rechit collection
• information from logical mapping: waiting for update in hackathon branch (https://github.com/CMS-HGCAL/cmssw/tree/dev/hackathon_base_CMSSW_14_1_X)
• separate into two producers

@psilva, please let me know when the hackathon branch is updated. @jinglu, please have a look, and take over the code once extracting information from the logical mapping part is done.

marked the checklist item information from digi and rechit collection as completed

marked this merge request as draft

Thanks a lot just two comments:

• maybe we call the table producer something more generic?
• can we add a flag to switch of digi table in case we just want to look at rechits ?

added 1 commit

• 5da5b58c - removing test and giving final name

Compare with previous version

@yumiao I have updated the table producer with the indexer I took the liberty of renaming to HGCalNanoTableProducer and remove the 2023/2024 references. It should run with https://github.com/CMS-HGCAL/cmssw/pull/121 (tested on 200 events)

Maybe two comments:

• can we add a flag to switch of digi table in case we just want to look at rechits ? (same as above)
• how are we dealing with ZS events: does the unpacker set a flag on the suppressed digis? if not we should so that we propagate to the rec hits and then we suppress both in the nano table.

Tagging @hqu as well

marked the checklist item information from logical mapping: waiting for update in hackathon branch (https://github.com/CMS-HGCAL/cmssw/tree/dev/hackathon_base_CMSSW_14_1_X) as completed

@ykao please take a look also at L98-107 I think this was the missing info for DQM as well.

added 1 commit

• c7cad8e0 - added skipDigi flag for flattable (default: false)

Compare with previous version

marked the checklist item separate into two producers as completed

added 1 commit

• e3cd5e92 - added the isSiPM to the DIGIs table

Compare with previous version

marked this merge request as ready

mentioned in merge request !19 (merged)

resolved all threads

mentioned in commit 9b986dce

merged