Skip to content

Add VertexGNNReco to categories map - #2839

Open
prabhatsolanki wants to merge 1 commit into
cms-sw:masterfrom
prabhatsolanki:patch-1
Open

Add VertexGNNReco to categories map#2839
prabhatsolanki wants to merge 1 commit into
cms-sw:masterfrom
prabhatsolanki:patch-1

Conversation

@prabhatsolanki

Copy link
Copy Markdown

New package DataFormats/VertexGNNReco, introduced in cms-sw/cmssw#51862 (vertex-slot GNN primary-vertex reconstruction for Phase-2). It holds the Alpaka portable SoA collections of the track features and of the GNN output, so it is added to the reconstruction and heterogeneous categories, next to DataFormats/VertexReco and DataFormats/VertexSoA.

@cmsbuild

Copy link
Copy Markdown
Contributor

A new Pull Request was created by @prabhatsolanki for branch master.

@akritkbehera, @cmsbuild, @iarspider, @raoatifshad, @smuzaffar can you please review it and eventually sign? Thanks.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@cmsbuild

cmsbuild commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

cms-bot internal usage

@smuzaffar

Copy link
Copy Markdown
Contributor

@cms-sw/heterogeneous-l2 @cms-sw/reconstruction-l2 any objections on adding this package VertexGNNReco under your responsibility ?

@Moanwar

Moanwar commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

+1

@makortel

Copy link
Copy Markdown
Contributor

The DataFormats/VertexGNNReco in cms-sw/cmssw#51862 looks rather straightforward so I don't see a particular reason for it to be assigned to heterogeneous (not all alpaka-using DataFormats packages are assigned to heterogeneous). If others in @cms-sw/heterogeneous-l2 really want it, I don't object.

@fwyzard

fwyzard commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Mostly I wonder if we need a new package at all...

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.

6 participants