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

Use HCAL Phase1 digi/reco for Phase2 #16365

Merged
merged 8 commits into from Nov 3, 2016

Conversation

kpedro88
Copy link
Contributor

Now that the HCAL Phase1 digi/reco is near maturity, it is preferable to use the new dataframes and algorithms for Phase2 (rather than existing "generic" versions from the SLHC release). This PR addresses several points:

  1. Introduce run3_HB Era to propagate a few necessary changes for the HB upgrade (scheduled for LS2)
  2. Chain Phase1 HCAL Eras, digitization, reco algorithms, and validation/DQM into Phase2
  3. Add option to apply respcorr of 0.5/1.2 to depth 1 (layer 0) when using hardcode conditions

Some things are still skipped for the Phase2 HCAL (packing/unpacking, noise flagging) due to lack of hardware maps.

A subsequent PR will remove HcalUpgradeDataFrame and related classes/code.

This PR will conflict with #16315; I would prefer to have that one merged first and then rebase this one.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @kpedro88 (Kevin Pedro) for CMSSW_8_1_X.

It involves the following packages:

CalibCalorimetry/HcalAlgos
CalibCalorimetry/HcalPlugins
Configuration/Eras
Configuration/Geometry
Configuration/StandardSequences
DQM/HcalTasks
RecoLocalCalo/Configuration
SimCalorimetry/Configuration
SimCalorimetry/HcalSimProducers
SimCalorimetry/HcalTrigPrimProducers
Validation/HcalDigis

@ghellwig, @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @dmitrijus, @cmsbuild, @rekovic, @franzoni, @cerminar, @slava77, @mmusich, @vanbesien, @mulhearn, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @tocheng, @deguio, @mariadalfonso, @dgulhan this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

@kpedro88
Copy link
Contributor Author

@cmsbuild please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 26, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/15989/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

@kpedro88
Copy link
Contributor Author

Changes observed only in the 2023D3 workflow, primarily in HCAL- and Jet-related plots, as expected.

@cmsbuild
Copy link
Contributor

Pull request #16365 was updated. @ghellwig, @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @dmitrijus, @cmsbuild, @rekovic, @franzoni, @cerminar, @slava77, @mmusich, @vanbesien, @mulhearn, @davidlange6 can you please check and sign again.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 2, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/16160/console

@slava77
Copy link
Contributor

slava77 commented Nov 3, 2016

compared to #16365 (comment)
things are much more symmetric now
sign791_318f57e_hbhe_eraw1_d1233

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2016

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2016

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2016

@ianna
Copy link
Contributor

ianna commented Nov 3, 2016

+1

@dmitrijus
Copy link
Contributor

+1

@kpedro88
Copy link
Contributor Author

kpedro88 commented Nov 3, 2016

@mmusich @rekovic @slava77 @civanch @davidlange6 I would like to get this into pre16...

@slava77
Copy link
Contributor

slava77 commented Nov 3, 2016

After the last updates:

  • the asymmetry is gone at jet response level as well
    wf1338in2023d3timing_318f57e_ak4calo_corogen_v_eta_200to600
    wf1338in2023d3timing_318f57e_ak4pf_corogen_v_eta_200to600
  • the average response now slightly undershoots with a minimum around 200-400 GeV, which will get calibrated at an appropriate level. This is still about a half better than the overshot before
    wf1338in2023d3timing_318f57e_ak4calo_corogen_v_pt_b
    wf1338in2023d3timing_318f57e_ak4pf_corogen_v_pt_b
  • it's somewhat odd how the HF scale is treated in pf: caloJets see no change while the PF changes quite noticeably
    wf1338in2023d3timing_318f57e_ak4calo_corogen_v_pt_f
    wf1338in2023d3timing_318f57e_ak4pf_corogen_v_pt_f

@slava77
Copy link
Contributor

slava77 commented Nov 3, 2016

+1

for #16365 318f57e

  • code changes appear to be in line with expectations and the follow up review
  • jenkins tests pass and comparisons with baseline show differences only in D3 2023 workflow
  • local tests show somewhat expected behavior (see notes posted earlier)

@civanch
Copy link
Contributor

civanch commented Nov 3, 2016

+1

@davidlange6
Copy link
Contributor

hi all - any further comments on this PR?

@mmusich
Copy link
Contributor

mmusich commented Nov 3, 2016

+1

@davidlange6 davidlange6 merged commit 1ca2d42 into cms-sw:CMSSW_8_1_X Nov 3, 2016
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.

None yet

8 participants