Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report names of metadata duplicates in filter #1008

Merged
merged 3 commits into from
Jul 28, 2022

Conversation

huddlej
Copy link
Contributor

@huddlej huddlej commented Jul 22, 2022

Description of proposed changes

Reports the names of duplicate strains as part of the error message when augur filter finds duplicates in the metadata, so the user knows where to start debugging their data.

Related issue(s)

Related to #918

Testing

  • Tested by CI

@codecov
Copy link

codecov bot commented Jul 23, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.68%. Comparing base (e623dd8) to head (7b48451).
Report is 1281 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1008       +/-   ##
===========================================
+ Coverage        0   59.68%   +59.68%     
===========================================
  Files           0       53       +53     
  Lines           0     6320     +6320     
  Branches        0     1586     +1586     
===========================================
+ Hits            0     3772     +3772     
- Misses          0     2285     +2285     
- Partials        0      263      +263     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@joverlee521 joverlee521 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Member

@victorlin victorlin left a comment

Choose a reason for hiding this comment

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

Looks good! Could you add a changelog entry before merging?

Reports the names of duplicate strains as part of the error message when
augur filter finds duplicates in the metadata, so the user knows where
to start debugging their data.
@huddlej huddlej force-pushed the report-duplicate-metadata-in-filter branch from 8134299 to 7b48451 Compare July 28, 2022 22:56
@huddlej huddlej merged commit 421a581 into master Jul 28, 2022
@huddlej huddlej deleted the report-duplicate-metadata-in-filter branch July 28, 2022 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

3 participants