Skip to content
New issue

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

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

Already on GitHub? # to your account

Add all the standard views for the virulence factors #58

Merged
merged 40 commits into from
Feb 23, 2024
Merged

Conversation

njohner
Copy link
Contributor

@njohner njohner commented Feb 15, 2024

With this pull request we add all the standard views for the virulence factors, i.e.

  • detail view
  • comparison index view
  • entry list
  • tabular comparison
  • Venn diagram
  • presence/absence table
  • heatmap
  • accumulation/rarefaction plot

Checklist

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

Setting publishDir with the move option in nextflow processes
will actually move the symlinks and not the files themselves.
This is not the desired behavior for the reference database.
Not all VF genes have an entry in the VF definition file, so we
now extract as much info as possible from the sequence description
and complement that info if there is an entry in the definition
file.
The vf_id column is not necessary as vf_gen_id is unique and can
be used as index. It also makes it easier to use the same column
as index and as identifier than having two distinct columns.
@njohner njohner marked this pull request as ready for review February 20, 2024 08:59
@njohner njohner requested review from bkm and tpillone February 20, 2024 09:00
@njohner njohner changed the title Nj/vfdb views Add all the standard views for the virulence factors Feb 20, 2024
@njohner njohner merged commit 89524d1 into master Feb 23, 2024
@njohner njohner deleted the nj/vfdb_views branch February 23, 2024 07:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant