Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

geowombat v1.11.4 breaks conda #188

Closed
mmann1123 opened this issue Aug 31, 2022 · 12 comments
Closed

geowombat v1.11.4 breaks conda #188

mmann1123 opened this issue Aug 31, 2022 · 12 comments
Assignees
Labels
dependencies Pull requests that update a dependency file

Comments

@mmann1123
Copy link
Collaborator

Just got this report back from the conda feedstock or for details go here

Assuming pinning the xarray version pulls the older rasterio - breaking things downstream? What do you think @jgrss

The reported errors are:
- Encountered problems while solving:
-   - package rasterio-1.0.23-py27h5b3f9e8_0 requires libgdal >=2.4.1,<2.5.0a0, but none of the providers can be installed
-   - package pyproj-1.9.4-py27_0 requires python 2.7*, but none of the providers can be installed
@mmann1123 mmann1123 added the dependencies Pull requests that update a dependency file label Aug 31, 2022
@jgrss
Copy link
Owner

jgrss commented Aug 31, 2022

Hmm, geowombat==1.11.4 pulls a higher version of rasterio than v1.0.23. I just did a fresh installation from main and it installed rasterio==1.3.2. Is conda free to build any version of rasterio in the build? I don't see a version requirement for it.

@mmann1123
Copy link
Collaborator Author

Yeah I don't pin any versions in the feedstock and it should be able to pull whatever solves all requirements. So not sure... Instead of pinning xarray could we just implement the np.max min as suggested in #186? (although that seemed to open other issues possibly)

@jgrss
Copy link
Owner

jgrss commented Sep 1, 2022

Can you see if this is still an issue after #189?

@mmann1123
Copy link
Collaborator Author

Ok looks like it builds 2.0 fine on everything but python 3.7. regardless of OS. Should we pin python >=3.8

Looks like opencv is the issue.

    • nothing provides libopencv 4.2.0 py36_5 needed

@mmann1123
Copy link
Collaborator Author

@jgrss I think finally step is to pin python 3.8 or greater

@jgrss
Copy link
Owner

jgrss commented Sep 14, 2022

See #200

@jgrss
Copy link
Owner

jgrss commented Sep 15, 2022

#200 is merged into main so let me know how the conda build goes.

@mmann1123
Copy link
Collaborator Author

After much futzing I figured out that adding the skip line to meta.yml in the feedstock would allow us to avoid building in py3.7. Should be merging soon.

build:
  number: 0
  skip: true  # [py<38]
  script: {{ PYTHON }} setup.py build && pip install .  

@mmann1123
Copy link
Collaborator Author

Resolved! @jgrss new conda build should be available in a few hours

@mmann1123
Copy link
Collaborator Author

mmann1123 commented Oct 11, 2022 via email

@jgrss
Copy link
Owner

jgrss commented Oct 11, 2022

This is closed from ~month ago. Is this correct?

@mmann1123
Copy link
Collaborator Author

mmann1123 commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

2 participants