search

ShobiStassen / VIA

trajectory inference
https://pyvia.readthedocs.io/en/latest/
MIT License
78 stars 20 forks source link

Problems installing package #10

Closed GreenGilad closed 2 years ago

GreenGilad commented 2 years ago

Hi there,

I have been trying to install the package on both windows (11) computer and a mac computer. In both cases once the library is installed it fails to import because palantir is not installed (which itself fails in installation on MulticoreTSNE).

Why is palantir required for VIA?

>>> import pyVIA.core as via
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".....................ViaEnv\lib\site-packages\pyVIA\__init__.py", line 1, in <module>
    from . import core
  File "................\ViaEnv\lib\site-packages\pyVIA\core.py", line 27, in <module>
    import palantir # /home/shobi/anaconda3/envs/ViaEnv/lib/python3.7/site-packages/palantir
ModuleNotFoundError: No module named 'palantir'
GreenGilad commented 2 years ago

After fixing the cmake+palantir installations (by downgrading cmake: pip install cmake==3.18.4) I managed to import. However, is palantir installation actually required for VIA? and if perhaps it should be added to the instructions.

Thanks

ShobiStassen commented 2 years ago

hi, sorry for the issues you had during installation. Palantir is not required for VIA, but is included in the script for some testing/benchmarking functions I was testing (it seems to have made its way back in the code). I will change this later today so that there is no palantir import. Thanks for raising this.

ShobiStassen commented 2 years ago

Hi, the installation should no longer require palantir, thanks for bringing this up. (version pyVIA==0.1.23)

GreenGilad commented 2 years ago

Perfect! Thanks for the quick fix