fixed default HF file name

[civanch]

Fixed default name of HF shower library

[cmsbuild]

A new Pull Request was created by @civanch (Vladimir Ivantchenko) for CMSSW_5_3_X.

fixed default HF file name

It involves the following packages:

SimG4CMS/Calo
SimG4Core/Application

@cmsbuild, @civanch, @nclopezo, @mdhildreth can you please review it and eventually sign? Thanks.
@makortel 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.
@smuzaffar you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[civanch]

please test

[civanch]

+1

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_5_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar, @ktf

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_5_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar, @ktf

[cmsbuild]

+1
Tested at: 03a19fa
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8492/3749/summary.html

[cmsbuild]

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

[davidlange6]

+1

[franzoni]

@bendavid @srimanob

hello @civanch,
what is the effect of these changes on the standard SIM sequence of 5_3 x ?

We need to deploy this release in production to acquire some GEN new features.
Can we be sure we're not affecting the physics performance of SIM?

Cheers,
Giovanni

[davidlange6]

Hi @franzoni

There is no effect. the HF shower library is not used in 53x.

On Mar 26, 2015, at 7:22 PM, Giovanni Franzoni notifications@github.com wrote:

@bendavid @srimanob

hello @civanch,
what is the effect of these changes on the standard SIM sequence of 5_3 x ?

We need to deploy this release in production to acquire some GEN new features.
Can we be sure we're not affecting the physics performance of SIM?

Cheers,
Giovanni

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

[franzoni]

Thanks, David @davidlange6

[civanch]

Hi Giovanni,

New HF SL included in 5_3_X is not used by default, it may be enabled via
customisation. So, by default nothing should be changed in SIM.

Cheers,
Vladimir

On Thu, 26 Mar 2015, Giovanni Franzoni wrote:

@bendavid @srimanob

hello @civanch,
what is the effect of these changes on the standard SIM sequence of 5_3 x ?

We need to deploy this release in production to acquire some GEN new features.
Can we be sure we're not affecting the physics performance of SIM?

Cheers,
Giovanni

?
Reply to this email directly or view it on GitHub.[AEub7tHReAc1Up4B87YKg_k4Im5qWPbDks5n5EX5gaJpZM4Dzur4.gif]

[franzoni]

Thanks, Vladimir. g.

On 27 March 2015 at 18:18, Vladimir Ivantchenko notifications@github.com
wrote:

Hi Giovanni,

New HF SL included in 5_3_X is not used by default, it may be enabled via
customisation. So, by default nothing should be changed in SIM.

Cheers,
Vladimir

On Thu, 26 Mar 2015, Giovanni Franzoni wrote:

@bendavid @srimanob

hello @civanch,
what is the effect of these changes on the standard SIM sequence of 5_3
x ?

We need to deploy this release in production to acquire some GEN new
features.
Can we be sure we're not affecting the physics performance of SIM?

Cheers,
Giovanni

?
Reply to this email directly or view it on
GitHub.[AEub7tHReAc1Up4B87YKg_k4Im5qWPbDks5n5EX5gaJpZM4Dzur4.gif]

—
Reply to this email directly or view it on GitHub
#8492 (comment).