RHEOS.jl icon indicating copy to clipboard operation
RHEOS.jl copied to clipboard

Installation output showing in the newly built docs

Open akabla opened this issue 11 months ago • 6 comments

          BTW, I'm not sure why but there's some installation output showing in the newly built docs that wasn't there before:

https://juliarheology.github.io/RHEOS.jl/dev/examples/

Originally posted by @moustachio-belvedere in https://github.com/JuliaRheology/RHEOS.jl/issues/161#issuecomment-1975304989

akabla avatar Mar 04 '24 06:03 akabla

I tried to stop this output with ";", but it doesn't help here.

akabla avatar Mar 04 '24 06:03 akabla

I found updating the docs/ project environment to remove Conda and PyCall (possible now due to your removal of explicit PyCall usage) and upgrading didn't produce the output locally which is promising.

moustachio-belvedere avatar Mar 05 '24 23:03 moustachio-belvedere

If this doesn't help on the remote build I think we can get the CI pipeline to force an early PyPlot installation before the doc build gets triggered

moustachio-belvedere avatar Mar 05 '24 23:03 moustachio-belvedere

Actually, adding this near the top of docs/make.jl might be better than changing CI:

using Pkg
ENV["PYTHON"]=""
Pkg.build("PyCall")
using PyCall
pyimport_conda("matplotlib.pyplot", "matplotlib")

moustachio-belvedere avatar Mar 05 '24 23:03 moustachio-belvedere

Actually, adding this near the top of docs/make.jl might be better than changing CI:

using Pkg
ENV["PYTHON"]=""
Pkg.build("PyCall")
using PyCall
pyimport_conda("matplotlib.pyplot", "matplotlib")

Thanks Louis, that fixed it!

We might as well simplify the environment and remove Conda and Pycall. I leave this open for now.

akabla avatar Mar 06 '24 07:03 akabla

Sounds good. Though I think with the above fix we'll have to leave PyCall in as an explicit dependency. (Conda will still be a dependency but can prob be relegated to an implicit one.)

moustachio-belvedere avatar Mar 09 '24 18:03 moustachio-belvedere