Changes in / [3bcb88c:1392831] in sasmodels

Location:

sasmodels

Files:

• generate.py (modified) (1 diff)
• kernelcl.py (modified) (1 diff)
• kerneldll.py (modified) (1 diff)

sasmodels/generate.py

@@ -316 +316 @@
 
 
-def timestamp(model_info):
+def dll_timestamp(model_info):
     # type: (ModelInfo) -> int
     """
     Return a timestamp for the model corresponding to the most recently
     changed file or dependency.
+    """
+    # TODO: fails DRY; templates appear two places.
+    model_templates = [joinpath(DATA_PATH, filename)
+                       for filename in ('kernel_header.c', 'kernel_iq.c')]
+    source_files = (model_sources(model_info)
+                    + model_templates
+                    + [model_info.filename])
+    # Note: file may not exist when it is a standard model from library.zip
+    times = [getmtime(f) for f in source_files if exists(f)]
+    newest = max(times) if times else 0
+    return newest
+
+def ocl_timestamp(model_info):
+    # type: (ModelInfo) -> int
+    """
+    Return a timestamp for the model corresponding to the most recently
+    changed file or dependency.
 
     Note that this does not look at the time stamps for the OpenCL header
     information since that need not trigger a recompile of the DLL.
     """
+    # TODO: fails DRY; templates appear two places.
+    model_templates = [joinpath(DATA_PATH, filename)
+                       for filename in ('kernel_header.c', 'kernel_iq.cl')]
     source_files = (model_sources(model_info)
-                    + model_templates()
+                    + model_templates
                     + [model_info.filename])
-    newest = max(getmtime(f) for f in source_files)
+    # Note: file may not exist when it is a standard model from library.zip
+    times = [getmtime(f) for f in source_files if exists(f)]
+    newest = max(times) if times else 0
     return newest
-
-
-def model_templates():
-    # type: () -> List[str]
-    # TODO: fails DRY; templates appear two places.
-    # should instead have model_info contain a list of paths
-    # Note: kernel_iq.cl is not on this list because changing it need not
-    # trigger a recompile of the dll.
-    return [joinpath(DATA_PATH, filename)
-            for filename in ('kernel_header.c', 'kernel_iq.c')]
 
 

sasmodels/kernelcl.py

@@ -397 +397 @@
         if self.program is None:
             compile_program = environment().compile_program
-            timestamp = generate.timestamp(self.info)
+            timestamp = generate.ocl_timestamp(self.info)
             self.program = compile_program(
                 self.info.name,

sasmodels/kerneldll.py

@@ -226 +226 @@
     else:
         dll_time = os.path.getmtime(dll)
-        newest_source = generate.timestamp(model_info)
+        newest_source = generate.dll_timestamp(model_info)
         need_recompile = dll_time < newest_source
     if need_recompile: