Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Bill Kreamer
    @williamkreamer
    Hi patrick
    thanks for your help!
    Patrick Snape
    @patricksnape
    Hi Bill
    NO problem at all, you did all the hard work!
    Bill Kreamer
    @williamkreamer
    I saw your changes and they look great. I merged them in
    Patrick Snape
    @patricksnape
    Great, does it work for you?
    Bill Kreamer
    @williamkreamer
    I think the code runs now - I'd like to add a few more unit tests that verify the functionality. Once I do that, what's next?
    Patrick Snape
    @patricksnape
    I'll just give it a once over, make sure it passes on the continuous integration, merge it and then tag it and the new release will get automatically bundled and pushed out via conda :)
    Which hopefully makes everything easy for you
    Bill Kreamer
    @williamkreamer
    ok - I'll let you know when I'm ready
    Patrick Snape
    @patricksnape
    Thanks a lot for contributing, I'm impressed by how quickly you got the hang of Cython, it took me a lot longer!
    Bill Kreamer
    @williamkreamer
    I'm a good mimic :)
    Patrick Snape
    @patricksnape
    Unfortunately the Windows builds are a bit slow, so it's always best to try and push your commits in blocks to minimize how many times Appveyor tries to rebuild (since it defaults to only ever building the last commit in a push)
    Bill Kreamer
    @williamkreamer
    ok, I'll minimize my pushes in future
    Bill Kreamer
    @williamkreamer
    All set now. I pushed up a second unit test case which also passes. Ready for your review
    Bill Kreamer
    @williamkreamer
    I'd like to go ahead and "install" this version for my local use. I tried to follow the instructions in the README.md but the command "conda build ./conda" failed with the error below
    ....

    install_name_tool -add_rpath @loader_path/../../../../ /Users/williamkreamer/anaconda/envs/_build/lib/python2.7/site-packages/cyvlfeat/fisher/cyfisher.so

    Fixing linking of @rpath/./libvl.dylib in /Users/williamkreamer/anaconda/envs/_build/lib/python2.7/site-packages/cyvlfeat/sift/cysift.so
    Fixing linking of /usr/lib/libSystem.B.dylib in /Users/williamkreamer/anaconda/envs/_build/lib/python2.7/site-packages/cyvlfeat/sift/cysift.so
    install_name_tool -id @rpath/python2.7/site-packages/cyvlfeat/sift/libvl.dylib /Users/williamkreamer/anaconda/envs/_build/lib/python2.7/site-packages/cyvlfeat/sift/cysift.so

    install_name_tool -add_rpath @loader_path/../../../../ /Users/williamkreamer/anaconda/envs/_build/lib/python2.7/site-packages/cyvlfeat/sift/cysift.so

    Ignored hard-coded path in lib/python2.7/site-packages/cyvlfeat/sift/cysift.so
    Fixing permissions
    BUILD END: cyvlfeat--py27_0
    TEST START: cyvlfeat--py27_0
    Fetching package metadata: ........
    Error: No packages found in current osx-64 channels matching: cyvlfeat py27_0

    And if I try to "from cyvlfeat.fisher import fisher" I get the error below (probably as a consequence of the above error):

    File "/Users/williamkreamer/censio_repos/user/wkreamer/classify_exit_time.py", line 8, in <module>
    from cyvlfeat.fisher import fisher
    File "/Users/williamkreamer/censiorepos/cyvlfeat/cyvlfeat/_init.py", line 1, in <module>
    import cyvlfeat.sift
    File "/Users/williamkreamer/censiorepos/cyvlfeat/cyvlfeat/sift/_init.py", line 1, in <module>
    from .dsift import dsift
    File "/Users/williamkreamer/censio_repos/cyvlfeat/cyvlfeat/sift/dsift.py", line 2, in <module>
    from .cysift import cy_dsift

    ImportError: dlopen(/Users/williamkreamer/censio_repos/cyvlfeat/cyvlfeat/sift/cysift.so, 2): Library not loaded: @rpath/./libvl.dylib
    Referenced from: /Users/williamkreamer/censio_repos/cyvlfeat/cyvlfeat/sift/cysift.so
    Reason: image not found

    Patrick Snape
    @patricksnape
    Ah! That is totally my fault :(
    That's because the version number is automatically taken from the CI, and when you run it locally you have to provide a version number
    Patrick Snape
    @patricksnape
    Just call the command like this:
    CONDACI_VERSION=0.4.3 conda build conda
    I will update the docs!
    Patrick Snape
    @patricksnape
    The latest version (v0.4.3) should be uploaded now! Let me know if you have any problems!