Include 2m timespan in Nextstrain GISAID and open profiles #957
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.
Description of proposed changes
This PR extends the previous logic of splitting out
6m
andall-time
timespans from PR #910 to include a new2m
timespan.2m
was chosen over1m
to allow "logistic growth" calculation using the previous 6 weeks of frequencies pivots to work without modification.With this narrow of timespans there is some unavoidable funny interaction with how
augur filter
subsamples based on--vpm
, ie viruses per month. We have common situations where if current date is say May 15 we end up withaugur filter
to equally sample viruses from March, April and May categoriesso that March and May have 2 weeks for sampling of X viruses and April has 4 weeks for sampling of X viruses. This results in more densely sampled, in terms of viruses per day, months of March and May compared to April.
This effect will be more pronounced in scenarios where current date is, say, May 28, and so X viruses are sampled in 3 days in March and 30 days in April.
To fully address this we'd need to extend
augur filter
to have the option of per-week sampling categories in addition to per-month sampling categories. Or perhaps some continuous specification. However, I don't think this is too big of an issue in terms of the current PR and it's something we can refine once Augur is updated.cc @victorlin @huddlej for Augur issue: nextstrain/augur#960
Testing
Trial builds are available at:
Release checklist
docs/src/reference/change_log.md
in this pull request to document these changes by the date they were added.After merging of this PR, we should:
manifest_guest.json
to allowing viewing of2m
datasets