freezing-photographer-88553
02/14/2022, 3:27 PMpants tests
. Unfortunately, these binaries are not detected.
Any idea of how to solve this issue ?enough-analyst-54434
02/14/2022, 3:32 PMhundreds-father-404
02/14/2022, 3:39 PM$PATH
?enough-analyst-54434
02/14/2022, 3:41 PMconda install
that is most definitely != pip install
/ doesn't use PyPI.hundreds-father-404
02/14/2022, 3:44 PMenough-analyst-54434
02/14/2022, 3:47 PMLD_LIBRARY_PATH
territory)freezing-photographer-88553
02/14/2022, 4:05 PM./anaconda3/envs/PantsBuild/lib/libspatialindex_c.dylib
geopandas
. This pkg makes calls to libspatialindex
which is a C++ library.
To setup the project with the geopandas dependency I have 2 options:
Option 1:
apt-get install gdal libspatialindex rtree ...
pip install geopandas
Option 2:
conda install geopandas rtree -c conda-forge
The second option is way better because: 1) these libs are installed in a virtual env 2) the versions are resolved across python and C++ libraries 3) it is platform agnostic (same command for MacOs and Linux)enough-analyst-54434
02/14/2022, 4:29 PMfreezing-photographer-88553
02/14/2022, 4:32 PMhundreds-father-404
02/14/2022, 4:57 PMIs it possible to add a command to the script that runs the test ?We do have a plugin hook to set up tests before Pytest executes, like starting databases. But I'm not sure that would help: 1) It wouldn't help with other goals like
package
and run
, which will still be missing the deps
2) I'm not confident we could get that plugin hook to wire things up properly into the test runner
Option 1: apt-get install gdal libspatialindex rtree ...@enough-analyst-54434 any reason you think this would not work here? That's how users normally get native things like databases to build: make sure you have the correct non-Python deps installed and exposed to Pants FYI @freezing-photographer-88553, when Pex installs something, it is using pip. So if you can get
pip install
to work, it should be possible for Pants to install things. (Ack that you prefer to keep using conda)enough-analyst-54434
02/14/2022, 4:57 PMLD_LIBRARY_PATH
but I expect this will be tricky and brittle.hundreds-father-404
02/14/2022, 5:00 PMapt-get
. They prefer option 2 working, but if we can't get it to, then at least we can do option 1enough-analyst-54434
02/14/2022, 5:00 PMfreezing-photographer-88553
02/14/2022, 5:01 PMenough-analyst-54434
02/14/2022, 5:01 PMfreezing-photographer-88553
02/14/2022, 5:02 PMenough-analyst-54434
02/14/2022, 5:02 PMfreezing-photographer-88553
02/14/2022, 5:02 PMenough-analyst-54434
02/14/2022, 5:02 PMfreezing-photographer-88553
02/14/2022, 5:03 PMenough-analyst-54434
02/14/2022, 5:03 PMfreezing-photographer-88553
02/14/2022, 5:05 PMenough-analyst-54434
02/14/2022, 5:05 PMfreezing-photographer-88553
02/14/2022, 5:05 PMenough-analyst-54434
02/14/2022, 5:07 PMhappy-kitchen-89482
02/14/2022, 5:18 PMbusy-vase-39202
02/14/2022, 5:28 PM