Fixes model zoo unit tests

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 403 forks source link

Fixes model zoo unit tests #423

Closed wyli closed 5 years ago

wyli commented 5 years ago

Status

ready

Description

upgrades CI tests to TF 1.13/1.14 (requires cuda 10);
resumes model zoo functionality, fixes unit tests

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] All new and existing tests passed.

fepegar commented 5 years ago

Hi @wyli,

I'm getting this error and I think it's because of this new requirement:

ImportError                               Traceback (most recent call last)
<ipython-input-5-a4ef6d4138a8> in <module>()
----> 1 import niftynet

1 frames
/usr/local/lib/python3.6/dist-packages/niftynet/io/misc_io.py in set_logger(file_name)
    754     """
    755     # pylint: disable=no-name-in-module
--> 756     from tensorflow.python.platform.tf_logging import _get_logger
    757 
    758     logger = _get_logger()

ImportError: cannot import name '_get_logger'

wyli commented 5 years ago

Hi @wyli,

I'm getting this error and I think it's because of this new requirement:

ImportError                               Traceback (most recent call last)
<ipython-input-5-a4ef6d4138a8> in <module>()
----> 1 import niftynet

1 frames
/usr/local/lib/python3.6/dist-packages/niftynet/io/misc_io.py in set_logger(file_name)
    754     """
    755     # pylint: disable=no-name-in-module
--> 756     from tensorflow.python.platform.tf_logging import _get_logger
    757 
    758     logger = _get_logger()

ImportError: cannot import name '_get_logger'

Hi @fepegar the pip version is out of date, hopefully we'll roll out a new version before miccai