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

Port merge-user-metadata to augur curate #1495

Closed joverlee521 closed 3 months ago

joverlee521 commented 3 months ago

Description of proposed changes

Porting over merge-user-metadata from nextstrain/ingest to augur curate apply-record-annotations.

Includes a functional change to skip annotations for fields that do not exist in the record because it's hard to ensure that the new field gets added across all records that are streamed through the command.

This is a deviation from the original nextstrain/ingest script, but I think a necessary one to avoid bugs in the final output metadata.

Related issue(s)

Resolves https://github.com/nextstrain/augur/issues/1482

Checklist

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.28%. Comparing base (0adfe3d) to head (8ee9d82).

:exclamation: Current head 8ee9d82 differs from pull request most recent head 63744c6

Please upload reports for the commit 63744c6 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1495 +/- ## ========================================== + Coverage 69.15% 69.28% +0.13% ========================================== Files 70 71 +1 Lines 7694 7727 +33 Branches 1887 1895 +8 ========================================== + Hits 5321 5354 +33 Misses 2087 2087 Partials 286 286 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.