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

ECAL for Time dependent MC. Premix option C. Handle for APD/PN tag for time dependence #28214

Merged

Conversation

amassiro
Copy link
Contributor

First implementation of the ECAL run dependent MC
Similar to #27970, but that was in 10_6_X
Similar to #27974, but after an email exchange with db experts some of the issues from previous PR have been fixed.

To be tested it needs this PR: #27649

Physics details in the presentation at ECAL DPG here: https://indico.cern.ch/event/847097/

This PR will affect only the mixing step, when the premixed library is mixed with the hard scattering. The idea is that the pileup response, that has been simulated with a given transparency for ECAL crystals, will be scaled to match the transparency of the corresponding IOV.
A new object is introduced ( EcalLaserAPDPNRatiosMCRcd ), that has 1 single IOV, thus allowing this to be used as reference for scaling.
Goal: correct simulation of the photostatistics for pileup, when producing run dependent MC with several IOVs (main changes in run dependent MC for ECAL are transparency and noise).

This PR is needed for testing via relval, checking physics performance and possibly imploved data-MC comparison.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@amassiro amassiro changed the title Amassiro premix option c 2tags ecal 11 0 x Premix option c, with 2 tags, ECAL, 11_0_X, no new object for apdpnMC Oct 18, 2019
@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28214/12331

Code check has found code style and quality issues which could be resolved by applying following patch(s)

…m:amassiro/cmssw into amassiro_premix_OptionC_2tags_ECAL_11_0_X
@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28214/12332

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 8, 2020

+1
Tested at: e4810f5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0c4e95/4161/summary.html
CMSSW: CMSSW_11_1_X_2020-01-07-2300
SCRAM_ARCH: slc7_amd64_gcc820

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 8, 2020

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 8, 2020

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0c4e95/4161/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 4 differences found in the comparisons
  • DQMHistoTests: Total files compared: 34
  • DQMHistoTests: Total histograms compared: 2815513
  • DQMHistoTests: Total failures: 2
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2815165
  • DQMHistoTests: Total skipped: 346
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
  • Checked 147 log files, 16 edm output root files, 34 DQM output files

@silviodonato
Copy link
Contributor

alca: @christopheralanwest @franzoni @tlampen @pohsun @tocheng
db: @ggovi
simulation: @mdhildreth @civanch
upgrade: @kpedro88
please review/sign

@kpedro88
Copy link
Contributor

kpedro88 commented Jan 9, 2020

+upgrade

@civanch
Copy link
Contributor

civanch commented Jan 9, 2020

+1

@christopheralanwest
Copy link
Contributor

+1

@ggovi
Copy link
Contributor

ggovi commented Jan 13, 2020

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

@silviodonato
Copy link
Contributor

+1

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