Skip to content

Commit

Permalink
remove redundant io in main & remove irrelevant warning/exit in compi…
Browse files Browse the repository at this point in the history
…ler (#284)

* add user args for workflows built with python api

* remove redundant io in main & remove irrelevant warning/exit in compiler

---------

Co-authored-by: Vasu Jaganath <vasu.jaganath@axleinfo.com>
  • Loading branch information
vjaganat90 and Vasu Jaganath authored Oct 16, 2024
1 parent 7719daf commit edbe28e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
4 changes: 0 additions & 4 deletions src/sophios/compiler.py
Original file line number Diff line number Diff line change
Expand Up @@ -881,10 +881,6 @@ def compile_workflow_once(yaml_tree_ast: YamlTree,
new_keyval = {key: newval}
elif 'Directory' == in_dict['type']:
if not args.ignore_dir_path:
if in_dict['value'].startswith('/'):
print("Warning! directory can not start with '/'")
print("It is most likely an incorrect path! Can't create directories!")
sys.exit(1)
ldir = Path(in_dict['value'])
if not ldir.is_absolute():
ldir = Path('autogenerated') / ldir
Expand Down
3 changes: 0 additions & 3 deletions src/sophios/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -194,9 +194,6 @@ def main() -> None:
print("(This may happen if you installed the graphviz python package")
print("but not the graphviz system package.)")

if args.generate_cwl_workflow:
io.write_to_disk(rose_tree, Path('autogenerated/'), True, args.inputs_file)

if args.run_local or args.generate_run_script:
# cwl-docker-extract recursively `docker pull`s all images in all subworkflows.
# This is important because cwltool only uses `docker run` when executing
Expand Down

0 comments on commit edbe28e

Please sign in to comment.