Raise warning on statically-linked Python binaries #237
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Have you ever tried a version that uses JuliaCall / PythonCall instead of pyjulia / PyCall? |
|
I haven't yet, no. But I suppose it would be easy to try? The only tricky parts would be changing the project management/installation, as well as some of the lower-level PyJulia calls like |
|
Edit: posting this instead in the issue #238 rather than here. |
|
Added in 49ecfff |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Time-to-first-search is very slow on statically-linked versions of Python (such as packaged with conda), as precompiled code cannot be used, so things are compiled from scratch. I think this adds some friction to the user experience, so this PR introduces a warning that recommends the user try
pyenvif startup time is important.When JuliaPy/pyjulia#496 is solved, this warning is no longer needed.
See conda-forge/python-feedstock#222 for the discussion on the conda page.