Skip to content

ADD JSON_PARSE #106

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

Open
GillesDuvert opened this issue Apr 8, 2018 · 2 comments
Open

ADD JSON_PARSE #106

GillesDuvert opened this issue Apr 8, 2018 · 2 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed priority!

Comments

@GillesDuvert
Copy link
Contributor

json_serialize.pro is already written, in src/pro
We need json_parse, ideally with both the /toarray and /tostruct options implemented.
This can be done in IDL language for a start, for a first issue.
But of course a more expert c++ approach using one of the C++ JSON libraries would be a must!

@GillesDuvert GillesDuvert added help wanted Extra attention is needed good first issue Good for newcomers idl/gdl-only No C++ skills needed labels Apr 8, 2018
@alaingdl
Copy link
Contributor

What is the status of that ?

@GillesDuvert GillesDuvert removed the idl/gdl-only No C++ skills needed label Jan 17, 2022
@GillesDuvert
Copy link
Contributor Author

In retrospect, pure IDL/GDL may be uneffective. The demand we had was on huge (>100Mo) JSONs files, so using a c++ json parser is mandatory.

https://json.nlohmann.me/ is the thing we need. I've made some tests. We can exchange externally if you wish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed priority!
Projects
None yet
Development

No branches or pull requests

2 participants