Skip to content

Conversation

@hildebra
Copy link
Owner

@hildebra hildebra commented Nov 1, 2025

No description provided.

Anthony Duncan (EI) and others added 30 commits May 19, 2025 16:03
Added a script to download and format GTDB and GTDBtk databases, as well as to try to automatically udpate environment MGTKgtdbtk for different versions.
When concatenating parts of the GTDBtk database, the correct directory is now used. Created stub data (`helpers/install/get_gtdb`) for testing extraction logic.  Use flag `--test` to use this stub data. Additionally fixed an error in arguments passed to  `_processing_metadata` when downloading.
* scripts to autogenerate a QC report

* changes to existing files to create the QC report

* remove hardcoded path in QC_html_report.Rmd

* Remove hardcoded path in QC_html_report.Rmd

* The actual call was commented out in MG-TK.pl

* bugs anthony noticed - coerce char type to num, some report looks weird
* GTDBtk v2.5

* remove mash, don't install tk default

* fix(get_gtdb.py): remove mash for new versions, fix DBDir iterpolation, remove default GTDB-Tk install

Newer versions of GTDB-Tk (>=2.5) do not use mash, so when configuring for this, comments out the GTDBtk_mash line in config_DBs.txt

[DBDir] is now appropriately used in config, was previously not detected due to a newline character.

No longer install GTDB-Tk by default, allow MG-TK to install based on environment yamls. Added --install-tk flag to install a version of GTDB-Tk which is compataible with the GTDB released being configured.

---------

Co-authored-by: Falk Hildebrand <falk.hildebrand@gmail.com>
* scripts to autogenerate a QC report

* changes to existing files to create the QC report

* remove hardcoded path in QC_html_report.Rmd

* Remove hardcoded path in QC_html_report.Rmd

* The actual call was commented out in MG-TK.pl

* bugs anthony noticed - coerce char type to num, some report looks weird
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants