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

DD4Hep complains about duplicate volume #31798

Closed
bsunanda opened this issue Oct 14, 2020 · 9 comments
Closed

DD4Hep complains about duplicate volume #31798

bsunanda opened this issue Oct 14, 2020 · 9 comments

Comments

@bsunanda
Copy link
Contributor

DD4Hep complains of duplicate volume when it finds the same name and copy number is positioned multiple time. Those volumes are skipped and thus the geometry is incomplete for dd4hep. This happens for EB geometry as well as for PPS. In case of PPS, the same name is used in 2 separate XML files. So a combination of name and namespace resolves the case for DDD in case of PPS. The issue for EB is not understood - it may have something to do with the way DD4Hep assigns name of a volume. Name is important for EB numbering scheme.

@cmsbuild
Copy link
Contributor

A new Issue was created by @bsunanda Sunanda Banerjee.

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

cms-bot commands are listed here

@makortel
Copy link
Contributor

assign geometry

@cmsbuild
Copy link
Contributor

New categories assigned: geometry

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

@vargasa
Copy link
Contributor

vargasa commented Oct 15, 2020

@bsunanda Is this different from #31466 ?

@cvuosalo
Copy link
Contributor

FYI @slomeo

@cvuosalo
Copy link
Contributor

cvuosalo commented Dec 4, 2020

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 4, 2020

This issue is fully signed and ready to be closed.

@cvuosalo
Copy link
Contributor

cvuosalo commented Dec 4, 2020

@bsunanda I think this issue is fixed. Could you please close it?

@bsunanda
Copy link
Contributor Author

bsunanda commented Dec 4, 2020

This issue is fixed

@bsunanda bsunanda closed this as completed Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants