Skip to content

Commit

Permalink
self.pipestat_results_file should take priority over self.pipeline_st…
Browse files Browse the repository at this point in the history
…ats_file related to databio/pepatac#257
  • Loading branch information
donaldcampbelljr committed Nov 20, 2023
1 parent 1a677da commit ed95993
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions pypiper/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,9 @@ def __init__(
self, filename="pipestat_results.yaml"
)

if pipestat_results_file:
self.pipestat_results_file = pipestat_results_file

def _get_arg(args_dict, arg_name):
"""safely get argument from arg dict -- return None if doesn't exist"""
return None if arg_name not in args_dict else args_dict[arg_name]
Expand All @@ -350,8 +353,9 @@ def _get_arg(args_dict, arg_name):
schema_path=pipestat_schema
or _get_arg(args_dict, "pipestat_schema")
or default_pipestat_output_schema(sys.argv[0]),
results_file_path=self.pipeline_stats_file
or _get_arg(args_dict, "pipestat_results_file"),
results_file_path=self.pipestat_results_file
or _get_arg(args_dict, "pipestat_results_file")
or self.pipeline_stats_file,
config_file=pipestat_config or _get_arg(args_dict, "pipestat_config"),
multi_pipelines=multi,
)
Expand Down

0 comments on commit ed95993

Please sign in to comment.