Change TTI beam spot from 'Gauss' to 'HLLHC'

[mark-grimes]

No description provided.

[cmsbuild]

A new Pull Request was created by @mark-grimes (Mark Grimes) for CMSSW_6_2_X_SLHC.

Change TTI beam spot from 'Gauss' to 'HLLHC'

It involves the following packages:

Configuration/PyReleaseValidation

@nclopezo, @vlimant, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig 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.
@andersonjacob, @mark-grimes you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[mark-grimes]

Ping @boudoul
Thought I should bring this to your attention in case it conflicts with anything you're doing. I will test and merge (if it passes) in a minute.

[boudoul]

thanks Mark, that's ok!

[mark-grimes]

All other test are as before. The TTI test (12800) now fails in step 2:

----- Begin Fatal Exception 10-Mar-2014 15:58:57 CET-----------------------
An exception of category 'Conditions mismatch' occurred while
   [0] Processing run: 1 lumi: 1 event: 1
   [1] Running path 'digitisation_step'
   [2] Calling event method for module MixingModule/'mix'
Exception Message:
Requested conditions of type HcalMCParams for cell (0x49180c30) (HF 3,48,1) got conditions for cell (0x48007329) (HF 38,41,1)
----- End Fatal Exception -------------------------------------------------

I don't understand how this change could cause that. I'll double check and make sure I'm not doing anything stupid with the integration.

[andersonjacob]

Hi Mark,

This is an interesting message. There isn't an HF 3 nor has there ever
been or will there probably ever be one. I suspect this is a problem with
a SLHC flag being set or not set correctly in a customization.

Jake

On Mon, Mar 10, 2014 at 10:10 AM, Mark Grimes notifications@github.comwrote:

All other test are as before. The TTI test (12800) now fails in step 2:

----- Begin Fatal Exception 10-Mar-2014 15:58:57 CET-----------------------
An exception of category 'Conditions mismatch' occurred while
[0] Processing run: 1 lumi: 1 event: 1
[1] Running path 'digitisation_step'
[2] Calling event method for module MixingModule/'mix'
Exception Message:
Requested conditions of type HcalMCParams for cell (0x49180c30) (HF 3,48,1) got conditions for cell (0x48007329) (HF 38,41,1)
----- End Fatal Exception -------------------------------------------------

I don't understand how this change could cause that. I'll double check and
make sure I'm not doing anything stupid with the integration.

Reply to this email directly or view it on GitHubhttps://github.com//pull/2790#issuecomment-37192584
.

[mark-grimes]

Found the problem. Copy and paste screw up didn't apply the upgrade customisation. Thanks Jake. I'll apply a fix in a minute.

[cmsbuild]

Pull request #2790 was updated. @nclopezo, @vlimant, @cmsbuild, @franzoni, @Degano, @davidlange6 can you please check and sign again.

[mark-grimes]

merge

Tests 10000, 10200, 10400, 11200, 11400, 11600, 11800, 12000, 12800 and 13000 pass all steps.
Tests 12200, 12400, 12600, 13200 and 13400 fail in step 2 with known errors.