Skip to content

feat(config, website): multi-seg, multi-ref searchUI mockup#5936

Draft
anna-parker wants to merge 32 commits intoadd_countfrom
search_ui_demo
Draft

feat(config, website): multi-seg, multi-ref searchUI mockup#5936
anna-parker wants to merge 32 commits intoadd_countfrom
search_ui_demo

Conversation

@anna-parker
Copy link
Contributor

@anna-parker anna-parker commented Feb 4, 2026

resolves #5842, #5945

Screenshot

Screen.Recording.2026-02-06.at.19.25.32.mov

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

@claude claude bot added deployment Code changes targetting the deployment infrastructure website Tasks related to the web application config Configuration related issues, i.e. helm processing labels Feb 4, 2026
@claude
Copy link
Contributor

claude bot commented Feb 4, 2026

This PR may be related to: #5566 (feature: add option to hide metadata fields from web interface)

@anna-parker anna-parker added the preview Triggers a deployment to argocd label Feb 4, 2026
@anna-parker
Copy link
Contributor Author

anna-parker commented Feb 5, 2026

Comments from @rneher and @theosanderson in https://loculus.slack.com/archives/C094HTT8XEJ/p1770225352248569 (I made issues only for the more complicated sections:

Other issues:

  • Make aligned sequence download segment-dependent (now need to select references for all to download any sequence)
  • Fix integration tests

@anna-parker anna-parker changed the base branch from calculate_gca_groups to add_count February 5, 2026 10:31
@anna-parker anna-parker changed the title feat(config): add new metadataScope param feat(config, website): multi-seg, multi-ref searchUI mockup Feb 6, 2026
@anna-parker anna-parker removed the preview Triggers a deployment to argocd label Feb 6, 2026
inputs: {input: nextclade.clade}
- <<: *evMetadataAdd
name: clade_ev_d68
metadataScope: sequence
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We should slim the config a bit:

When we have

metadataScope: sequence
sequenceMetadataScope: L

preprocessing should get the arg

segment: L

Additionally if we have

metadataScope: sequence
sequenceMetadataScope: L
onlyForReference: ref1

preprocessing should get the args

segment: L
reference: ref1

The prepro args are added by the template when

perSegment: true

we should also add them automatically in the cases above to reduce to amount of boilerpoint config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Configuration related issues, i.e. helm processing deployment Code changes targetting the deployment infrastructure website Tasks related to the web application

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant