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

Remove lumi PCC stream for allForExpress and make a new key for Run3 #37711

Closed
wants to merge 1 commit into from

Conversation

tvami
Copy link
Contributor

@tvami tvami commented Apr 27, 2022

PR description:

ALCALumiPixelsCountsExpress is only present in Run-3 data, but @allForExpress is also used on Run-2 (and Run-1 data). Like this the relvals running on Run-2 data will be fine.

PR validation:

runTheMatrix.py -l 1001.2

if this PR is a backport please specify the original PR and why you need to backport that PR:

The content will be backport to 12_3_X

resolves #37685

@tvami
Copy link
Contributor Author

tvami commented Apr 27, 2022

test parameters:

  • workflows = 1001.2

@mmusich
Copy link
Contributor

mmusich commented Apr 27, 2022

@tvami I think #37712 offers a solution which doesn't require splitting by era.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37711/29543

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @tvami (Tamas Vami) for master.

It involves the following packages:

  • Configuration/AlCa (alca)

@cmsbuild, @malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @missirol, @mmusich, @fabiocos, @tocheng this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@tvami
Copy link
Contributor Author

tvami commented Apr 27, 2022

Closing in favor of #37712

@tvami tvami closed this Apr 27, 2022
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.

ProductNotFound exception in AlcaPCCIntegrator in workflow 1001.2
3 participants