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

Update nextclade to version to 3.9.0 #6479

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bwlang
Copy link
Contributor

@bwlang bwlang commented Oct 22, 2024

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

@bwlang bwlang changed the title Update version to 3.9.0 Update nextclade to version to 3.9.0 Oct 22, 2024
@bwlang
Copy link
Contributor Author

bwlang commented Oct 22, 2024

@pvanheus
I thought this might be a simple upgrade to a new tool version... but it seems not.
Looks like this was mostly your work - want to to collaborate a bit on the update? ( I want RSV features...)

@pvanheus
Copy link
Contributor

@pvanheus I thought this might be a simple upgrade to a new tool version... but it seems not. Looks like this was mostly your work - want to to collaborate a bit on the update? ( I want RSV features...)

Sure. One problem with the current wrapper though, is related to how Galaxy handles column metadata: this is a static list, whereas in Nextclade the output columns change dependent on what dataset you're using. So this means that the column headers have to be included in the Nextclade output because Galaxy's metadata-based column headers don't work. Which is annoying if you want to be able to concatenate Nextclade results together.

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