theiagen / public_health_bacterial_genomics

GNU Affero General Public License v3.0
27 stars 14 forks source link

kSNP3 task edit: changed file suffix from .tree to .nwk #146

Closed kapsakcj closed 2 years ago

kapsakcj commented 2 years ago

Relatively small change - output tree files end in .nwk now instead of .tree

Also made file-renaming mv commands verbose for logging purposes

Should be a more recognizable suffix to users

No need to edit the mashtree task, those tree files end in .nwk already

This will address issue #39

Ran successfully in Terra here: https://app.terra.bio/#workspaces/cdph-terrabio-taborda-manual/curtis_sandbox/job_history/957ed8bf-d5d6-4a11-96a7-490cd6922d36

kapsakcj commented 2 years ago

And for futher confirmation - both output .nwk files were valid and viewable in MEGA

kapsakcj commented 2 years ago

Thanks for reviewing and testing!