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 pA autoconditions for 80X #16962

Closed

Conversation

mandrenguyen
Copy link
Contributor

Update to latest GT with updated reco BS. Needed for self-consistency of relval wf 281 after #16787
Unfortunately, this GT crashes wf 281, seemingly due to an update to the L1 menu. @mmusich can you please advise? The error is as follows.

%MSG-e GlobalObjectMapRecord: HLTL1TSeed:hltL1sCastorMediumJetBptxAND 10-Dec-2016 10:29:47 CET Run: 1 Event: 1

ERROR: The requested algorithm name = L1_CastorMediumJet_BptxAND
does not exists in the trigger menu.
Returning zero pointer for getObjectMap

%MSG
----- Begin Fatal Exception 10-Dec-2016 10:29:47 CET-----------------------
An exception of category 'FailModule' occurred while
[0] Processing run: 1 lumi: 1 event: 1
[1] Running path 'HLT_PAL1CastorMediumJet_BptxAND_v1'
[2] Calling event method for module HLTL1TSeed/'hltL1sCastorMediumJetBptxAND'
Exception Message:

Algorithm L1_CastorMediumJet_BptxAND, requested as seed by a HLT path, cannot be matched to a L1 algo name in any GlobalObjectMap
Please check if algorithm L1_CastorMediumJet_BptxAND is present in the L1 menu

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mandrenguyen for CMSSW_8_0_X.

It involves the following packages:

Configuration/AlCa

@ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @ghellwig, @tocheng 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

@mmusich
Copy link
Contributor

mmusich commented Dec 10, 2016

-1
@mandrenguyen
there are several problems with this PR.

  • why is it not part of pA vertex smearing update (80X) #16787 ?
  • this GT (as it was this morning) was not suitable for tests. It has been fixed now. From where did you got the impression that it was OK for your purposes?

As of now there are no leftover issues with the proposed GT (i.e. 80X_mcRun2_pA_v3).
The only thing changed w.r.t 80X_mcRun2_pA_v2 is the BeamSpot: https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/80X_mcRun2_pA_v3/80X_mcRun2_pA_v2
This can be checked via:

cmsrel CMSSW_8_0_X_2016-12-09-2300
cd CMSSW_8_0_X_2016-12-09-2300/src/
git cms-merge-topic 16962
scramv1 b -j 8
cmsenv
voms-proxy-init -voms cms -rfc
time runTheMatrix.py -i all -l 281.0

which returns:

281.0_EPOS_PPb_8160GeV_MinimumBias+EPOS_PPb_8160GeV_MinimumBias+DIGIUP15_PPb+RECOUP15_PPb+HARVESTUP15_PPb Step0-PASSED Step1-PASSED Step2-PASSED Step3-PASSED  - time date Sat Dec 10 12:35:56 2016-date Sat Dec 10 12:21:25 2016; exit: 0 0 0 0
1 1 1 1 tests passed, 0 0 0 0 failed

@mmusich
Copy link
Contributor

mmusich commented Dec 10, 2016

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 10, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/16937/console Started: 2016/12/10 12:57

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

@mmusich
Copy link
Contributor

mmusich commented Dec 11, 2016

+1
provided it is merged together #16787

@cmsbuild
Copy link
Contributor

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

@mandrenguyen
Copy link
Contributor Author

Superseded by #16975

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

3 participants