Skip to content

Add schema for unaligned sequences - #2038

Merged
victorlin merged 1 commit into
masterfrom
victorlin/schema-subsample-config-unaligned
Aug 12, 2026
Merged

Add schema for unaligned sequences#2038
victorlin merged 1 commit into
masterfrom
victorlin/schema-subsample-config-unaligned

Conversation

@victorlin

@victorlin victorlin commented Aug 12, 2026

Copy link
Copy Markdown
Member

Description of proposed changes

Some workflows use unaligned sequences as input to augur subsample. In this case, proximal samples will not work properly. This schema makes it easier for workflows to guard against configs with proximal samples.

At run time, augur subsample itself still validates configs against the default schema that allows proximal samples. This is fine because augur proximity errors if sequence lengths differ.

Related issue(s)

Checklist

Some workflows use unaligned sequences as input to augur subsample. In
this case, proximal samples will not work properly. This schema makes it
easier for workflows to guard against configs with proximal samples.

At run time, augur subsample itself still validates configs against the
default schema that allows proximal samples. This is fine because augur
proximity errors if sequence lengths differ.
@victorlin
victorlin force-pushed the victorlin/schema-subsample-config-unaligned branch from 6fbc6ad to f8e00fd Compare August 12, 2026 18:13

@joverlee521 joverlee521 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unfortunate it's not easy to layer JSON schemas for this, but I think having separate schemas here is worth it for workflows!

@victorlin
victorlin merged commit 405c162 into master Aug 12, 2026
29 of 35 checks passed
@victorlin
victorlin deleted the victorlin/schema-subsample-config-unaligned branch August 12, 2026 23:11
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