-
Notifications
You must be signed in to change notification settings - Fork 7
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
Manaus parser for SRAG data #1755
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good.
Codecov Report
@@ Coverage Diff @@
## main #1755 +/- ##
===========================================
+ Coverage 67.54% 87.33% +19.79%
===========================================
Files 127 32 -95
Lines 4313 916 -3397
Branches 1147 190 -957
===========================================
- Hits 2913 800 -2113
+ Misses 1400 116 -1284
Continue to review full report at Codecov.
|
@Mougk @fojackson8 Re-wrote this to allow for ingestion of all data, not just from Manaus. Note I've only tested it locally as e2e not set up yet. |
* Manaus parser for SRAG data * parsing.brazil_manaus.manaus: Use newer Batch functions * Changes to parser to allow parsing of data from all Brazil * Fix parsing_lib import in Manaus parser Co-authored-by: Abhishek Dasgupta <abhishek.dasgupta@cs.ox.ac.uk>
First look at parser which takes Manaus cases in SRAG data.
Please note this is slightly 'bare-bones', i.e. I have included essential data but there is a fair amount more that could go into 'notes'.