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.

Sign up for GitHub

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

Jump to bottom

Doc build failing because of PyCall issue #2

Closed
cossio opened this issue Dec 6, 2021 · 5 comments
Closed

Doc build failing because of PyCall issue #2

cossio opened this issue Dec 6, 2021 · 5 comments

Comments

@cossio
Copy link
Owner

cossio commented Dec 6, 2021

image
@cossio
Copy link
Owner Author

cossio commented Dec 6, 2021

This should eventually be fixed by JuliaML/MLDatasets.jl#81, when a new version of MLDatasets is released.

@cossio cossio closed this as completed in ede25cb Dec 6, 2021
@cossio
Copy link
Owner Author

cossio commented Dec 6, 2021

Setting the PYTHON environment variable to the empty string "" forces PyCall.jl to use its own Python distro (via Conda.jl). See https://github.com/JuliaPy/PyCall.jl.

cossio pushed a commit that referenced this issue Dec 6, 2021
docs build: Make PyCall manage its own Python distro
4f4c62d 
Fixes #2
@cossio cossio reopened this Dec 6, 2021
@cossio
Copy link
Owner Author

cossio commented Dec 6, 2021 •
edited
Loading

Closed by 4f4c62d
Now for real.
We need to set PYTHON = "" before PyCall is built.

@cossio cossio closed this as completed Dec 6, 2021
@cossio cossio reopened this Dec 6, 2021
@cossio
Copy link
Owner Author

cossio commented Dec 6, 2021

Trying again ... 0b9b466

@cossio
Copy link
Owner Author

cossio commented Dec 7, 2021 •
edited
Loading

It was 1b09f72 that finally did the trick. We need also the environment variable DATADEPS_ALWAYS_ACCEPT set to true, in addition to PYTHON = "". See https://www.oxinabox.net/DataDeps.jl/stable/z10-for-end-users/#Configuration-1.

Also MLDatasets now tagged a release which drops the PyCall dependency, so PYTHON = "" should be no longer needed.

Closing.

@cossio cossio closed this as completed Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cossio