Refactor file_cache function to include func_source_code_hash (#3332)

Author: wwzeng1

docs/pages/blogs/file-cache.mdx

@@ -153,6 +153,10 @@ We have another problem - we want to miss our cache under two conditions:
 To handle 2. we used `inspect.getsource(func)` to add the function's source code to the hash, correctly missing the cache when the the code changes.
 
 ```python /hash_code(inspect.getsource(func))/
+    func_source_code_hash = hash_code(inspect.getsource(func))
+```
+
+```python /func_source_code_hash/
     args_names = func.__code__.co_varnames[: func.__code__.co_argcount]
     args_dict = dict(zip(args_names, args))
 
@@ -166,7 +170,7 @@ To handle 2. we used `inspect.getsource(func)` to add the function's source code
     arg_hash = (
         recursive_hash(args_dict, ignore_params=ignore_params)
         + recursive_hash(kwargs_clone, ignore_params=ignore_params)
-        + hash_code(inspect.getsource(func))
+        + func_source_code_hash
     )
     cache_file = os.path.join(
         cache_dir, f"{func.__module__}_{func.__name__}_{arg_hash}.pickle"

docs/public/file_cache.py

@@ -57,6 +57,7 @@ def decorator(func):
             if verbose:
                 print("Cache is disabled for function: " + func.__name__)
             return func
+        func_source_code_hash = hash_code(inspect.getsource(func))
 
         def wrapper(*args, **kwargs):
             cache_dir = "/tmp/file_cache"
@@ -76,7 +77,7 @@ def wrapper(*args, **kwargs):
             arg_hash = (
                 recursive_hash(args_dict, ignore_params=ignore_params)
                 + recursive_hash(kwargs_clone, ignore_params=ignore_params)
-                + hash_code(inspect.getsource(func))
+                + func_source_code_hash
             )
             cache_file = os.path.join(
                 cache_dir, f"{func.__module__}_{func.__name__}_{arg_hash}.pickle"
@@ -103,4 +104,4 @@ def wrapper(*args, **kwargs):
 
         return wrapper
 
-    return decorator
+    return decorator

sweepai/logn/cache.py

@@ -56,6 +56,7 @@ def file_cache(ignore_params=[], verbose=False):
     def decorator(func):
         if DEBUG:
             return func
+        func_source_code_hash = hash_code(inspect.getsource(func))
 
         def wrapper(*args, **kwargs):
             cache_dir = "/tmp/file_cache"
@@ -75,7 +76,7 @@ def wrapper(*args, **kwargs):
             arg_hash = (
                 recursive_hash(args_dict, ignore_params=ignore_params)
                 + recursive_hash(kwargs_clone, ignore_params=ignore_params)
-                + hash_code(inspect.getsource(func))
+                + func_source_code_hash
             )
             cache_file = os.path.join(
                 cache_dir, f"{func.__module__}_{func.__name__}_{arg_hash}.pickle"