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

Updating JEC in data/mc and fixes for HI #12349

Merged
merged 1 commit into from Nov 11, 2015

Conversation

mmusich
Copy link
Contributor

@mmusich mmusich commented Nov 10, 2015

Summary of changes in GT

RunII simulation

RunI data

  • RunI Offline processing : 76X_dataRun1_v10 as 76X_dataRun1_v11 with the following changes:
    • adding event plane calibrations for HI trough HeavyIonRPRcd;
    • adding label JEC with AK4Calo label for pp reference run at 5TeV;
    • updating AlCaReco Trigger bits to cover 2010 HI runs for new Tracker Alignment AlCaReco producers.

RunII data

  • RunII Offline processing : 76X_dataRun2_v10 as 76X_dataRun2_v11 with the following changes:
    • adding event plane calibrations for HI trough HeavyIonRPRcd;
    • adding label JEC with AK4Calo label for pp reference run at 5TeV;
    • updating AlCaReco Trigger bits to cover 2010 HI runs for new Tracker Alignment AlCaReco producers.
    • updated DATA JEC to v3 with updated L2L3Residuals and uncertainties for 25ns;

@mmusich
Copy link
Contributor Author

mmusich commented Nov 10, 2015

please test

@cmsbuild cmsbuild added this to the Next CMSSW_7_6_X milestone Nov 10, 2015
@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_7_6_X.

Updating JEC in data/mc and fixes for HI

It involves the following packages:

Configuration/AlCa

@cmsbuild, @mmusich, @diguida, @franzoni, @cerminar can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' or '@cmsbuild, please test' 'please test with cms-sw/cmsdist#PR' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@cmsbuild
Copy link
Contributor

Pull request #12349 was updated. @cmsbuild, @mmusich, @diguida, @franzoni, @cerminar can you please check and sign again.

@mmusich
Copy link
Contributor Author

mmusich commented Nov 10, 2015

@cmsbuild please test

@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@mmusich
Copy link
Contributor Author

mmusich commented Nov 11, 2015

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@davidlange6
Copy link
Contributor

a bit surprised that nothing seems to have changed in the comparisons of this PR.. Approving but might be worth a double check as I would have expected the DQM to consume the new JEC (naively)

davidlange6 added a commit that referenced this pull request Nov 11, 2015
Updating JEC in data/mc and fixes for HI
@davidlange6 davidlange6 merged commit fb0ece1 into cms-sw:CMSSW_7_6_X Nov 11, 2015
@mmusich
Copy link
Contributor Author

mmusich commented Nov 11, 2015

@davidlange6 as far as I could understand the JETMET POG provided new payloads with updates concerning only the uncertainties (used in the miniAOD and possibly nowhere else). Please @blinkseb @schoef confirm.

@schoef
Copy link
Contributor

schoef commented Nov 13, 2015

No changes expected in MC since MC JEC are identical. Uncertainties are presumably not tested.

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

4 participants