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

Add dd4hep-drawer util #356

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

Zehvogel
Copy link
Contributor

BEGINRELEASENOTES

  • Thank you for writing the text to appear in the release notes. It will show up
    exactly as it appears between the two bold lines
  • ...

ENDRELEASENOTES

Allows to draw parts of the detector tree while de-duplicating elements with counters in their names.
I did not quickly find something like this so it might also be useful for others :)

example output
CLD_o2_v06.gv.pdf

@atolosadelgado
Copy link
Contributor

Hi @Zehvogel

I think it is useful indeed, thank you!

Do you think it could be extended to display a similar diagram for the volume tree? I posted here a similar function that walks though the volume tree, but in C++... :/

I think visualizing the volume tree is useful to spot potential optimizations of geometry

@tmadlener
Copy link
Contributor

Is it possible to color the boxes of each layer in a different color? Or are they guaranteed to align horizontally for each layer?

@Zehvogel
Copy link
Contributor Author

Is it possible to color the boxes of each layer in a different color? Or are they guaranteed to align horizontally for each layer?

It is possible to color the nodes. It is plain graphviz so any node attribute should work.
Layers are not guaranteed to align. They do align when starting from the deepest common element, which is probably not what you are after... The default layout engine is dot. It might be possible to tune that but I do not care enough at the moment :)

Here another example of a detector with a bit more consistent structure where it is better visible what aligns when.

OpenDataDetector_OpenDataTracker_Pixels.gv.pdf

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

Successfully merging this pull request may close these issues.

3 participants