Skip to content
This repository has been archived by the owner on Apr 26, 2022. It is now read-only.

Constrain scikit-image #54

Merged
merged 2 commits into from
Feb 7, 2020
Merged

Constrain scikit-image #54

merged 2 commits into from
Feb 7, 2020

Conversation

marcelotrevisani
Copy link
Member

@marcelotrevisani marcelotrevisani commented Feb 7, 2020

Checklist

  • Used a fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@marcelotrevisani
Copy link
Member Author

That is necessary because of the latest scikit-image removed one function which caffe is using
https://github.com/scikit-image/scikit-image/blob/master/doc/release/release_0.16.rst#api-changes

caffe uses as_grey function and it was renamed to as_gray

I think we should not patch it to fix this problem because we might face new problems. I will just constrain the scikit-image and it will be enough for now

@marcelotrevisani
Copy link
Member Author

@conda-forge-admin, please rerender

@marcelotrevisani
Copy link
Member Author

@ocefpaf , It would be nice to mark the packages with build number 203 and 204 as broken as well.

@marcelotrevisani marcelotrevisani merged commit 7023e0e into conda-forge:master Feb 7, 2020
@marcelotrevisani marcelotrevisani deleted the constrain-scikit-image branch February 7, 2020 14:36
@marcelotrevisani marcelotrevisani mentioned this pull request Feb 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants