NifTK / NiftyNet

[unmaintained] An open-source convolutional neural networks platform for research in medical image analysis and image-guided therapy
http://niftynet.io
Apache License 2.0
1.36k stars 404 forks source link

Support for tensorflow 1.15 #459

Closed tvercaut closed 4 years ago

tvercaut commented 4 years ago

Compatibility with TF 1.1.5 needs to be confirmed.

Additionally, it would probably be good to make TF 1.15 the default backend for tensorflow. Indeed, we can read that

For the 1.15 release, CPU and GPU support are included in a single package

https://www.tensorflow.org/install/pip?lang=python3

This should simplify installation requirements notably by allowing to merge https://github.com/NifTK/NiftyNet/blob/dev/requirements-cpu.txt https://github.com/NifTK/NiftyNet/blob/dev/requirements-gpu.txt

danieltudosiu commented 4 years ago

I will take this issue.

On Sat, 2 Nov 2019, 13:21 Tom Vercauteren, notifications@github.com wrote:

Compatibility with TF 1.1.5 needs to be confirmed.

Additionally, it would probably be good to make TF 1.15 the default backend for tensorflow. Indeed, we can read that

For the 1.15 release, CPU and GPU support are included in a single package

https://www.tensorflow.org/install/pip?lang=python3

This should simplify installation requirements notably by allowing to merge https://github.com/NifTK/NiftyNet/blob/dev/requirements-cpu.txt https://github.com/NifTK/NiftyNet/blob/dev/requirements-gpu.txt

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/NifTK/NiftyNet/issues/459?email_source=notifications&email_token=ACQEEVHSSSPRABKJ4QD5J4TQRV5GPA5CNFSM4JIFKBYKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWKR42A, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACQEEVDFCMY6US35EBZFNCTQRV5GPANCNFSM4JIFKBYA .