[DD4hep] [Cosmetic] Harmless warning messages about overwriting some constants should be quieted by removing duplicate definitions

[cvuosalo]

When the DD4hep geometry is built, the following warning messages are output:

Evaluator        WARN  +++ Overwriting variable: name=world_x value=101*m  Evaluator::Object : existing variable [setVariable error]
Evaluator        WARN  +++ Overwriting variable: name=world_y value=101*m  Evaluator::Object : existing variable [setVariable error]
Evaluator        WARN  +++ Overwriting variable: name=world_z value=450*m  Evaluator::Object : existing variable [setVariable error]

Step 1 of the DD4hep workflow 11634.911 shows these messages.
The problem is that these variables have duplicate definitions in XML files and in DetectorDescription/DDCMS/plugins/dd4hep/DDDefinitions2Objects.cc. The duplicate definitions need to be found and removed so these constants are only defined once.
These warnings are harmless, but it would be better to remove them by eliminating the duplicate definitions.

[cmsbuild]

A new Issue was created by @cvuosalo Carl Vuosalo.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[cvuosalo]

@ianna FYI, as you requested

[cvuosalo]

assign geometry

[cmsbuild]

New categories assigned: geometry

@Dr15Jones,@cvuosalo,@civanch,@ianna,@mdhildreth,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cvuosalo]

This issue is more complex than expected. These parameters do not appear in any of the xml files used in defining the scenario. They are entered in the file:
DetectorDescription/DDCMS/plugins/dd4hep/DDDefinitions2Objects.cc

They may need to be defined there. There may be conflicts within dd4hep code which may try to define the same name elsewhere.

[cvuosalo]

The source of these warnings is inside of DD4hep. A DD4hep issue has been opened:
AIDASoft/DD4hep#844

[cvuosalo]

Issue resolved by #35594.

[cvuosalo]

+1

[cmsbuild]

This issue is fully signed and ready to be closed.