fix keras-* requirements

[jjhelmus]

No description provided.

[conda-forge-linter]

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.

[jjhelmus]

Requirements in the setup.py file:

numpy>=1.9.1
scipy>=0.14
six>=1.9.0
pyyaml
h5py
keras_applications>=1.0.6
keras_preprocessing>=1.0.5

[mariusvniekerk]

@conda-forge-admin please rerender

[kyleabeauchamp]

What's the status of this PR? I think it might unblock conda-forge/tensorflow-feedstock#66 due to numpy pinning stuff.

[dhirschfeld]

@conda-forge-admin please rerender

[dhirschfeld]

Looks like it's failing on py37 but the error message doesn't appear to make much sense to me:

TEST START: C:\bld\win-64\keras-2.2.4-py37_1.tar.bz2
Adding in variants from C:\Users\appveyor\AppData\Local\Temp\1\tmpoh1k8b4m\info\recipe\conda_build_config.yaml
INFO:conda_build.variants:Adding in variants from C:\Users\appveyor\AppData\Local\Temp\1\tmpoh1k8b4m\info\recipe\conda_build_config.yaml
Renaming work directory,  C:\bld\keras_1551223013718\work  to  C:\bld\keras_1551223013718\work_moved_keras-2.2.4-py37_1_win-64
Collecting package metadata: ...working... done
Solving environment: ...working... failed
failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform win-64: {'keras==2.2.4=py37_1'}
WARNING:conda_build.build:failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform win-64: {'keras==2.2.4=py37_1'}
Tests failed for keras-2.2.4-py37_1.tar.bz2 - moving package to C:\bld\broken
WARNING:conda_build.build:Tests failed for keras-2.2.4-py37_1.tar.bz2 - moving package to C:\bld\broken
TESTS FAILED: keras-2.2.4-py37_1.tar.bz2

[mariusvniekerk]

That generally means some dependency is missing for py37

[kyleabeauchamp]

I definitely saw some strange keras-numpy-TF pinning / requirement issues over in conda-forge/tensorflow-feedstock#66; I only saw issues on windows, though, so I skipped windows on that PR to get things green.

Seems like stuff is busted here on linux as well, though, so my experience won't be it...

[dhirschfeld]

Yeah, I figured but AFAICS the error message only lists the package itself rather than the missing dependency which isn't very helpful :(

I thought newer conda was supposed to have better error messages than that. It does look like the CI is running the latest 4.6.7 though

[jjhelmus]

closing in favor of #37