search

nextstrain / augur

Pipeline components for real-time phylodynamic analysis
https://docs.nextstrain.org/projects/augur/
GNU Affero General Public License v3.0
268 stars 129 forks source link

augur curate format-dates should mask empty date fields #1507

Closed joverlee521 closed 3 months ago

joverlee521 commented 3 months ago

Originally brought up by @emmahodcroft on Slack.

The command currently skips fields that have falsy values, so it ends up skipping empty date fields.

https://github.com/nextstrain/augur/blob/f6ee377336ec3813468fe641fa7910c14e54ced3/augur/curate/format_dates.py#L169-L173

The correct behavior should be to mask empty values as XXXX-XX-XX.