cmssw icon indicating copy to clipboard operation
cmssw copied to clipboard

PPS lhcInfo split: per fill and per LS PopCon

Open JanChyczynski opened this issue 3 years ago • 3 comments

PR description:

PR validation:

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

Before submitting your pull requests, make sure you followed this checklist:

JanChyczynski avatar Oct 09 '22 16:10 JanChyczynski

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39685/32497

  • This PR adds an extra 28KB to repository

Code check has found code style and quality issues which could be resolved by applying following patch(s)

  • code-format: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39685/32497/code-format.patch e.g. curl -k https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39685/32497/code-format.patch | patch -p1 You can also run scram build code-format to apply code format directly

cmsbuild avatar Oct 09 '22 16:10 cmsbuild

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-39685/32501

  • This PR adds an extra 28KB to repository

cmsbuild avatar Oct 09 '22 23:10 cmsbuild

A new Pull Request was created by @JanChyczynski (jan_chyczynski) for master.

It involves the following packages:

  • CondTools/RunInfo (db)

@malbouis, @cmsbuild, @saumyaphor4252, @ggovi, @francescobrivio, @tvami can you please review it and eventually sign? Thanks. @mmusich this is something you requested to watch as well. @perrotta, @dpiparo, @rappoccio you are the release manager for this.

cms-bot commands are listed here

cmsbuild avatar Oct 09 '22 23:10 cmsbuild

-1

  • until an update on the PR, let's remove it from the alca/db queues

tvami avatar Dec 15 '22 02:12 tvami

@JanChyczynski @cms-sw/ctpps-dpg-l2 this draft PR is dormient since four months, and evidently unattended: can it get closed then, to clean up the queue?

perrotta avatar Feb 07 '23 10:02 perrotta

This feature is now submitted in a new PR:

https://github.com/cms-sw/cmssw/pull/40817

JanChyczynski avatar Feb 20 '23 01:02 JanChyczynski