[TheiaCoV] Update nextclade dataset tag for H5N1 to the latest version #683
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR closes #682
🗑️ This dev branch should be deleted after merging to main.
🧠 Summary
Updates wf_organism_paramters ds tag for H5N1 HA segment
⚡ Impacted Workflows/Tasks
wf_organism_paramters
This PR may lead to different results in pre-existing outputs: No
This PR uses an element that could cause duplicate runs to have different results: No
🛠️ Changes
h5n1_ha_nextclade_ds_tag
changed from old2024-05-08--11-39-52Z
ds tag to new2024-12-04--17-05-31Z
tag.⚙️ Algorithm
h5n1_ha_nextclade_ds_tag
version bump for nextclade.➡️ Inputs
⬅️ Outputs
🧪 Testing
Tested specifically on h5n1 flu data here
-This output of this will show the
2024-12-04--17-05-31Z
ds tag as well as NA segment being correctly identified.Tested illumina_pe validation set here
Suggested Scenarios for Reviewer to Test
Please repeat this test here and check the nextclade tags. Feel free to test any other data set. The h5n1 data I was able to find was PE.
🔬 Final Developer Checklist
🎯 Reviewer Checklist