Skip to content

L1S(Nano): add subpackage L1TriggerScouting/NanoAOD#50941

Open
missirol wants to merge 1 commit into
cms-sw:masterfrom
missirol:devel_cmssw50636
Open

L1S(Nano): add subpackage L1TriggerScouting/NanoAOD#50941
missirol wants to merge 1 commit into
cms-sw:masterfrom
missirol:devel_cmssw50636

Conversation

@missirol
Copy link
Copy Markdown
Contributor

PR description:

This PR addresses #50636, moving (most of) the NanoAOD source code specific to L1-Scouting to a new subpackage named L1TriggerScouting/NanoAOD (assigned to L1T and XPOG in cms-sw/cms-bot#2752).

Minor cleanup of #include statements in some of the files involved was also done in the process.

Merely technical, no changes expected.

PR validation:

The nano workflows passed, i.e.

runTheMatrix.py -w nano -l all --ibeos

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:

No backports needed.

Move most of the NanoAOD source code specific to L1-Scouting
to a dedicated subpackage named L1TriggerScouting/NanoAOD.
@cmsbuild
Copy link
Copy Markdown
Contributor

cms-bot internal usage

@cmsbuild
Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50941/49346

@cmsbuild
Copy link
Copy Markdown
Contributor

A new Pull Request was created by @missirol for master.

It involves the following packages:

  • Configuration/EventContent (operations)
  • L1TriggerScouting/NanoAOD (****)
  • L1TriggerScouting/Utilities (daq)
  • PhysicsTools/NanoAOD (xpog)

The following packages do not have a category, yet:

L1TriggerScouting/NanoAOD
Please create a PR for https://github.com/cms-sw/cms-bot/blob/master/categories_map.py to assign category

@battibass, @cmsbuild, @davidlange6, @emeschi, @fabiocos, @ftenchini, @ftorrresd, @mandrenguyen, @smorovic can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @Mmiglio, @fabiocos, @gpetruc, @mmusich this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

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.

2 participants