-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DataFormat and unpacker changes for HF QIE10 (part A) #13331
Conversation
A new Pull Request was created by @kpedro88 (Kevin Pedro) for CMSSW_8_0_X. It involves the following packages: DataFormats/HcalDigi @civanch, @cvuosalo, @mdhildreth, @cmsbuild, @slava77, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Please test From: Kevin Pedro [notifications@github.com] @davidlange6https://github.com/davidlange6 - I picked out the DataFormats and unpacker changes from #13232#13232. I'm still compiling privately to make sure I didn't forget any dependencies, but I think it should work. For the moment I'll leave #13232#13232 open while I prepare part B. You can view, comment on, or merge this pull request online at: Commit Summary
File Changes
Patch Links:
� |
b9926f7
to
956475a
Compare
Just noticed a dependency that I missed, updated the PR |
Pull request #13331 was updated. @civanch, @cvuosalo, @mdhildreth, @cmsbuild, @slava77, @davidlange6 can you please check and sign again. |
please test From: Kevin Pedro [notifications@github.com] Just noticed a dependency that I missed, updated the PR � |
The tests are being triggered in jenkins. |
will test in parallel with IB..sorry for taking a shortcut.. |
DataFormat and unpacker changes for HF QIE10 (part A)
@davidlange6 - I'm privately compiling partB now (with partA merged manually) to make sure it works. I assume this will get into the 2300 IB, in which case I'll submit the partB PR once that IB is available. |
@davidlange6 - I picked out the DataFormats and unpacker changes from #13232. I'm still compiling privately to make sure I didn't forget any dependencies, but I think it should work. For the moment I'll leave #13232 open while I prepare part B.