added recipe for bilm

[PhilipMay]

added recipe for bilm - nothing else to say :-)

[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 (recipes/bilm) and found it was in an excellent condition.

[PhilipMay]

Does anybody have an idea why this fails on Mac and Linux build but not on Windows? On my local Mac build everything is fine.

[marcelotrevisani]

It seems you will need the c compiler

[marcelotrevisani]

I think you have to remove the noarch and add a build section inside of the requirements and after that add

  build:
    - {{ compiler('c') }}

[PhilipMay]

This is a 100% pure python package. See here: https://github.com/allenai/bilm-tf
Why do I need a compiler?

[marcelotrevisani]

It seems that tensorflow needs it

[PhilipMay]

Even the setup.py file is very simpel to build the PyPI package: https://github.com/PhilipMay/bilm-tf/blob/setup.py/setup.py

[PhilipMay]

It seems that tensorflow needs it

Yes TF needs c compiler for sure. But I do not build tensorflow... I am confused.

[marcelotrevisani]

probably TF needs the runtime libraries and it's not adding it into the package
It may be an error in the build process of TF or it's normal to add this, lets see if the build will pass now

[marcelotrevisani]

I'm just doing some tests and investigating that if you don't mind

[PhilipMay]

probably TF needs the runtime libraries and it's not adding it into the package
It may be an error in the build process of TF or it's normal to add this, lets see if the build will pass now

The Tensorflow build does not add the c runtime https://github.com/conda-forge/tensorflow-feedstock/blob/master/recipe/meta.yaml at run level. But I am sure that c is needed. Should that recipe be changed then?

[PhilipMay]

Well, now all builds fail. Even the Mac Build. Do you have any idears?

[marcelotrevisani]

Yes, maybe. As I don't have experience with TF I will mark @jakirkham and @dougalsutherland they may have a hint here

[marcelotrevisani]

If they do not answer you I will take a look on that later

[jjhelmus]

The tensorflow package in conda-forge is a repack of the wheels provided on PyPI. These wheels do not have the same compatibility as other packages in conda-forge. As such, tensorflow may not import on the build machines. For example, on Linux the wheel supports Ubuntu 16.04 or later and is incompatible with the CentOS 6 docker image used for builds, see conda-forge/tensorflow-feedstock#19, tensorflow/tensorflow#8802. Building from source, conda-forge/tensorflow-feedstock#6 may solve this issue but is challenging.

[PhilipMay]

@jjhelmus Ok - so I have to wait until the tensorflow package is fixed so my build does not fail?

[PhilipMay]

@jjhelmus Or should I just remove the tests like the tensorflow guys did:
https://github.com/conda-forge/tensorflow-feedstock/blob/034249e4e9a5a026f4bda812dfadef89ff5199b3/recipe/meta.yaml#L46

[PhilipMay]

The windows build has this failure:

failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform win-64: {'bilm==0.1=py27_0'}
WARNING:conda_build.build:failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform win-64: {'bilm==0.1=py27_0'}
Tests failed for bilm-0.1-py27_0.tar.bz2 - moving package to C:\bld\broken
WARNING:conda_build.build:Tests failed for bilm-0.1-py27_0.tar.bz2 - moving package to C:\bld\broken
TESTS FAILED: bilm-0.1-py27_0.tar.bz2

I think the reason is that tensorflow is not build for python 2.7 on windows. See here: https://github.com/conda-forge/tensorflow-feedstock/blob/master/recipe/meta.yaml#L10

How do I fix this in case of windows?

[jjhelmus]

tensorflow is not build for python 2.7 on windows

Tensorflow is limited to Python 3.x on Windows. If you require tensorflow at run time you will need to skip the Python 2.7 build on Windows.

[PhilipMay]

I removed the tensorflow dependency so you can decide if you want to use tensorflow-gpu or just tensorflow. Like it has been done in the setup.py of bilm: https://github.com/allenai/bilm-tf/blob/master/setup.py

[PhilipMay]

Can someone trigger a rebuild please?

[jakirkham]

Could you please merge with master? A lot has changed with our CI setup since this PR was opened.

[conda-forge-linter]

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

I wanted to let you know that I linted all conda-recipes in your PR (recipes/bilm) and found some lint.

Here's what I've got...

For recipes/bilm:

• The recipe must have some tests.

[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 (recipes/bilm) and found it was in an excellent condition.

[scopatz]

How do we know this is version 0.1 if there are no releases?

[scopatz]

Can this be noarch: python?