-
Notifications
You must be signed in to change notification settings - Fork 262
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 more JSON libaries #9
Comments
sajson was added in https://github.com/miloyip/nativejson-benchmark/tree/sajson branch but it is unable to parse twitter.json. Reported as chadaustin/sajson#7. Confirmed it is fixed. |
JSON Voorhees fails to compile on 32-bit target. Reported as tgockel/json-voorhees#53. |
ArduinoJson was added in https://github.com/miloyip/nativejson-benchmark/tree/ArduinoJson branch. But it cannot parse properly due to stack overflow. Reported as bblanchon/ArduinoJson#65. Confirmed it is fixed. |
nbsdx/SimpleJSON was added but there seems no way to iterate object and array. Reported as nbsdx/SimpleJSON#1. Update:The functions were added. And statistics has been completed. |
jbson seems not working widely:
Thus not add it at this moment. |
ArduinoJson crashes when printing big number in conformance test. Reported as bblanchon/ArduinoJson#67. |
@miloyip as a notice the Nlohmann/json has released v1.0.0, 16 days ago and would be worth to update. (Crossref: nlohmann/json#177) |
@miloyip hello, I want to find a pure C, easy to use and intergrated json library, I found these:
maybe these can be added into benchmark, I found libraries here, Thanks for these good results :-) |
Homepage: http://zserge.com/jsmn.html FEATURES
|
@miloyip, thanks for updating nlohmann/json to 1.0.0. Is there a way to have a look at the new results? |
@nlohmann I will rerun the benchmarks after adding a few more libraries. You may also run it locally as well. |
@miloyip is there no way to automate running the benchmarks with continues integration. E.g Travis CI ? |
@xor-gate Ideally of course that will be the best. |
You are right about that, I have overseen this. But a colleague tried to run the tests but he could not get it to work. Maybe a alternative would be to have the test suite executable automatically build under Docker. With a Dockerfile supplied, e.g Debian Jessie as base image then people can easy reproduce binaries and run on real metal. |
I understand there is difficult to build and run the tests. I faced these problems often as well. |
@miloyip I will open a separate issue for this so we when I need some help we can discus there. I will give it a shot this weekend. |
@starwing json.h done |
@starwing There are some issues when integrating MMX vurtun/lib#3 |
@starwing Sigh... the author of mmx has removed the mm_json.h. I think I have not provoked him anyway vurtun/lib#3 |
Hey @miloyip, today I released version 1.1.0 of JSON for Modern C++. It would be great if you could update it in your benchmark. This release also fixes some bugs that have been detected with your floating-point tests. :-) |
@nlohmann It's done. BTW, I just added a FAQ for updating libraries. Welcome for updating it via pull request. Good to know this test has helped improving your library 😄 |
@miloyip so sad things :( But still very thank you for the work!! |
@miloyip do you think FlatBuffers may have a place in your benchmark ? |
In my opinion, this is a different format (just like MessagePack, Protocol Buffers, BSON) and should not be added to the nativejson-benchmark. As the name contains ... json. |
@xor-gate FlatBuffers can Parse JSON so maybe in some part of the benchmark this lib may be compared to the others |
That make sense, I was not aware of that. |
Please add js0n. |
Please add zzzJSON |
Above released in v1.0.0
jbson (C++14 & boost)mm_json.h (C)JSON root can be other types vurtun/lib#3The text was updated successfully, but these errors were encountered: