Skip to content

Conversation

@anna-parker
Copy link
Contributor

@anna-parker anna-parker commented Sep 22, 2025

required for https://github.com/GenSpectrum/servers/issues/294

Testing on genspectrum servers shows this is WAI

Screenshot

PR Checklist

  • [ ] All necessary documentation has been adapted.
  • [ ] The implemented feature is covered by appropriate, automated tests.
  • Any manual testing that has been done is documented (i.e. what exactly was tested?)

🚀 Preview: Add preview label to enable

@anna-parker anna-parker force-pushed the add_dataset_tag_map branch 2 times, most recently from be821b7 to b6d3a47 Compare September 22, 2025 11:42
@anna-parker anna-parker marked this pull request as ready for review September 22, 2025 11:54
Copy link
Contributor

@corneliusroemer corneliusroemer left a comment

Choose a reason for hiding this comment

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

Would be good to refactor to avoid having X and X_map in config

@anna-parker anna-parker changed the title Add dataset tag map feat(prepro): add dataset tag map Sep 25, 2025
@anna-parker anna-parker changed the title feat(prepro): add dataset tag map feat(prepro): add nextclade_dataset_tag_map Sep 25, 2025
@anna-parker anna-parker merged commit 394daed into main Sep 25, 2025
28 checks passed
@anna-parker anna-parker deleted the add_dataset_tag_map branch September 25, 2025 14:15
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