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

MiniAOD: met filter flag on bad muon #15761

Closed
wants to merge 16 commits into from

Conversation

mariadalfonso
Copy link
Contributor

Here additional MET filter flags

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 7, 2016

A new Pull Request was created by @mariadalfonso for CMSSW_8_1_X.

It involves the following packages:

PhysicsTools/PatAlgos
RecoMET/METFilters

@cmsbuild, @cvuosalo, @slava77, @monttj, @davidlange6 can you please review it and eventually sign? Thanks.
@ferencek, @TaiSakuma, @imarches, @ahinzmann, @acaudron, @mmarionncern, @rappoccio, @jdolen, @nhanvtran, @JyothsnaKomaragiri, @gkasieczka, @schoef, @gpetruc, @mverzett, @pvmulder this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@slava77
Copy link
Contributor

slava77 commented Sep 7, 2016

@cmsbuild please test

looks like the various filters are generated by copy-paste with minor modifications.
It would be better to have a more flexible filter that can handle more cases so that we do not generate new copy-pasted code that may have maintenance issues if ever needs to be updated.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 7, 2016

@slava77
Copy link
Contributor

slava77 commented Sep 7, 2016

since this is considered not essential for rereco, I'd rather we properly review this at regular pace.
Code quality is not really at production level.

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 7, 2016

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 7, 2016

@cvuosalo
Copy link
Contributor

cvuosalo commented Sep 8, 2016

@mariadalfonso: Massive copy/paste of code is not an advisable software development method for maintainable production code. Instead of five nearly identical filter classes, there should only be one which is parameterized and configurable for multiple needs. This configurability can be achieved with constructor parameters, templates, inheritance, or maybe multiple method functions.

@cvuosalo
Copy link
Contributor

@mariadalfonso: The 810pre12 deadline is rapidly approaching. What is the plan for completing this PR?

@mariadalfonso
Copy link
Contributor Author

I close these PR since I made those in a super hurry to meet the 8_0_20 re-reco deadline.

For the 81X - that target next year, if needed we can re-open other PRs that

  1. rework the class as you requested in MiniAOD: met filter flag on bad muon #15761
    and allow the needed large scale testing
  2. look at the Prompt-Reco H dataset with the various charged hadron PF-fixes and alignments and hw HIP fix.
  3. come up with a real PF fix for the remaining muon problems.

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.

5 participants