Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve podio dump #427

Merged
merged 6 commits into from
Jun 15, 2023
Merged

Improve podio dump #427

merged 6 commits into from
Jun 15, 2023

Conversation

tmadlener
Copy link
Collaborator

BEGINRELEASENOTES

  • Delay library loading as long as possible, mainly for quicker responses for --help
  • Add a --version flag for dumping the podio version
  • Display collections and parameters in alphabetical order and automatically adjust column widths to fit contents (using the tabulate package).

ENDRELEASENOTES

Fixes #352

@tmadlener
Copy link
Collaborator Author

Key4hep workflows are failing because they do not yet provide the tabulate package.

@hegner hegner self-requested a review June 15, 2023 07:46
@hegner
Copy link
Collaborator

hegner commented Jun 15, 2023

@tmadlener - is there a corresponding PR for the addition of tabulate to the stack?

@tmadlener
Copy link
Collaborator Author

Not yet, I would add it as a run (and test) dependency to the podio spack package once this is merged, then the stack should pick it up automatically.

@jmcarcell
Copy link
Member

Unfortunately this doesn't work in the nightlies (the fix is to install tabulate) since spack hasn't been updated to pick up the change and it always uses master

@tmadlener tmadlener deleted the improve-podio-dump branch June 19, 2023 15:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

podio-dump output of overview table can be improved
3 participants