fix: proper error message if resource types do not match

snakemake/rules.py

@@ -1081,13 +1081,23 @@ def apply(name, res, threads=None):
 
             if not isinstance(res, int) and not isinstance(res, str):
                 raise WorkflowError(
-                    "Resources function did not return int, float (floats are "
-                    "rouded to the nearest integer), or str.",
+                    f"Resource {name} is neither int, float(would be rounded to nearest int), or str.",
                     rule=self,
                 )
-            if isinstance(res, int):
-                global_res = self.workflow.global_resources.get(name, res)
-                if global_res is not None:
+
+            global_res = self.workflow.global_resources.get(name)
+            if global_res is not None:
+                if not isinstance(res, TBDString) and type(res) != type(global_res):
+                    global_type = (
+                        "an int" if isinstance(global_res, int) else type(global_res)
+                    )
+                    raise WorkflowError(
+                        f"Resource {name} is of type {type(res).__name__} but global resource constraint "
+                        f"defines {global_type} with value {global_res}. "
+                        "Resources with the same name need to have the same types (int, float, or str are allowed).",
+                        rule=self,
+                    )
+                if isinstance(res, int):
                     res = min(global_res, res)
             return res
 

tests/test_github_issue1389/Snakefile

@@ -0,0 +1,22 @@
+rule all:
+    input:
+        "test1.txt",
+        "test2.txt",
+
+
+rule a:
+    output:
+        "test1.txt",
+    resources:
+        foo=1,
+    shell:
+        "touch {output}"
+
+
+rule b:
+    output:
+        "test2.txt",
+    resources:
+        foo="bar",
+    shell:
+        "touch {output}"

tests/tests.py

@@ -1597,3 +1597,8 @@ def test_github_issue1498():
 
 def test_cleanup_metadata_fail():
     run(dpath("test09"), cleanup_metadata=["xyz"])
+
+
+@skip_on_windows  # same on win, no need to test
+def test_github_issue1389():
+    run(dpath("test_github_issue1389"), resources={"foo": 4}, shouldfail=True)