Navigation Menu

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

GE2/1 geometry #1923

Closed
wants to merge 1 commit into from
Closed

GE2/1 geometry #1923

wants to merge 1 commit into from

Conversation

ianna
Copy link
Contributor

@ianna ianna commented Dec 27, 2013

  • GE2/1 geometry back ported from 70x
  • Corrected volume names and selection of pilot GE1/1 geometries
  • Bugfix in Heavy borated concrete material to avoid warning from G4
  • Extended 2023 scenario changed to use current HCAL for the moment

@ianna
Copy link
Contributor Author

ianna commented Dec 27, 2013

@davidlange6 Please, take it for the next SLHC release

@ianna
Copy link
Contributor Author

ianna commented Dec 27, 2013

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @davidlange6 can you please take care of it?

@davidlange6
Copy link
Contributor

@ianna - why the HCAL change?

@ianna
Copy link
Contributor Author

ianna commented Dec 27, 2013

@davidlange6 - so that a full SIM-DIGI-RECO chain would work with GEM. There are two other 2023 scenarios: Extended2023HGCal and Extended2023SHCal. None of them is ready for the workflow yet.

@davidlange6
Copy link
Contributor

What errors come in with the phase 1 hcal? It was working

-----Original Message-----
From: ianna [notifications@github.commailto:notifications@github.com]
Sent: Friday, December 27, 2013 02:07 PM Pacific Standard Time
To: cms-sw/cmssw
Cc: davidlange6
Subject: Re: [cmssw] GE2/1 geometry (#1923)

@davidlange6https://github.com/davidlange6 - so that a full SIM-DIGI-RECO chain would work with GEM. There are two other 2023 scenarios: Extended2023HGCal and Extended2023SHCal. None of them is ready for the workflow yet.


Reply to this email directly or view it on GitHubhttps://github.com//pull/1923#issuecomment-31282268.

@ianna
Copy link
Contributor Author

ianna commented Dec 28, 2013

The 2023 default was set with HGCal.

@davidlange6
Copy link
Contributor

@ianna - I see - indeed, I see I was using something w/ be5d as the default for our "2023" tests… Need to change that - but indeed we should keep the 2023 default geometry working. I'll take this and then try to switch the gen-sim-digi-reco relval tests to this geometry

On Dec 27, 2013, at 11:51 PM, ianna notifications@github.com wrote:

The 2023 default was set with HGCal.


Reply to this email directly or view it on GitHub.

@ianna
Copy link
Contributor Author

ianna commented Jan 6, 2014

Preliminary validation is presented by @dildick during last Friday meeting:
https://indico.cern.ch/getFile.py/access?contribId=3&resId=3&materialId=slides&confId=290659

To test GEN-SIM-DIGI step:

cmsDriver.py SingleMuPt100_cfi -s GEN,SIM,DIGI --conditions auto:upgrade2019 --customise SLHCUpgradeSimulations/Configuration/combinedCustoms.cust_2019WithGem --magField 38T_PostLS1 --datatier GEN-SIM-DIGI --geometry Extended2023 --eventcontent FEVTDEBUG -n 10000 --no_exec --fileout out_digi.root

@ianna
Copy link
Contributor Author

ianna commented Jan 15, 2014

Please, take #2031 instead.

@ianna ianna closed this Jan 15, 2014
@dildick dildick deleted the backport-the-ge21-geometry-to-SLHC branch January 28, 2014 09:07
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