Skip to content

Comments

Show installed metaphlan db version with the metaphlan --version command#238

Merged
Cengoni merged 5 commits intomasterfrom
feature/db-version-show
Oct 10, 2025
Merged

Show installed metaphlan db version with the metaphlan --version command#238
Cengoni merged 5 commits intomasterfrom
feature/db-version-show

Conversation

@sagun98
Copy link
Contributor

@sagun98 sagun98 commented Oct 1, 2025

Hi @Cengoni ,
Here I have open a PR for to show installed metaphlan db version with the metaphlan --version command if available. Please feel free to review/merge the code to master or let me know if anything needs to be changed here.

I will follow up with you in slack as well.

Regards,
Sagun

@sagun98 sagun98 requested a review from Cengoni October 1, 2025 14:02
@sagun98
Copy link
Contributor Author

sagun98 commented Oct 7, 2025

Hi @Cengoni ,
As discussed per our conversation, I have update the pull request to support multiple metaphlan DB + non-default metaphlan DB folder with the metaphlan --version command. Please let me know if anything needs to be changed here. I will follow up in slack as well.

@Cengoni Cengoni merged commit 45727b0 into master Oct 10, 2025
1 check passed
@Cengoni
Copy link
Collaborator

Cengoni commented Oct 10, 2025

Hi @sagun98,
Thank you for adding this feature, I slightly modified the criteria for detection of the database for consistency with the latest commit I made on master. Instead of checking the presence of the fasta file - which is now removed after installation - it checks for presence of bowtie2 indexes, which are required together with the pickle file to be present in order to be able to run MetaPhlAn.

Best,
Claudia

@Cengoni Cengoni deleted the feature/db-version-show branch October 10, 2025 13:26
@SimHolz
Copy link

SimHolz commented Feb 13, 2026

Hi,

I think this PR breaks the compatibility with Humann 3.9.
At least I get the following error from Humann 3.9 with MetaPhlAn 4.1.2 in the same environment/container.
CRITICAL ERROR: Can not call software version for metaphlan

It runs with no problems using MetaPhlAn 4.1.1

Just wanted to note it somewhere in case someone runs into the same problem.

Best wishes
Simon

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.

3 participants