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

Streamline reading of json and yaml files #213

Closed
domna opened this issue Jan 17, 2024 · 0 comments · Fixed by #250
Closed

Streamline reading of json and yaml files #213

domna opened this issue Jan 17, 2024 · 0 comments · Fixed by #250

Comments

@domna
Copy link
Collaborator

domna commented Jan 17, 2024

Many of our readers read json and yaml files to add additional metadata.
Some of these features are made generally available in the pynxtools.utils but reader implementations differ.

  1. I think we should streamline yaml and json reading and offer this as a general solution for adding additional keys to an appdef. There is a generic YamlJsonReader already going in that direction, but it is just used by few readers.
  2. I think this would also be useful for plugin developers as they can concentrate on writing the actual data conversion and just can use the general handling of additional metadata through pynxtools.
@domna domna mentioned this issue Feb 21, 2024
2 tasks
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 a pull request may close this issue.

1 participant