Skip to content

Commit

Permalink
subprocess.check_output needs to be in string format
Browse files Browse the repository at this point in the history
  • Loading branch information
crosenth committed Sep 19, 2024
1 parent 255d433 commit 0986394
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion deenurp/wrap.py
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ def rppr_min_adcl_tree(newick_file, leaves, algorithm='pam', always_include=None
if always_include:
cmd.extend(('--always-include', always_include))
logging.debug(' '.join(cmd))
output = subprocess.check_output(cmd)
output = subprocess.check_output(cmd, text=True)
return output.splitlines()


Expand Down

1 comment on commit 0986394

@crosenth
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nhoffman - Fixes the too many species bug uncovered during our Monday morning meeting with Yee Mey. Basically, leaves were not being pruned to 5 because data was returned in bytes

Please sign in to comment.