votca / votca

The source of the votca-csg and xtp packages

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Conda installation for pyxtp

NicoRenaud opened this issue · comments

Hi @junghans,

together with @baumeier and @suvayu we are trying to finish the development of the python interface for xtp. Most users of the python library would most likely want to install it in a dedicated conda env with all the dependencies they need.

With the current build system, I think users would have to specify the PYTHON_INSTDIR to point to their current conda environment. One solution I could find to allow the installation in the active conda env was to change the the python prefix in the pyxtp Cmake : NicoRenaud@7cffc5a

However I am by no means a CMAKE person and there might be a better solution.

Would you have any idea regarding the install of pyxtp in a conda env ?

Thanks

Hmm, I am not sure about it.

Wouldn't the right approach be to use -DCMAKE_INSTALL_PREFIX=<your conda env prefix>. Otherwise parts of VOTCA will be installed in the conda env and the rest in CMAKE_INSTALL_PREFIX, which seems bad.

For packaging we need the following use cases:

As a side note cg.get_python_lib() is obsolete so we will need to find another solution for that one, see pdoc3/pdoc#392

Maybe @jan-janssen can help here, I am not very familiar with setup.py and all these things.

Maybe @jan-janssen can help here, I am not very familiar with setup.py and all these things.

I am a bit confused here. If pyxtp is a python package which can be installed using pip install . then I would assume that it looks for the corresponding libraries in the LIBPATH. So if votac was installed via conda then the corresponding library should already be in the corresponding LIBPATH. The main focus on the conda side is that we want to build the library once and then link to it from all the packages that depend on it. In this case it might be helpful to release the xtp library as standalone package without requiring a full votac installation. Still I guess all this would be much easier to understand if someone could point me to the pyxtp code.

@NicoRenaud is this still an issue?

Not anymore ! I'll close the issue