Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding in slimmed superclusters for EGM efficiency measurements to Mini #42007

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sam-Harper
Copy link
Contributor

@Sam-Harper Sam-Harper commented Jun 19, 2023

PR description:

There is a current push for getting rid of AOD and expand miniAOD with the missing features for needed workflows. One such workflow is the EG reco efficiency measurement which measures the supercluster -> electron efficiency

To do this we need an unbiased collection of superclusters which is only in the AOD as the mini collection is only for SC which are photons or electrons and have some selection on them. We also need to apply some sort of ID on those objects, currently this is track isolation. The track isolation can be calculated on the fly from candidates in the miniAOD but its easier to have it precomputed.

For the EGM efficiency measurement we just need to know the energy/eta/phi of the SC and its track isolation value. It would also be useful to have the rawEnergy and the seed det ids of its clusters (this is absolutely necessary for the seed cluster, useful for the others). Thus we dont need the "full fat" supercluster so a reduced format has been created which will singificantly reduce the size stored

This is a SlimmedSuperCluster, the name can be changed but this is the general idea. Also I'm debating having a vector of floats for the id variables rather than a hardcoded trkIso (ala pat::UserFloats and similar) which if its affordable would improve flexablity.

This is a work in progress and at this stage is here so EGamma experts can comment on it while the rest of the tests are done.

This superseeds #41745 (and in takes much of the inspiration on the track isolation calculator from there)

PR validation:

PR is ongoing. For a RelVal TTBar PU 13_2_0_pre1 sample reminied ,

recoSlimmedSuperClusters_reducedEgamma_slimmedSuperClusters_PAT. 504.762 358.507

further size tests are on going.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42007/35969

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 19, 2023

A new Pull Request was created by @Sam-Harper (Sam Harper) for master.

It involves the following packages:

  • DataFormats/EgammaReco (reconstruction)
  • RecoEgamma/EgammaIsolationAlgos (reconstruction)
  • RecoEgamma/EgammaPhotonProducers (reconstruction)

@cmsbuild, @mandrenguyen, @clacaputo can you please review it and eventually sign? Thanks.
@a-kapoor, @jainshilpi, @sameasy, @valsdav, @rovere, @lgray, @missirol, @sobhatta, @afiqaize, @wrtabb, @Prasant1993, @varuns23, @ram1123 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

@clacaputo
Copy link
Contributor

type egamma

@smuzaffar
Copy link
Contributor

Milestone for this pull request has been moved to CMSSW_14_0_X.Please open a backport if it should also go in to CMSSW_13_3_X.

@smuzaffar smuzaffar modified the milestones: CMSSW_13_3_X, CMSSW_14_0_X Nov 6, 2023
@cmsbuild cmsbuild modified the milestones: CMSSW_14_0_X, CMSSW_13_3_X Nov 6, 2023
@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 6, 2024

Milestone for this pull request has been moved to CMSSW_14_1_X. Please open a backport if it should also go in to CMSSW_14_0_X.

@cmsbuild cmsbuild modified the milestones: CMSSW_14_0_X, CMSSW_14_1_X Feb 6, 2024
@cmsbuild cmsbuild added this to the CMSSW_14_1_X milestone Feb 12, 2024
@mandrenguyen
Copy link
Contributor

-reconstruction
Cleaning queue

@vlimant
Copy link
Contributor

vlimant commented Jul 4, 2024

assign xpog
I wonder why we were not tagged for MINI

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 4, 2024

New categories assigned: xpog

@vlimant,@hqucms,@ftorrresd you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

Milestone for this pull request has been moved to CMSSW_14_2_X. Please open a backport if it should also go in to CMSSW_14_1_X.

@antoniovilela
Copy link
Contributor

ping (to make bot change milestone)

@cmsbuild
Copy link
Contributor

Milestone for this pull request has been moved to CMSSW_15_0_X. Please open a backport if it should also go in to CMSSW_14_2_X.

@cmsbuild cmsbuild modified the milestones: CMSSW_14_2_X, CMSSW_15_0_X Nov 22, 2024
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.

7 participants