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

Re-enable ctppsProtons from recoCTPPSTask (Run-3 only) #33265

Closed
wants to merge 1 commit into from

Conversation

silviodonato
Copy link
Contributor

@silviodonato silviodonato commented Mar 25, 2021

After #33090, we are finally testing GluGluTo2Jets_M_300_2000_14TeV_Exhume (11725.0,11925.0) in the IB tests (#32969). This workflow was added specifically to test the PPS reco (#32765).

Brief summary of the history:

Reverts #32352

@mundim @fabferro @jan-kaspar @cms-sw/alca-l2 @cms-sw/reconstruction-l2

PS. The last open pre-release of 11_3_X is scheduled on April 13.

@silviodonato
Copy link
Contributor Author

test parameters:

  • workflow = 11725.0,11925.0

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33265/21756

  • This PR adds an extra 8KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @silviodonato (Silvio Donato) for master.

It involves the following packages:

RecoPPS/Configuration

@perrotta, @jpata, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@jan-kaspar this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@silviodonato
Copy link
Contributor Author

moved to #33266

@silviodonato silviodonato deleted the revert-32352-removeCtppsProtons branch April 15, 2021 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants