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

Aligning jet correction services for non-AK4 algorithms with current JME... #7088

Merged
merged 1 commit into from Jan 13, 2015

Conversation

rappoccio
Copy link
Contributor

Aligning jet correction services for non-AK4 algorithms with current JME recommendations. Was not properly propagated when the new constants were derived, so some users who used the CMSSW-based services applied the wrong corrections.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 8, 2015

A new Pull Request was created by @rappoccio for CMSSW_7_3_X.

Aligning jet correction services for non-AK4 algorithms with current JME...

It involves the following packages:

JetMETCorrections/Configuration

@cmsbuild, @vadler, @nclopezo, @monttj can you please review it and eventually sign? Thanks.
@TaiSakuma, @ahinzmann, @mmarionncern, @nhanvtran, @schoef, @mariadalfonso 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' in the first line of a comment.
@nclopezo 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

cmsbuild commented Jan 9, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 9, 2015

davidlange6 added a commit that referenced this pull request Jan 10, 2015
@monttj
Copy link
Contributor

monttj commented Jan 12, 2015

+1

@cmsbuild
Copy link
Contributor

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

@davidlange6
Copy link
Contributor

+1

@Martin-Grunewald
Copy link
Contributor

@rappoccio

I belive this PR is responsible for the following problem I see in an HIon workflow:

# with command line options: RelVal --step=RAW2DIGI,L1Reco,RECO,DQM --conditions=auto:run2_data_HIon --filein=file:RelVal_HLT_HIon_DATA.root --custom_conditions= --fileout=RelVal_RECO_HIon_DATA.root --number=25 --data --no_exec --datatier SIM-RAW-HLT-RECO --eventcontent=RAW --customise=HLTrigger/Configuration/CustomConfigs.Base --customise=SLHCUpgradeSimulations/Configuration/postLS1Customs.customisePostLS1 --customise= --scenario=HeavyIons --python_filename=RelVal_RECO_HIon_DATA.py

namely:

%MSG
----- Begin Fatal Exception 14-Jan-2015 11:52:01 CET-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing run: 182838 lumi: 365 event: 13651230
   [1] Running path 'dqmoffline_step'
   [2] Calling event method for module L1FastjetCorrectorProducer/'ak4PFL1FastjetCorrector'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: double
Looking for module label: fixedGridRhoFastjetAll
Looking for productInstanceName: 

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exce
ption,
add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet
 in the configuration.

----- End Fatal Exception -------------------------------------------------

@Martin-Grunewald
Copy link
Contributor

Sorry, no, I take it back, it is in 74X.
Anyway all HIon workflows, such as 300/301/302, seem broken with the above error in 74X.

@davidlange6
Copy link
Contributor

yes - seems most likely to be this one:

#6942
On Jan 14, 2015, at 3:44 PM, Martin Grunewald notifications@github.com
wrote:

Sorry, no, I take it back, it is in 74X.
Anyway all HIon workflows, such as 300/301/302, seem broken with the above error in 74X.


Reply to this email directly or view it on GitHub.

@rappoccio rappoccio deleted the JEC_Fix_73x branch April 22, 2015 18:45
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

5 participants