Open dataspider opened 3 weeks ago
Also, please add pytest to the dev dependencies in your pyproject.toml and be at least one level more specific than "*" for pinning dependencies.
Thank you for the feedback. I had trouble recreating your issue, but I am aware that gdal can be a difficult dependency to install. I therefore added some information on the installation guide about this potential requirement (depending on the user’s coding environment). Please let me know if this is insufficient and I can work on additional documentation. I also added the most recent versions of the dependencies and added pytest as a development dependency.
Thank you for the updates; I would probably still add the conda workaround I sketched above somewhere in the docs (I'd keep it simple in the README and point to a more elaborate Installation
section in your online docs for trouble-shooting).
Also, there is an "it's/its" typo in the installation instructions now that you may want to fix :)
Sounds good, I will add your more specific workaround (and other possible solutions) to the docs website and keep it simple in the README. Thanks for the typo find too
You seem to have a "hidden" dependency on gdal that can cause installation problems.
Below is the error I got when running
pip install SVInsight
on a MacBook Pro 2018 with macOS Ventura 13.4 in a regular virtual environment (as, e.g., PyCharm suggests based on the setup files when one opens the project locally). Using a conda environment and installing geopandas first fixed the problem without the need to install gdal via brew. Please either fix the dependency issue such thatpip install SVInsight
works in the "standard" virtual environments or document the dependency on gdal and suggest a workaround (e.g., the one I described above) for those who don't have gdal installed (one of gdal's dependencies is llvm, so this is not a light install). You might also want to consider distributing the package via conda-forge, which would allow you to handle the CPP dependencies already at build time.https://github.com/openjournals/joss-reviews/issues/7212