mrc-ide / epireview

https://mrc-ide.github.io/epireview/
GNU General Public License v3.0
25 stars 2 forks source link

Fix rows with a covidence id being associated with multiple ids #74

Closed sangeetabhatia03 closed 1 month ago

sangeetabhatia03 commented 1 month ago

This PR fixes #56 I have included extensive documentation on why this issue arises. Fixed it here to make things work for existing data sets but going forward I think we should ensure that a covidence_id is not associated with multiple ids in priority-pathogens pipeline.

cm401 commented 1 month ago

Wonderful - this fixes the issues I saw with the Lassa data. (and works fine for Marburg)