Skip to content

Add filtering from fragment#47333

Closed
vlimant wants to merge 3 commits intocms-sw:masterfrom
vlimant:filter_in_fragment_150X
Closed

Add filtering from fragment#47333
vlimant wants to merge 3 commits intocms-sw:masterfrom
vlimant:filter_in_fragment_150X

Conversation

@vlimant
Copy link
Copy Markdown
Contributor

@vlimant vlimant commented Feb 12, 2025

PR description:

looks like when a user adds a ProductionFilterSequence in the fragment, it is not systematically added in the final configuration for filtering. This PR remedies that.

PR validation:

should be rather transparent. tested using a request configuration.

cmsbuild and others added 3 commits February 10, 2025 10:05
…-overflow

Fix event number overflow in ES packer - 150X
[15_0_X] Reduce minYsizeB1 and minYsizeB2 CA cuts for Phase1
@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-47333/43666

@cmsbuild
Copy link
Copy Markdown
Contributor

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

It involves the following packages:

  • Configuration/Applications (operations)
  • Geometry/CommonTopologies (geometry)
  • RecoTracker/PixelSeeding (reconstruction)

@Dr15Jones, @antoniovilela, @bsunanda, @civanch, @cmsbuild, @davidlange6, @fabiocos, @jfernan2, @kpedro88, @makortel, @mandrenguyen, @mdhildreth, @rappoccio can you please review it and eventually sign? Thanks.
@GiacomoSguazzoni, @Martin-Grunewald, @VinInn, @VourMa, @bsunanda, @dgulhan, @fabiocos, @felicepantaleo, @gpetruc, @makortel, @martinamalberti, @missirol, @mmusich, @mtosi, @rovere this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@vlimant vlimant closed this Feb 12, 2025
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