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 a graph visualization #24

Open
c-dilks opened this issue Feb 27, 2023 · 3 comments
Open

Add a graph visualization #24

c-dilks opened this issue Feb 27, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@c-dilks
Copy link
Member

c-dilks commented Feb 27, 2023

Is your feature request related to a problem? Please describe.
Using AIDASoft/podio#377 and following along with key4hep/EDM4hep#193

Describe the solution you'd like
A graph to show all datatypes, relations, and associations; this would be especially helpful for new users.

Describe alternatives you've considered
No graph, since the YAML file already tells everything.

@c-dilks c-dilks added the enhancement New feature or request label Feb 27, 2023
@wdconinc
Copy link
Contributor

Do we not have a graph in the doxygen output? But yes, this would be nice!

@simonge
Copy link
Contributor

simonge commented Oct 24, 2024

Has this been created? @wdconinc contribution to podio in March (AIDASoft/podio@8ce5945) certainly allowed but is a more up to date version than installed in eic_shell.

With a local install of podio, the output of:

./podio-vis /opt/software/linux-debian12-x86_64_v2/gcc-12.2.0/edm4eic-7.0.0-gyx3u4q7mg7tfrxbme5nyopq3do6qqgo/share/edm4eic/edm4eic.yaml --upstream-edm edm4hep:/opt/software/linux-debian12-x86_64_v2/gcc-12.2.0/edm4hep-0.10.5-mafwdcsoggwoizgbquwc5oxlujqfl5om/
share/edm4hep/edm4hep.yaml 

image

While the extension of the datamodel from edm4hep components gets plotted correctly it doesn't capture the relationships between the components it's extended from i.e. OneToOneRelations between SimHits and MCParticles.

There are certainly aesthetic improvements which could be made with a graph-conf file.

@wdconinc
Copy link
Contributor

The PR above is in podio@1, but I am not sure if we are ready to upgrade from [email protected] to that yet. Let me see what is possible with some testing.

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

No branches or pull requests

3 participants