-
Notifications
You must be signed in to change notification settings - Fork 25
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
Latency increase from 1.10 to 1.11 #274
Comments
This is exactly why I proposed this JuliaLang/julia#48819. This is not sustainable if we require downstream developers to change their implementation every time something is changed upstream. I'm not sure which downstream dependency has latency changes. Comonicon only has 4 direct non-stdlib dependencies, which is why it doesn't matter that much before 1.11. Again if we require downstream packages with only 4 direct dependencies (now perhaps 5 or 6) to update its code, this is a very big red flag in maintaining the ecosystem. I'd rather just switch to rust. |
OK, so it turns out the current solution should be moving the Docs.doc query implementation out of the REPL package. |
If JuliaLang/julia#54499 indeed lands and is backported to 1.11, then REPL might not be needed, and can be removed from the dependencies (REPL will be available in versions of Julia before 1.11). Then this issue would be solved. |
I've been trying to investigate this from JuliaLang/julia#55171 but cannot build the reproducer. Can anyone help over there? |
Bump. I'm interested in investigating this if someone can help |
Sorry I got Covid after JuliaCon. I replied in that issue, please let me know if it works for you. |
A small CLI tool I created had its latency go from 0.9 s to 2.7 s when going from Julia 1.10 to 1.11. This might be due to REPL and Pkg being moved out of the system image.
In total, Comonicon brings in 40 transitive dependencies. That seems like a lot. Perhaps an approach to reducing Comonicon's latency is to cut dependencies.
The text was updated successfully, but these errors were encountered: