Skip to content

Commit

Permalink
Workflow setup fix (#117)
Browse files Browse the repository at this point in the history
* Fixed a workflow setup issue caused by spaces in the spec name

* Additional dirname formatting
  • Loading branch information
jsemler authored and FrankD412 committed Jul 7, 2018
1 parent 1122fb2 commit b05d4c8
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions maestrowf/maestro.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def run_study(args):
environment.add(Variable("OUTPUT_PATH", output_path))

# Now that we know outpath, set up logging.
setup_logging(args, output_path, spec.name)
setup_logging(args, output_path, spec.name.replace(" ", "_").lower())

# Addition of the $(SPECROOT) to the environment.
spec_root = os.path.split(args.specification)[0]
Expand Down Expand Up @@ -191,7 +191,8 @@ def run_study(args):
raise NotImplementedError("The 'dryrun' mode is in development.")

# Pickle up the DAG
pkl_path = os.path.join(path, "{}.pkl".format(study.name))
pkl_path = os.path.join(path, "{}.pkl".format(
study.name.replace(" ", "_").lower()))
exec_dag.pickle(pkl_path)

# If we are automatically launching, just set the input as yes.
Expand Down

0 comments on commit b05d4c8

Please sign in to comment.