SimVertexs collection for FastTiming studies

[simonepigazzini]

Adding SimVertex collection for interaction vertices only (PU+Signal), the collection are produced as a slimmer version of the CrossingFrame transient collection. This PR addresses the memory issues of the default CFWriter module collections.

@amartelli

[cmsbuild]

A new Pull Request was created by @simonepigazzini for CMSSW_6_2_X_SLHC.

SimVertexs collection for FastTiming studies

It involves the following packages:

Configuration/Applications
Configuration/StandardSequences
SimGeneral/MixingModule

@civanch, @nclopezo, @mdhildreth, @cmsbuild, @franzoni, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @wmtan, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @appeltel, @cerati, @dgulhan 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.
@fratnikov, @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.

[cmsbuild]

-1

Tested at: 391122d
I found errors in the following unit tests:

---> test TestConfigDP had ERRORS

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8891/4420/summary.html

[cmsbuild]

-1

Tested at: 391122d
I found errors in the following unit tests:

---> test TestConfigDP had ERRORS

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8891/4440/summary.html

[simonepigazzini]

compiling with 'scram b runtests' locally does not show any error

[fratnikov]

@simonepigazzini The problem can be reproduced with (C-shell):
setenv LOCAL_TEST_DIR $CMSSW_RELEASE_BASE/src/Configuration/DataProcessing/test
$CMSSW_RELEASE_BASE/src/Configuration/DataProcessing/test/run_CfgTest.sh

[fratnikov]

Looks like the problem is not related to this PR

[davidlange6]

the unit test is also failing in the integration build - so indeed, its failure is unrelated.

On Apr 29, 2015, at 10:23 PM, Fedor Ratnikov notifications@github.com wrote:

Looks like the problem is not related to this PR

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

[mark-grimes]

Just to update this PR with details discussed in private mail:
There is a big memory leak when reloading crossing frames from disk. We are currently investigating doing the FastTimer studies using the TrackingVertex class instead of using SimVertex in CrossingFrames. If this bears fruit this PR will be obsolete. I propose keeping it open until we know that works though.

[mark-grimes]

@simonepigazzini, you can close this now. The required changes were introduced with #8975 and #8979.

[simonepigazzini]

thanks @mark-grimes. closing now