Castor enabled

[civanch]

Castor enabled by default.

[cmsbuild]

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

Castor enabled

It involves the following packages:

SimG4Core/Application

@cmsbuild, @civanch, @Degano, @mdhildreth, @nclopezo can you please review it and eventually sign? Thanks.
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.

[civanch]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests.

[davidlange6]

@civanch - I had the impression yesterday that we decided instead to enable via a customize function for the 1-2 samples that is interesting for castor. (?)

[civanch]

@davidlange6 - they need to do comparison with "what was before". One way to provide this comparison is to enable Castor by default in 7_2_0_pre2, so they will have MinBias sample similar to previous. After they will do their tests we switch off Castor again and define only few workflows where Castor will be enabled.

Alternative is to request a special GEN_SIM sample with 7_2_0_pre1 only for Castor, in that case this PR is not needed, my worry is that MinBias will be different in that case.

[davidlange6]

what differences are you thinking can which can affect anyone (aside from the castor group of course)?

On Jun 25, 2014, at 12:14 PM, Vladimir Ivantchenko notifications@github.com
wrote:

@davidlange6 - they need to do comparison with "what was before". One way to provide this comparison is to enable Castor by default in 7_2_0_pre2, so they will have MinBias sample similar to previous. After they will do their tests we switch off Castor again and define only few workflows where Castor will be enabled.

Alternative is to request a special GEN_SIM sample with 7_2_0_pre1 only for Castor, in that case this PR is not needed, my worry is that MinBias will be different in that case.

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

[civanch]

if log file is not grow (main problem in past) - no effect, except a little performance penalty (few %) and ~50 M extra memory.

[cmsbuild]

+1
Tested at: 9b00d0a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4380/182/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine).

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4380/182/summary.html