Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Scenarios for SD: HGCAL ee18 without Tracker Extension #10156

Merged
merged 5 commits into from Jul 13, 2015

Conversation

boudoul
Copy link
Contributor

@boudoul boudoul commented Jul 12, 2015

As requested for the Scope Document, I created a new scenario which corresponds to
the scope scenario HGCAL version ee18 but where the tracker is not extended at high eta .
This PR can only work with #10152

the New scenario are
No PU --> -l 176XX
PU --> -l 178XX
(the latter cannot work yet, we need MinBias which this new geometry which will be generated with the next set of relvals )

NB at the same time , I reduced the length of the name for the already existing hgcal descope scenario, the string length was preventing the workflows to be injected in relvals production..

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @boudoul (boudoul) for CMSSW_6_2_X_SLHC.

New Scenarios for SD: HGCAL ee18 without Tracker Extension

It involves the following packages:

Configuration/PyReleaseValidation
RecoParticleFlow/PandoraTranslator

The following packages do not have a category, yet:

RecoParticleFlow/PandoraTranslator

@cmsbuild, @srimanob, @boudoul, @franzoni can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @mmarionncern, @bachtis, @lgray 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.

@mark-grimes
Copy link

For 17800 (Extended2023HGCalNoExtPix_ee18 with pileup) I get a segfault in digitisation:

#6  0x00007f4cbfd4d537 in cms::SiPixelDigitizer::accumulatePixelHits(edm::Handle<std::vector<PSimHit, std::allocator<PSimHit> > >, unsigned long, unsigned int, TrackerTopology const*) () from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc472/cms/cmssw/CMSSW_6_2_X_SLHC_2015-07-12-1100/lib/slc6_amd64_gcc472/pluginSimTrackerSiPixelDigitizerPlugins.so
#7  0x00007f4cbfd4d852 in cms::SiPixelDigitizer::accumulate(PileUpEventPrincipal const&, edm::EventSetup const&) () from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc472/cms/cmssw/CMSSW_6_2_X_SLHC_2015-07-12-1100/lib/slc6_amd64_gcc472/pluginSimTrackerSiPixelDigitizerPlugins.so
#8  0x00007f4cae602c6b in edm::MixingModule::accumulateEvent(PileUpEventPrincipal const&, edm::EventSetup const&) () from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc472/cms/cmssw/CMSSW_6_2_X_SLHC_2015-07-12-1100/lib/slc6_amd64_gcc472/pluginSimGeneralMixingModulePlugins.so
#9  0x00007f4cae602d04 in edm::MixingModule::pileAllWorkers(edm::EventPrincipal const&, int, int, int&, edm::EventSetup const&) () from /afs/cern.ch/cms/sw/ReleaseCandidates/vol1/slc6_amd64_gcc472/cms/cmssw/CMSSW_6_2_X_SLHC_2015-07-12-1100/lib/slc6_amd64_gcc472/pluginSimGeneralMixingModulePlugins.so

17662 (same but without pileup, different physics process but I don't think that's the problem) runs fine though. I guess the minimum bias file in defaultDataSets is invalid. @boudoul, is this expected since you need a release to create the minimum bias file? Should I ignore this error before creating the release?

@boudoul
Copy link
Contributor Author

boudoul commented Jul 13, 2015

thanks Mark, yes please ignore the PU (i mentionned this is the description of the PR) the minbias will be generated with a first set of relvals - Since it's a different geometry we need to produce first a consistent minbias before the PU scenario can work

@mark-grimes
Copy link

Oops, so you did. I really should pay more attention, sorry. I'll give the automatic tests on #10152 another half hour or so to complete, if they're not done by then I'll merge and ask for a release. My private tests of both PRs together should be enough, but since the tests are running might as well wait for the results.

@boudoul
Copy link
Contributor Author

boudoul commented Jul 13, 2015

perfect, thanks a lot Mark !

@mark-grimes
Copy link

merge

Tested privately with #10152, details of the testing are on there.

cmsbuild added a commit that referenced this pull request Jul 13, 2015
New Scenarios for SD: HGCAL ee18 without Tracker Extension
@cmsbuild cmsbuild merged commit 88250f6 into cms-sw:CMSSW_6_2_X_SLHC Jul 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants