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

Bph reco refactored 12 4 x #38525

Closed
wants to merge 237 commits into from

Conversation

ronchese
Copy link
Contributor

@ronchese ronchese commented Jun 27, 2022

PR description:

The BPH reconstruction package (doc. in https://twiki.cern.ch/twiki/bin/view/CMS/BPHReco_V2 , to be updated with new features)
in HeavyFlavorAnalysis/RecoDecay and HeavyFlavorAnalysis/SpecificDecay has been redesigned using templated interfaces for common decay patterns.

PR validation:

Tested on a UL MINIAOD file (Charmonium dataset), same output as previous code.
All test in runtheMatrix passed.

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:

Backport to 12_4 of PR 38524 for 12_5
Needed to include BPH reconstruction plots in offline DQM

Jiwon and others added 30 commits May 17, 2022 03:43
…te_header_size

Add a method to `EventMsgBuilder` to precompute the message header size [12.4.x]
RPC DQM updates for Run3 [backport to 12_4_X]
…sToTaus

link lost tracks to taus [12_4_X]
…_for124X

HLT menu development for `12_3_X` [`12_4_X`]
…_deepTau-v2p5

Changes to enable new DeepTauId trainig [12_4_X]
Production jobs were seeing scrambled job reports caused by
pileup file entries.
…ionalTask

[12_4_X] Fix behavior of SwitchProducer in ConditionalTask with non-chosen EDAlias case(s) that are consumed explicitly with callWhenNewProductsRegistered()
[12.4.X] address undef output of dnn - for issue cms-sw#37970
Pixel Heterogeneous DQM: add module rechit @cpu / @gpu comparisons [Backport to 12.4.X]
@cmsbuild
Copy link
Contributor

@tvami
Copy link
Contributor

tvami commented Jun 27, 2022

-1

  • seems like the branch is for 12_4_X

@ronchese
Copy link
Contributor Author

This IS a branch for 12_4_X, checked out from CMSSW_12_4_X_2022-06-26-2300, backporting PR 38524.
Did I do something wrong?

@tvami
Copy link
Contributor

tvami commented Jun 28, 2022

Did I do something wrong?

Yes, that this is the 12_5_X target
Screen Shot 2022-06-28 at 09 08 34

@cmsbuild
Copy link
Contributor

@tvami
Copy link
Contributor

tvami commented Jun 28, 2022

-1

  • the PR is closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment