Move hash_from_code to theano.utils and remove theano.gof.utils.hash_from_file

theano/configparser.py

-import hashlib
 import logging
 import os
 import shlex
@@ -15,7 +14,7 @@ from configparser import (
 from functools import wraps
 from io import StringIO
 
-from theano.utils import deprecated
+from theano.utils import deprecated, hash_from_code
 
 
 _logger = logging.getLogger("theano.configparser")
@@ -67,17 +66,6 @@ class _ChangeFlagsDecorator:
             v.__set__(self._root, self.old_vals[k])
 
 
-def _hash_from_code(msg):
-    """This function was copied from theano.gof.utils to get rid of that import."""
-    # hashlib.sha256() requires an object that supports buffer interface,
-    # but Python 3 (unicode) strings don't.
-    if isinstance(msg, str):
-        msg = msg.encode()
-    # Python 3 does not like module names that start with
-    # a digit.
-    return "m" + hashlib.sha256(msg).hexdigest()
-
-
 class _SectionRedirect:
     """Functions as a mock property on the TheanoConfigParser.
 
@@ -136,7 +124,7 @@ class TheanoConfigParser:
             [c for c in self._config_var_dict.values() if c.in_c_key],
             key=lambda cv: cv.name,
         )
-        return _hash_from_code(
+        return hash_from_code(
             "\n".join(
                 [
                     "{} = {}".format(cv.name, cv.__get__(self, self.__class__))

theano/gof/utils.py

-import hashlib
 import linecache
 import sys
 import traceback
@@ -462,23 +461,3 @@ def flatten(a):
         return l
     else:
         return [a]
-
-
-def hash_from_code(msg):
-    # hashlib.sha256() requires an object that supports buffer interface,
-    # but Python 3 (unicode) strings don't.
-    if isinstance(msg, str):
-        msg = msg.encode()
-    # Python 3 does not like module names that start with
-    # a digit.
-    return "m" + hashlib.sha256(msg).hexdigest()
-
-
-def hash_from_file(file_path):
-    """
-    Return the SHA256 hash of a file.
-
-    """
-    with open(file_path, "rb") as f:
-        file_content = f.read()
-    return hash_from_code(file_content)

theano/link/c/cmodule.py

@@ -26,9 +26,14 @@ from theano.configdefaults import config, gcc_version_str
 
 # we will abuse the lockfile mechanism when reading and writing the registry
 from theano.gof import compilelock
-from theano.gof.utils import flatten, hash_from_code
+from theano.gof.utils import flatten
 from theano.link.c.exceptions import MissingGXX
-from theano.utils import LOCAL_BITWIDTH, output_subprocess_Popen, subprocess_Popen
+from theano.utils import (
+    LOCAL_BITWIDTH,
+    hash_from_code,
+    output_subprocess_Popen,
+    subprocess_Popen,
+)
 
 
 importlib = None

theano/sparse/utils.py

-from theano.gof.utils import hash_from_code
+from theano.utils import hash_from_code
 
 
 def hash_from_sparse(data):

theano/tensor/utils.py

 import numpy as np
 
 import theano
-from theano.gof.utils import hash_from_code
+from theano.utils import hash_from_code
 
 
 def hash_from_ndarray(data):

theano/utils.py

 """Utility functions that only depend on the standard library."""
 
 
+import hashlib
 import inspect
 import os
 import struct
@@ -284,3 +285,14 @@ def output_subprocess_Popen(command, **params):
     # the stdout/stderr pipe.
     out = p.communicate()
     return out + (p.returncode,)
+
+
+def hash_from_code(msg):
+    """Return the SHA256 hash of a string or bytes."""
+    # hashlib.sha256() requires an object that supports buffer interface,
+    # but Python 3 (unicode) strings don't.
+    if isinstance(msg, str):
+        msg = msg.encode()
+    # Python 3 does not like module names that start with
+    # a digit.
+    return "m" + hashlib.sha256(msg).hexdigest()