CMS-HGCAL / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
3 stars 2 forks source link

Refactor unpacker #95

Open hqucms opened 8 months ago

hqucms commented 8 months ago

Todos:

yulunmiao commented 8 months ago

Here are more detailed todos in "update unpacker and test units"

yulunmiao commented 8 months ago

@pfs @hqucms I checked the code and find that the subpackage EventFilter/HGCalRawToDigi in the dev branch is out of date, the most up-to-date version is in branch https://github.com/CMS-HGCAL/cmssw/tree/hgcal-condformat-HGCalNANO-13_2_0_pre3. Huiling suggested to copy by hand. Should I proceed with this and push to this branch so we have the correct starting point?

pfs commented 8 months ago

Hi @yulunmiao indeed we need to copy by hand just the bare files which we want to make part of the new pull request. We have to coordinate among us all for the following:

yulunmiao commented 8 months ago

@pfs Judged by checking the header files used in HGCalRawToDigi.cc We should also include things in DataFormats(mostly SOA related stuff) and CondFormats(module information).

pfs commented 8 months ago

ah right so we need the SoA for the digis but nothing in condformats, that we'll use what's in https://github.com/cms-sw/cmssw/pull/43751

yulunmiao commented 8 months ago

Great, do we also want to keep the digi to reco step, so we can do the comparison to the beam re-reco. In this case, we might also need to add Yuwei or Jeremi here to move the necessary subpackages to this branch. Otherwise, the only thing we can compare is the digis unpacked by the unpacker.