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

Point out alternative #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PallHaraldsson
Copy link

[skip ci]

@PallHaraldsson
Copy link
Author

PallHaraldsson commented Nov 22, 2021

Hi, I really liked/like your work, and this lazy idea, that I believe I may have heard of it first related to your package. I'm not disrespecting your work. I just believe JSON3.jl is well tested by now, and know it came later, and your and JSON.jl are pre-1.0 (I made similar suggestion there).

Is you fully supported, and better in some way?

https://discourse.julialang.org/t/announce-a-different-way-to-read-json-data-lazyjson-jl/9046

@quinnj, FYI (not a huge concern, I doubt many would use both at once): In my 1-day master, JSON3 loads a tiny bit faster, and combined almost as fast if loaded in reverse order, but with this order loading your first penalizes JSON3 a lot:

julia> @time using LazyJSON
  0.155977 seconds (167.09 k allocations: 16.904 MiB, 15.58% compilation time)

julia> @time using JSON3
  1.986343 seconds (3.24 M allocations: 176.131 MiB, 2.88% gc time, 99.03% compilation time)

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 this pull request may close these issues.

1 participant