Fixes for online visualization

[cerminar]

This PR addresses the following issues:

• GT for express is now used in he configuration adding the possibility to fetch it from Tier0das
• remove lumiProducer from cosmics reco sequences

Ideally we could test this in the ongoing MWGRs

[cmsbuild]

A new Pull Request was created by @cerminar for CMSSW_7_1_X.

Fixes for online visualization

It involves the following packages:

Configuration/DataProcessing
DQM/Integration

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

[deguio]

+1

[davidlange6]

the lumi producer changes are ok if we never plan to run the cosmics sequence on collisions data (certainly not an issue for today)

[danduggan]

Even if we do run this on collisions, we should suppress lumi producer as this won't be needed for the purposes of this configuration - event display at p5. i imagine for collisions the lumi producer will/should also be suppressed.

[davidlange6]

On Oct 8, 2014, at 9:42 AM, danduggan notifications@github.com
wrote:

Even if we do run this on collisions, we should suppress lumi producer as this won't be needed for the purposes of this configuration - event display at p5. i imagine for collisions the lumi producer will/should also be suppressed.

In the event display - sure - but this config (as implemented) is not just for the event display at p5.

—
Reply to this email directly or view it on GitHub.

[cerminar]

Hi @davidlange6 ,
not sure what do you mean: this is just editing the configuration for visualization: the call to remove the lumi producer is only in that particular method...
Cheers,
G

[davidlange6]

ah - I missed that…good. Thanks

On Oct 8, 2014, at 9:56 AM, cerminar notifications@github.com
wrote:

Hi @davidlange6 ,
not sure what do you mean: this is just editing the configuration for visualization: the call to remove the lumi producer is only in that particular method...
Cheers,
G

—
Reply to this email directly or view it on GitHub.

[cerminar]

Hi @danduggan ,
if I understand correctly the lumiProducer is already out of the reco for pp (and added back with a dedciated customize for Tier0). This is why I have not modified the pp scenarios.
Cheers,
G