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 configuration file for the podio visualization tool #193

Merged
merged 1 commit into from
May 2, 2023

Conversation

jmcarcell
Copy link
Member

@jmcarcell jmcarcell commented Feb 21, 2023

BEGINRELEASENOTES

  • Add a configuration file for the new podio visualization tool

ENDRELEASENOTES

To reproduce the last diagram shown in AIDASoft/podio#377 (i.e. with clustering the data types into categories) it is necessary to have a configuration file that defines which groups exists. While it can be in either place I believe it should be here so changes in the data types such as renames are more likely to be applied also to this configuration file.

As a reminder, to get the diagram run (once it's merged and installed) python path_to_podio/tools/podio-vis path_to_EDM4hep/edm4hep.yaml --graph-conf path_to_EDM4hep/utils/graph-conf.yaml

We can also agree on which Associations to remove to make the plot nicer which can be removed by adding

Filter:
  - datatype

@jmcarcell jmcarcell added enhancement New feature or request discussion Discussion item labels Feb 21, 2023
@jmcarcell jmcarcell changed the title Add a configuration file for the podio Add a configuration file for the podio visualization tool Feb 28, 2023
@jmcarcell
Copy link
Member Author

This is how it looks like with the current configuration file without removing any association

diagram

@jmcarcell jmcarcell merged commit 21b6ab9 into key4hep:master May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Discussion item enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant