Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add reference DB and software versions to database. #54

Merged
merged 15 commits into from
Feb 9, 2024
Merged

Conversation

njohner
Copy link
Contributor

@njohner njohner commented Feb 2, 2024

With this PR we add a new "versions" table to the database and store the versions of the different reference databases used as well as of the AMRFinderPlus software in it. We display that information on the home page of zDB.
Screenshot 2024-02-09 at 10 28 41

Checklist

  • Changelog entry
  • Check that tests still pass
  • Add tests for new features and regression tests for bugfixes whenever possible.

@njohner njohner changed the title Nj/db versions2 Add reference DB and software versions to database. Feb 9, 2024
@njohner njohner marked this pull request as ready for review February 9, 2024 09:37
@njohner njohner requested review from bkm and tpillone February 9, 2024 09:37
Copy link
Contributor

Choose a reason for hiding this comment

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

ko_hits, pfam_hits, swissprot_hits, amr_hits,... counts might also change depending the database version

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes of course you're right. I'd also thought about that but decided not to change the tests proactively, but adapt them when needed. But it's clear that the tests are a bit fragile...

Copy link
Contributor

@tpillone tpillone left a comment

Choose a reason for hiding this comment

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

Looks good

@njohner njohner merged commit 9f25f49 into master Feb 9, 2024
@njohner njohner deleted the nj/db_versions2 branch February 9, 2024 13:27
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.

2 participants