"Python exited unexpectedly" when using external conda environment

[charstnut]

Thank you for this great package!

I've come across this warning after configuring PythonCall to use the PyCall environment. It does not seem to interfere with usage though. When using "Ctrl + D" for exit this results in segmentation fault but still exits.

MWE after starting julia:

julia > ENV["JULIA_PYTHONCALL_EXE"] = "@PyCall"
julia > using PythonCall
julia> exit()
┌ Warning: Python exited unexpectedly
└ @ PythonCall.C ~/.julia/packages/PythonCall/zJpsh/src/cpython/context.jl:179

[cjdoris]

Thanks. The issue is that both PythonCall and PyCall act like they "own" the Python interpreter instance, which is a bug.

I'm rewriting the initialisation part of PythonCall soon so I'll make sure this bug is fixed as part of that.

[cjdoris]

I believe this is fixed now on the master branch. Can you check please?

[charstnut]

I checked from the python side and it works without any problem.

(Just a note, it is probably necessary to run Pkg.resolve() inside the julia_env folder in the custom conda environment after reinstalling the python package if the dependencies change.)

For the julia side, it still exits with this warning. I assume updating the package using Pkg.up() would be sufficient?

               _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.7.0 (2021-11-30)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

julia> using PythonCall

julia> exit()
┌ Warning: Python exited unexpectedly
└ @ PythonCall.C ~/.julia/packages/PythonCall/zJpsh/src/cpython/context.jl:179

[cjdoris]

Aha I missed a case, I can make that warning go away. Thanks for checking.

Yeah the resolve thing you mentioned only affects people running the development version.

[cjdoris]

Pretty sure this is correct on main now. Let me know if not.

[charstnut]

Thanks! Confirmed this is fixed.