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

no defaults channel in environment.yaml #144

Merged
merged 7 commits into from
Dec 2, 2020
Merged

Conversation

fneum
Copy link
Member

@fneum fneum commented Mar 17, 2020

Fixes #142
Let's see...

@coroa
Copy link
Member

coroa commented Mar 17, 2020

Channel pinning should be superseeded by setting channel priority to strict as explained here: https://conda-forge.org/docs/user/tipsandtricks.html

Apart from that, it seems descartes is pulled in too late by pip and should probably be uplifted to a conda dependency instead.

@fneum
Copy link
Member Author

fneum commented Mar 18, 2020

Pity there's not yet a way to request strict channel priority in the environment.yaml: conda/conda#8675

conda config --set channel_priority strict

@coroa
Copy link
Member

coroa commented Mar 18, 2020

Failure on mac:

ImportError: dlopen(/Users/travis/miniconda/envs/pypsa-eur/lib/python3.7/site-packages/fiona/ogrext.cpython-37m-darwin.so, 2): Library not loaded: @rpath/libwebp.7.dylib

  Referenced from: /Users/travis/miniconda/envs/pypsa-eur/lib/libtiff.5.dylib

  Reason: Incompatible library version: libtiff.5.dylib requires version 9.0.0 or later, but libwebp.7.dylib provides version 8.0.0

Both are from conda-forge:

libtiff                   4.1.0                h2ae36a8_5    conda-forge
libwebp                   1.0.2                hd3bf737_5    conda-forge
libwebp-base              1.1.0                         2    conda-forge

Should be missing deps in the libtiff or libwebp conda recipes.

Found conda-forge/libtiff-feedstock#56 and conda-forge/libwebp-feedstock#26 so seems to be a known active problem. We maybe could constrain libwebp down a bit, but this will be a pain to find out, if we can only test on the travis mac runners.

@fneum fneum marked this pull request as ready for review October 2, 2020 11:04
@fneum fneum requested a review from FabianHofmann October 2, 2020 13:57
@fneum
Copy link
Member Author

fneum commented Nov 26, 2020

@FabianHofmann everything seems to work with/without defaults channel. I'm on the fence about keeping or removing it. What's your view?

@fneum fneum added this to the Release v0.3 milestone Dec 1, 2020
@FabianHofmann FabianHofmann merged commit c144787 into master Dec 2, 2020
@FabianHofmann FabianHofmann deleted the no-default-channel branch December 2, 2020 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create conda environment, results in “Found conflicts!”
3 participants