Gray out early clades by trvrb · Pull Request #1132 · nextstrain/ncov (original) (raw)

@trvrb requested a review from a team

July 27, 2024 01:40

@trvrb trvrb mentioned this pull request

Jul 27, 2024

1 task

huddlej

@trvrb trvrb mentioned this pull request

Sep 26, 2024

victorlin

victorlin

victorlin

@trvrb @victorlin

When assigning colors gray out clades that are not in recent samples. Use clade_recency parameter to consider how many months back to consider "recent".

@trvrb @victorlin

Set up a build-specific override for colors: clade_recency parameter. Make the default to be empty string "" and have this keep all clades colored. Override for the Nextstrain GISAID profile to specify clade_recency of

@trvrb @victorlin

Per rationale in issue 1131 (#1131), switch to following a three step look up for setting build-specific parameters of colors:clade_recency.

@trvrb @victorlin

Rather than previous behavior of setting --clade-recency 6 by building --clade-recency {params.clade_recency}, instead build the entire argument via {params.clade_recency_argument}.

This allows setting colors.global_all-time.clade_recency to "all" in builds.yaml and in this case handing assign-colors.py nothing for clade_recency. This keeps the behavior of assign-colors.py as expected and backwards compatible, while simultaneously allowing for the "all" sensible default in parameters.yaml rather than the previous empty string to accomplish this same behavior.

@trvrb @victorlin

The colors.clade_recency parameter can be specified in a build-specific fashion. If specifying a default for use, use the pattern of specifying colors.default.clade_recency rather than the previous colors.clade_recency.

@trvrb @victorlin

@victorlin

@victorlin

Show a meaningful error message if the value is improperly formatted.

@victorlin

Keeps consistent with other existing usage of this format.

@trvrb

@trvrb trvrb deleted the gray-out-early-clades branch

October 3, 2024 21:11

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})