Skip to content
Permalink
Browse files
fix: wms monitor arg parsing now accepts any kind of value (#1181)
* Fixes #1180. No longer casting WMS args as integers.

* Fixes #1180. Removing unused parse_resources() import.

* Fixing WMS args parsing.

* Removed unused imports.

Co-authored-by: terneto1 <tobias.ternent@novartis.com>
Co-authored-by: Johannes Köster <johannes.koester@uni-due.de>
  • Loading branch information
3 people committed Oct 21, 2021
1 parent 72856ed commit 313de932e2e2a4f2c530df18c1abb15d37eb3217
Showing 1 changed file with 2 additions and 4 deletions.
@@ -11,10 +11,8 @@
import os
import json
import threading
import tempfile
from functools import partial
import inspect
import traceback
import textwrap

from snakemake.common import DYNAMIC_FILL
@@ -128,10 +126,10 @@ def __init__(self, address=None, args=None, metadata=None):
workflow will already be running and it would not be worth stopping it.
"""

from snakemake.resources import parse_resources
from snakemake.resources import DefaultResources

self.address = address or "http:127.0.0.1:5000"
self.args = parse_resources(args) or []
self.args = map(DefaultResources.decode_arg, args) if args else []
self.metadata = metadata or {}

# A token is suggested but not required, depends on server

0 comments on commit 313de93

Please sign in to comment.