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

Include pipeline version in 'get-released-sequences' response somehow (header or as metadata value) #3209

Open
corneliusroemer opened this issue Nov 13, 2024 · 0 comments
Labels
backend related to the loculus backend component feature Feature proposal

Comments

@corneliusroemer
Copy link
Contributor

corneliusroemer commented Nov 13, 2024

@fengelniederhammer raised a good point that we don't currently emit the pipeline version used to process sequences at all. This isn't great, the only way to work around this right now is to query the db.

We should somehow include the pipeline version in the response of get-released-sequences.

As it's NDJSON we can't put it in an info top-level-key unfortunately.

So we have options to:

  • include in HTTP header
  • emit as metadata field

We should probably just put it as a metadata field, that would also allow us in the future to use different pipeline versions for different organisms, something we've wanted to enable in the future to not have to reprocess everything if a single organism's Nextclade dataset is updated.

Some background discussion: https://loculus.slack.com/archives/C05G172HL6L/p1731500439841509

@corneliusroemer corneliusroemer added backend related to the loculus backend component feature Feature proposal labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend related to the loculus backend component feature Feature proposal
Projects
None yet
Development

No branches or pull requests

1 participant