DeepSurv icon indicating copy to clipboard operation
DeepSurv copied to clipboard

required Theano version

Open baharian opened this issue 7 years ago • 5 comments

Is it absolutely necessary to have Theano 0.8.2 to properly run DeepSurv? Would DeepSurv fail to function if the Theano version is different from 0.8.2? Installing DeepSurv rolls back my Theano to an older, outdated version.

baharian avatar Oct 23 '17 21:10 baharian

I haven't used the latest DeepSurv version that just got updated, but I used the previous version (a few weeks ago) and I'm running Theano 0.10.0b1, lasagne 0.2.dev1 without any conflicts on deepsurv.

dareneiri avatar Oct 23 '17 21:10 dareneiri

Thanks! I'll just update my Theano back to the new version then.

baharian avatar Oct 23 '17 21:10 baharian

Would it break DeepSurv if >= is used, as follows,

install_requires=[
        'theano>=0.8.2',
        'lasagne==0.2.dev1',
        'lifelines>=0.9.2',
        'tensorboard_logger==0.0.3'
]

instead of == in setup.py so installing DeepSurv doesn't overwrite the existing Theano and lifelines packages?

baharian avatar Oct 24 '17 13:10 baharian

I've done something very similar and haven't had any issues.

What I usually do is first install deepsurv, and then manually install theano/lasagne with the bleeding edge version (see their site's documentation) to avoid any issues with theano not installing correctly.

dareneiri avatar Oct 24 '17 16:10 dareneiri

Been working fine with version 0.9.0 for me

TeaPearce avatar Nov 22 '17 18:11 TeaPearce