New digitized collections cannot be included in the event content

[skkwan]

Artur opened a PR to include the new collections in the event content: https://github.com/cms-sw/cmssw/pull/43872 However, the tests fail since the digitized objects don’t have the proper dictionary for the ap_uint<64>. I assume you could persist one (or more) built-in uint type and convert to ap_uint when needed. Could you have a look and propose a fix? Saving the products to disk is crucial for downstream consumers.

unsigned long long int is 64 bits.

To-do

• [x] How to reproduce this error locally?
  • use: process.schedule = cms.Schedule(process.pL1EG, process.end) in this file: https://github.com/skkwan/phase2-l1Calo-analyzer/blob/on-phase2-l1t-1330pre3_v13/test/test-singleAnalyzer.py
  • the event content needs to be such that this step should succeed.
  • use keep * in the pool output module thing
  • Done in commit 0334501e61f38f9cddfe
• [x] Do I only need to change the (private) data member of DigitizedClusterCorrelator to an unsigned long long int (64 bits) and have the getter methods still return ap_uint

[skkwan]

Switched to unsigned long int [1] and in a local test with a keep command [2], the digitizing error is gone [1] https://github.com/skkwan/cmssw/commit/1b4ca950fc69aa358b95a2dd57e4e9148c46b4b2 [2] https://github.com/skkwan/phase2-l1Calo-analyzer/blob/fix-digitized-collections/test/test-singleAnalyzer.py#L67-L82

Marking issue as closed.