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

Roman Pots visualization not working when geometry dumped into ROOT file #493

Open
ajentsch opened this issue Aug 3, 2023 · 5 comments
Open

Comments

@ajentsch
Copy link
Contributor

ajentsch commented Aug 3, 2023

Environment: (where does this bug occur, have you tried other environments)

  • Which branch (main for latest released): main
  • Which revision (HEAD for the most recent):
  • Any specific OS or system where the issue occurs? Mac/Docker
  • Any special versions of ROOT or Geant4? No

Steps to reproduce: (give a step by step account of how to trigger the bug)

  1. Produce geometry: geoConverter -compact2tgeo -input [compact_file.xml] -output [geometry].root
  2. Open resulting file using a ROOT TBrowser. Other geometry component render just fine, but not the Roman Pots. It's likely to do with the overtly complicated structure to generate the geometry.

Expected Result: (what do you expect when you execute the steps above)

I except to be able to see the detector geometry.

Actual Result: (what do you get when you execute the steps above)

I do not see the detector geometry.

@simonge
Copy link
Contributor

simonge commented Aug 7, 2023

Some of this might stem from a ROOT bug/feature which is hopefully being fixed where the transparency of a volume is set at the material/medium level. The last volume which used that material appears to set the transparency for all preceding volumes built.
root-project/root#13402
I'm not certain this is the stem of all of the visualisation issues I've been seeing but the Roman Pots is not alone. A fix in ROOT will likely take a while to propagate to eic-shell if that is in fact the problem.

@ajentsch
Copy link
Contributor Author

ajentsch commented Aug 7, 2023 via email

@simonge
Copy link
Contributor

simonge commented Aug 7, 2023

Removing other detectors or swapping the order they are included in the main xml file changes the visualisation, it's very strange

@agheata
Copy link

agheata commented Aug 8, 2023

With the ROOT master, you should use TGeoVolume::ResetTransparency(char value) to set per volume (and not per material transparency). Calling ResetTransparency() with no value will use transparency per material (default behavior). See: root-project/root#13402

@Simple-Shyam
Copy link

@ajentsch, I have this feature in my visualization of geometry code where I loop over all the volumes and we can specify transparency and color for each volumes. The code can be used easily for event-display also.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants