search

stephankramer / uptide

python package for tidal calculations
GNU Lesser General Public License v3.0
44 stars 8 forks source link

The fes installation instruction is broken. #7

Closed local-minimum closed 5 years ago

local-minimum commented 5 years ago

The instruction

pip install git+https://bitbucket.org/cnes_aviso/fes.git#subdirectory=python

is no longer valid since fes seems to have dropped the general python installation support and requires manually compiling the package:

wget https://bitbucket.org/cnes_aviso/fes/downloads/fes-2.9.1-Source.tar.xz \ 
    ​    && tar -xJf fes-2.9.1-Source.tar.xz \
    ​    && cd fes-2.9.1-Source \
    ​    && mkdir build \
    ​    && cd build \
    ​    && cmake .. -DBUILD_PYTHON=on -DPYTHON_EXECUTABLE=/usr/bin/python3 -DBUILD_SHARED_LIBS=on -DCMAKE_INSTALL_PREFIX=/usr \
    ​    && make -j \
    ​    && make install

Which is really uncomfortable unfortunately.

stephankramer commented 5 years ago

Thanks. It appears that not only the python installation support via setup.py/pip has been dropped, but also the API has changed. I have now fixed this in the latest release, and updated the installation instructions for fes in the README.md. Please let me know if you have any further issues.