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

Change WorldVolume material from Air #1116

Closed
aciarma opened this issue May 25, 2023 · 7 comments
Closed

Change WorldVolume material from Air #1116

aciarma opened this issue May 25, 2023 · 7 comments
Labels
Fixed Problem is solved question Waiting for caller Waiting for issue opener to respond to question

Comments

@aciarma
Copy link
Contributor

aciarma commented May 25, 2023

Hi,

the WorldVolume is created automatically and it is assigned the material 'Air'.
Is there a way to change this material? (eg. to 'Vacuum')
Because I don't know where to put this attribute, as the creation of this volume is hidden.

I am working on lxplus with key4hep.

Thanks,
Andrea

@MarkusFrankATcernch
Copy link
Contributor

Hi,
I think you cannot only set the material, but you have to define externally the world volume.
Please see the test example here: WorldVolume.xml

@MarkusFrankATcernch
Copy link
Contributor

@aciarma Hi Andrea,
Did this answer solve your problem?
Markus

@MarkusFrankATcernch MarkusFrankATcernch added Fixed Problem is solved Waiting for caller Waiting for issue opener to respond to question labels Jun 6, 2023
@aciarma
Copy link
Contributor Author

aciarma commented Jun 15, 2023

Hi @MarkusFrankATcernch

sorry for the very late reply!
Actually if I do like that it seems like the world volume is created by default:

DD4hep           WARN  ++ STD conditions NOT defined by client. NTP defaults taken.
Detector         INFO  *********** Created World volume with size: 2500 2500 2500

and it is made of Air.
I am using k4run to run my scripts.

Edit:
if I do geoDisplay WorldMaterial.xml I get the following error:

DD4hep           WARN  ++ STD conditions NOT defined by client. NTP defaults taken.
RunPlugin        ERROR ++ Exception while executing plugin DD4hep_CompactLoader:
		Handle_t::child: Element [world] has no child of type 'shape'
dd4hep: while parsing ../Detector/DetFCCeeCLD/compact/FCCee_o2_v03/WorldMaterial.xml
dd4hep: with plugin:DD4hep_CompactLoader
Application      ERROR geoPluginRun: Uncaught exception: RunPlugin: ++ Exception while executing plugin DD4hep_CompactLoader:
		Handle_t::child: Element [world] has no child of type 'shape'

@MarkusFrankATcernch
Copy link
Contributor

@aciarma Did you update the code and recompile?
The test is running in the release procedure. There must be some difference.

@aciarma
Copy link
Contributor Author

aciarma commented Jun 15, 2023

I am sourcing the latest key4hep release, so DD4hep should be updated?

@MarkusFrankATcernch
Copy link
Contributor

Don't know which version key4hep uses. This only works with the dd4hep PR 1126.

@aciarma
Copy link
Contributor Author

aciarma commented Jun 15, 2023

Ok thank you very much, so I did a fresh install of DD4hep and the example works.
I expected the key4hep environment to be in sync with DD4hep, as it is an important part of it...

I tried sourcing thisdd4hep_only.sh after sourcing the key4hep setup, but I noticed that it does not overwrite the DD4hep env variables somehow, so pointing to the "wrong" ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed Problem is solved question Waiting for caller Waiting for issue opener to respond to question
Projects
None yet
Development

No branches or pull requests

2 participants