conda-forge issues about geowombat HOT 13 CLOSED

@jgrss Ok coming back to this since your push of 2.1.8 didn't resolve the issue.
Focusing on this error for now:

nothing provides freetype 2.5.* needed by graphviz-2.38.0-0

This is all very odd, it seems like graphviz maybe has multple wrappers for python. Oddly the equivalent of pip install graphviz is conda install python-graphviz. Maybe its to keep conda for install the java (or whatever version).

The thing that is warping my mind a bit is that mamba create -n test python=3.9 dask python-graphviz -c conda-forge installs graphviz 8.0.3, and pip install graphviz installs graphviz 0.20.1.

My assumption is that graphviz 8.0.3 in conda is equivalent to 0.20.1 in pip.

Since I need to focus on conda issues:

Its very odd the graphviz 2.38 is getting called here. Dask is currently using graphviz 8.03. I did notice that graphviz version updates aren't getting pushed to conda (or elsewhere??) https://anaconda.org/conda-forge/graphviz

The only easy solution I can think of here is to see if we can ensure dask installs the required graphviz components. For instance, is the following also installing the optional dependencies??

dask[array,dataframe]>=2023.1.0

from geowombat.

@jgrss I am trying to move everything over to conda-forge and thus need to become the maintainer of our build of sklearn-xarray, they are requesting that you add a comment to the following PR that you are OK with being a maintainer (I will do the work, but want you to have a say too).

conda-forge/staged-recipes#22655

from geowombat.

Where do you pin the conda Python requirement?

from geowombat.

it all seems to be pulled in from your module requirements (I think). The only thing that we needed to get together for conda-forge was this recipe

from geowombat.

Ok looks like something got pushed to conda-forge last night. Builds are now failing on linux, and osx. For example here

Looks like it can't find freetype 2.5.* needed by graphviz-2.38.0-0

Assuming we need to add graphviz to requirements, but I thought that came with dask.

from geowombat.

@mmann1123 I think graphviz does get installed via one of the other dependencies. The error looks to be coming from graphviz itself. See here.

graphviz-2.38.0-0 is conda's win-32 install. The Python graphviz is 0.20.1.

from geowombat.

@mmann1123 Couple of success comments: conda/conda#1851 (comment) and conda/conda#1851 (comment)

from geowombat.

This is a really old issue. Not sure why it would be breaking now. I will rerun the build tests, maybe graphviz broke something on their recipe? If not I'll open an issue with graphviz.

from geowombat.

Interestingly the graphviz error resolved for linux NOT osx, and we replaced with this

- package proj-9.1.1-h93bde94_0 requires libtiff >=4.4.0,<4.5.0a0, but none of the providers can be installed

OSX still has the graphviz error.

from geowombat.

Where do we specify the Python version range for the conda recipe? It's pinning the upper limit to v3.9.0a0. Not sure if that's related, though.

Also, as you probably saw, there's a conflict with proj and libtiff. However, it looks like the appropriate version of libtiff should be installed.

from geowombat.

I have been making some progress (started fresh)
conda-forge/geowombat-feedstock#89

Plus adding all the unit tests i can for the moment.

from geowombat.

@jgrss One issue that has come up is the use of with tempfile.TemporaryDirectory() for the windows builds for conda. Seems like it might not be compatable on windows. This come up with at least test_write and test_nodata. See error here: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=693951&view=logs&j=a70f640f-cc53-5cd3-6cdc-236a1aa90802&t=f5d15007-a01c-5ad8-c9ce-4d519d3b275f&l=1119

But its some kind of permission error

PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\VSSADM~1\\AppData\\Local\\Temp\\tmparp76n5r\\test.tif'

..\_test_env\lib\shutil.py:616: PermissionError

from geowombat.

This issue is resolved.

from geowombat.