cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.07k stars 4.28k forks source link

[Phase-2] Validation of Phase-2 HLT paths and high level objects #44761

Open SohamBhattacharya opened 5 months ago

SohamBhattacharya commented 5 months ago

Currently there is no proper DQM for the Phase-2 HLT paths and high level objects (objects that are used to build the paths -- leptons, jets, etc.). A tool (developed by @Sam-Harper and @finnlabe) already exists in CMSSW to do so. This has been used privately by some POGs, but not used centrally, afaik. However, the tool should be hooked to jenkins so that it can be triggered for Phase-2 HLT workflows. That will help us monitor the impact of any PR can potentially impact Phase-2 HLT.

The first step would be to have a config for the aforementioned tool with the Phase-2 HLT paths, for which I will be opening a PR very soon. I am raising this issue to discuss how to best proceed after that.

Tagging @rovere

cmsbuild commented 5 months ago

cms-bot internal usage

cmsbuild commented 5 months ago

A new Issue was created by @SohamBhattacharya.

@Dr15Jones, @makortel, @smuzaffar, @sextonkennedy, @antoniovilela, @rappoccio can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

mmusich commented 5 months ago

assign hlt, dqm, upgrade

cmsbuild commented 5 months ago

New categories assigned: hlt,dqm

@Martin-Grunewald,@mmusich,@rvenditti,@syuvivida,@tjavaid,@nothingface0,@antoniovagnerini you have been requested to review this Pull request/Issue and eventually sign? Thanks

cmsbuild commented 5 months ago

New categories assigned: upgrade

@srimanob,@subirsarkar you have been requested to review this Pull request/Issue and eventually sign? Thanks

SohamBhattacharya commented 5 months ago

With #44776 almost ready, we should move towards adding this to regular release validation configurations (as @mmusich also commented). I can take a look at doing so, but I'll need some pointers as to which configurations need to be changed.

N.B. @Sam-Harper is working on a few fixes to the tool -- eta for that PR is end of the week.