Skip to content
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

Add 2024 PPS relvals able to test the PPSAlCaRecoProcuder on new data (and remove obsolete relvals) #44095

Open
perrotta opened this issue Feb 27, 2024 · 5 comments

Comments

@perrotta
Copy link
Contributor

This issue is meant to remind that, after the merge of #44044 (and its 14_0_X backport), the PPS relvals 1043,1044,1045, which run PPS AlCaReco producer on a 2022 dataset, would be getting complains about missing product (Cannot unpack: no FEDRawDataCollection found) as the product now expected by the gtStage2Digis in this PR is missing in those 2022 data. That error message had been silenced in the log, but it makes no sense to have in the matrix relvals that cannot upack the data.

As soon as we will have new data this year one should add similar relvals which test PPSAlCaRecoProcuder on the 2024 data, and remove those obsolete relvals 1043-45 as they won’t make any sense with the code currently in CMSSW_14_1_X and CMSSW_!4_0_X.

This must be done as soon as PPS will confirm that a suitable data file with the updated products in it becomes available.

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 27, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @perrotta.

@Dr15Jones, @smuzaffar, @makortel, @sextonkennedy, @rappoccio, @antoniovilela can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@perrotta
Copy link
Contributor Author

assign alca, pdmv

@cmsbuild
Copy link
Contributor

New categories assigned: alca,pdmv

@AdrianoDee,@sunilUIET,@miquork,@saumyaphor4252,@perrotta,@consuegs you have been requested to review this Pull request/Issue and eventually sign? Thanks

@perrotta
Copy link
Contributor Author

type ctpps

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants