EE front shielding

[ianna]

• Remove Al structure

[ianna]

+1

[cmsbuild]

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

EE front shielding

It involves the following packages:

Fireworks/Geometry
Geometry/EcalCommonData

@Dr15Jones, @cmsbuild, @alja, @nclopezo, @Degano, @ktf can you please review it and eventually sign? Thanks.
@ghellwig, @alja, @argiro 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.
@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]

@ianna, presumably this is instead of #5028, correct?

[ianna]

@mark-grimes - yes, please, do not take the #5028

[alja]

@ianna

Why don't use dumpSimGeometry_cfg.py? What is the different use case for dumpSimGeometryXML_cfg.py?

[alja]

-1

Can we please hold this PR till usage of dumpSimGeometryXML_cfg.py/dumpSimGeometry_cfg.py is cleared out. I want to avoid duplication in maintenance.

[alja]

+1
I had a look and can see dumpSimGeometryXML_cfg.py is more generic version of dumpSimGeometry_cfg.py.

@ianna
It looks like dumpSimGeometry_cfg.py should be removed or dumpSimGeometryXML_cfg.py be renamed to dumpSimGeometry_cfg.py

Does the same change goes to 7x?

[cmsbuild]

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

[ianna]

@alja - yes, the directory structure is the same in 7xx. I'd also add a DB GT in the command line. But it's not urgent.

[mark-grimes]

I've separated off the geometry issue into a separate pull request #5074. Could you continue the dumpSimGeometry discussion on #5061, and anything to do with the shielding on #5074, so that the two issues are discussed separately?

@ianna could you close this please? Thanks.

[ianna]

@mark-grimes - this one is fully signed - why it cannot be integrated in an IB?

[alja]

+1

We have cleared the dumpSimGeometry discussion.

[mark-grimes]

merge