Deprecate giftiio

Author: Ben Cipollini

nibabel/gifti/giftiio.py

@@ -10,12 +10,15 @@
 # Stephan Gerhard, Oktober 2010
 ##############
 
-import os
-import codecs
+import warnings
 
-from .parse_gifti_fast import parse_gifti_file
+import numpy as np
 
 
+warnings.warn('Please use nibabel.load/save.', DeprecationWarning)
+
+
+@np.deprecate_with_doc("Use nibabel.load() instead.")
 def read(filename):
     """ Load a Gifti image from a file
 
@@ -29,11 +32,11 @@ def read(filename):
     img : GiftiImage
         Returns a GiftiImage
      """
-    if not os.path.isfile(filename):
-        raise IOError("No such file or directory: '%s'" % filename)
-    return parse_gifti_file(filename)
+    from ..loadsave import load
+    return load(filename)
 
 
+@np.deprecate_with_doc("Use nibabel.save() instead.")
 def write(image, filename):
     """ Save the current image to a new file
 
@@ -79,6 +82,6 @@ def write(image, filename):
 
     The Gifti file is stored in endian convention of the current machine.
     """
-    # Our giftis are always utf-8 encoded - see GiftiImage.to_xml
-    with open(filename, 'wb') as f:
-        f.write(image.to_xml())
+    from ..loadsave import save
+    return save(image, filename)
+

nibabel/gifti/tests/test_giftiio.py

@@ -0,0 +1,35 @@
+# emacs: -*- mode: python-mode; py-indent-offset: 4; indent-tabs-mode: nil -*-
+# vi: set ft=python sts=4 ts=4 sw=4 et:
+### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
+#
+#   See COPYING file distributed along with the NiBabel package for the
+#   copyright and license terms.
+#
+### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
+
+import warnings
+
+from nose.tools import (assert_true, assert_false, assert_equal,
+                        assert_raises)
+from ...testing import clear_and_catch_warnings
+
+
+from .test_parse_gifti_fast import (DATA_FILE1, DATA_FILE2, DATA_FILE3,
+                                    DATA_FILE4, DATA_FILE5, DATA_FILE6)
+
+
+class TestGiftiIO(object):
+    def setUp(self):
+        with clear_and_catch_warnings() as w:
+            warnings.simplefilter('always', DeprecationWarning)
+            import nibabel.gifti.giftiio
+            assert_equal(len(w), 1)
+
+
+def test_read_deprecated():
+    with clear_and_catch_warnings() as w:
+        warnings.simplefilter('always', DeprecationWarning)
+        from nibabel.gifti.giftiio import read
+
+        img = read(DATA_FILE1)
+        assert_equal(len(w), 1)