Automatically convert integers to doubles when argument type is double

CHANGELOG.md

@@ -1,3 +1,9 @@
+# Viash 0.8.8 (xxxx-xx-xx): xxxx
+
+## BUG FIXES
+
+* `NextflowRunner`: Automatically convert integers to doubles when argument type is `double` (PR #xxx).
+
 # Viash 0.8.7 (2025-04-01): Backport support upcoming version of Nextflow
 
 The upcoming release of Nextflow introduces a new class for loading scripts and renamed the old class.

src/main/resources/io/viash/platforms/nextflow/arguments/_checkArgumentType.nf

@@ -106,10 +106,10 @@ def _checkArgumentType(String stage, Map par, Object value, String errorIdentifi
         // do nothing
       }
     }
-    if (value instanceof java.math.BigDecimal) {
+    if (value instanceof java.math.BigDecimal || value instanceof java.math.BigInteger) {
       value = value.doubleValue()
     }
-    if (value instanceof Float) {
+    if (value instanceof Float || value instanceof Integer || value instanceof Long) {
       value = value.toDouble()
     }
     expectedClass = value instanceof Double ? null : "Double"

src/test/resources/testnextflowvdsl3/src/integer_as_double/config.vsh.yaml

@@ -0,0 +1,25 @@
+functionality:
+  name: integer_as_double
+  arguments:
+    - name: "--input"
+      type: file
+      required: true
+      example: input.txt
+    - name: "--output"
+      type: file
+      required: true
+      direction: output
+      example: output.txt
+    - name: "--double"
+      type: double
+      required: true
+      direction: input
+      example: 10.5
+  resources:
+    - type: bash_script
+      path: script.sh
+platforms:
+  - type: native
+  - type: docker
+    image: nextflow/bash:latest
+  - type: nextflow

src/test/resources/testnextflowvdsl3/src/integer_as_double/script.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+
+## VIASH START
+par_input='resources/lines3.txt'
+par_double='10.5'
+par_output='output.txt'
+## VIASH END
+
+echo "Copying $par_input to $par_output"
+cp "$par_input" "$par_output"
+
+echo "Adding $par_double to $par_output"
+echo "Double: $par_double" >> "$par_output"
+
+exit 0

src/test/scala/io/viash/platforms/nextflow/Vdsl3StandaloneTest.scala

@@ -211,6 +211,46 @@ class Vdsl3StandaloneTest extends AnyFunSuite with BeforeAndAfterAll {
     }
   }
 
+
+  test("Whether integers can be converted to doubles", NextflowTest) {
+    // create params.yaml file
+    val paramsPath = temporaryFolder.resolve("params.yaml")
+    val paramsStr = """id: foo
+      |input: resources/lines3.txt
+      |double: 10
+      |publish_dir: integerAsDouble
+      |""".stripMargin
+    Files.write(paramsPath, paramsStr.getBytes(StandardCharsets.UTF_8))
+
+    val (exitCode, stdOut, stdErr) = NextflowTestHelper.run(
+      mainScript = "target/nextflow/integer_as_double/main.nf",
+      args = Nil,
+      paramsFile = Some(paramsPath.toString()),
+      cwd = tempFolFile
+    )
+
+    assert(exitCode == 0, s"\nexit code was $exitCode\nStd output:\n$stdOut\nStd error:\n$stdErr")
+
+    val expectedFiles =
+      Map(
+        "state" -> "state.yaml",
+        "output" -> "output.txt", 
+      ).map{ case (id, suffix) =>
+        val path = temporaryFolder.resolve("integerAsDouble/foo.integer_as_double." + suffix)
+        (id, path)
+      }
+
+    // check if files exist
+
+    val src = Source.fromFile(tempFolStr + "/integerAsDouble/foo.integer_as_double.output.txt")
+    try {
+      val moduleOut = src.getLines().mkString(",")
+      assert(moduleOut.equals("one,two,three,Double: 10.0"), s"Expected output 'one,two,three,10.0' but got '$moduleOut'")
+    } finally {
+      src.close()
+    }
+  }
+
   override def afterAll(): Unit = {
     IO.deleteRecursively(temporaryFolder)
   }