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

Covid worked example #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Covid worked example #2

wants to merge 3 commits into from

Conversation

jc-harrison
Copy link
Member

Adds a worked example to demonstrate producing mobility indicators.

Binder link: https://mybinder.org/v2/gh/Flowminder/FlowKit-tutorials-Ghana/worked-example?urlpath=lab/tree/mobility-indicators.ipynb

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

@traceyli
Copy link

I like the content and structure here, but there is a lot of content to digest in one go. I would find the following useful:

  • A 'table of contents' at the start, with each of the sections listed, so that people know what to expect. And may choose to prioritise some sections over others.
  • A clear mapping of which aggregates are used in each QA check, and each mobility indicator. Maybe include this somehow in the table of contents?
  • Explain all acronyms and jargon (e.g. 'QA check', 'pcod') prior to the first usage of the term.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants