-
-
Notifications
You must be signed in to change notification settings - Fork 321
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 JSON and YAML C library dependency #3020
Comments
Adding here that the preferred JSON encoding for |
@cwhite911 could this be closed? |
@nilason I'm still using this issue to track all the work getting done implementing json outputs. I'm happy to close this issue and move this somewhere else if you have a good suggestion. |
The original issue was solved. Let's create a new issue or I can create a project. |
A project would be nice to keep track of this. |
Hey @cwhite911 @wenzeslaus , has JSON support been added to all the required modules, or are there still some modules left that need JSON support? If so, please let me know—I would love to contribute in this area! |
Project to track it created a couple days ago: https://github.com/orgs/OSGeo/projects/10 |
@NishantBansal2003 any of the above tools that do not have an associated PR would be good to include. I also have some schema improvements in mind for some of the existing json enabled modules that I can make issues for. |
Sure, I’ll look into this. Meanwhile, could you create issues for some schema improvements in the existing JSON-enabled modules? I’ll review them and work on the fixes. Thanks |
I don't have the analysis for the schema. I wish. From my perspective, that's up for grabs if you feel like it. I noticed that r.univar always outputs a list of objects, but for one input, it seems to me it should output a single object. That's a one clear issue I noticed. |
From a data parsing point of view, as json would be targeted around tool usage, it would be a nightmare to use if the schema/outputted format changed shape depending on the number of inputs. If it can be one or more, the. It should always be a list, and even empty if there is nothing. |
Hi @wenzeslaus and @cwhite911! It looks like Would you prefer that I open an issue for discussion first, or should I go ahead and create a PR? |
Sounds like discussing it in an issue might be better. |
Discussed in #3019
Originally posted by cwhite911 June 5, 2023
In an effort add additional output data formats to modules (e.g., r.report, r.info, etc..) we are discussing adding new library dependencies to aid in JSON and YAML encoding.
Please share your feedback on the following candidate dependencies.
JSON
parson
https://github.com/kgabis/parson/
Current release: 1.5.2 (2023)
Parson is a lightweight json library written in C.
YAML
libyaml
https://github.com/yaml/libyaml
Current release: 1.2.2 (2021-10-01)
LibYAML is a YAML parser and emitter library.
Related Issues and PRs
Candidate Modules
Which modules should have JSON & YAML output support?
General
Raster
3d Raster
Vector
Database
JSON output support without a library
The text was updated successfully, but these errors were encountered: