search

BRML / climin

Optimizers for machine learning
Other
180 stars 66 forks source link

climin now runs on both Python2.7 and Python3.4 #31

Closed superbobry closed 9 years ago

superbobry commented 9 years ago

The tests in test_rprop.py fail in the master version. I've verified that the failure is unrelate to the compatibility fixes.

Also, it seems the builds on Travis CI fail with timeout. I can alter .travis.yml to use conda for NumPy/SciPy. This should significantly speed up the installation step. What do you think?

f0k commented 9 years ago

In Lasagne, we also use conda: https://github.com/Lasagne/Lasagne/blob/master/.travis.yml We only spend around 40 seconds on before_install and install: https://travis-ci.org/Lasagne/Lasagne/jobs/77645621 Feel free to copy some of our configuration. (You could also use travis_wait to shield against the "10 minutes without output" error, but speeding up the installation process is probably a better idea.)

bayerj commented 9 years ago

Thanks, go ahead.

I am currently travelling and wont be able to merge before sep 15, but please go ahead. On Tue 1 Sep 2015 at 12:40 Jan Schlüter notifications@github.com wrote:

In Lasagne, we also use conda: https://github.com/Lasagne/Lasagne/blob/master/.travis.yml We only spend around 40 seconds on before_install and install: https://travis-ci.org/Lasagne/Lasagne/jobs/77645621 Feel free to copy some of our configuration. (You could also use travis_wait to shield against the "10 minutes without output" error, but speeding up the installation process is probably a better idea.)

— Reply to this email directly or view it on GitHub https://github.com/BRML/climin/pull/31#issuecomment-136669588.

f0k commented 9 years ago

In between, we've switched to Travis' new container-based infrastructure: https://github.com/Lasagne/Lasagne/blob/master/.travis.yml Due to caching, that's a lot faster even without conda.