-
Notifications
You must be signed in to change notification settings - Fork 7
Added first Flash blog post #908
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
Conversation
✅ Deploy Preview for cmu-delphi-main ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Yet, as data volumes continue to grow quickly (Delphi's data volume expanded 1000x in just 3 years), it is infeasible for data reviewers to inspect every one of these data points for subtle changes in | ||
* quality (like those resulting from data delays) or | ||
* disease dynamics (like an outbreak). | ||
These issues, if undetected, can have critical downstream impacts on data users (as shown by the example in Fig 1). |
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.
These issues, if undetected, can have critical downstream impacts on data users (as shown by the example in Fig 1). | |
These issues, if undetected, can have critical downstream impact on data users (as shown by the example in Fig 1). |
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.
I think your suggestion makes sense but I'm not sure how to capture that there could be multiple impacts - maybe it already does, but please let me know what you think!
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.
i think "impact" functions as a mass noun here, and its plurality is further implied by the mention of multiple "issues".
if you want something explicitly plural, maybe you could swap in something like "ramifications" or "effects"?
idk, english is stupid sometimes ¯\_(ツ)_/¯
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.
That makes sense - just changed it to ramifications like you suggested!
Co-authored-by: melange396 <george.haff@gmail.com>
Co-authored-by: melange396 <george.haff@gmail.com>
Co-authored-by: melange396 <george.haff@gmail.com>
Thank you so much, George for those corrections and formatting - I think the bullet points look good now! Just working on trying to get the authors to render correctly now ! |
Those corrections should be taken care of now as per the preview! |
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.
one more suggestion, plus i noticed that Richa and Tina don't have "description" blurbs at the bottom of the article... do you wanna add something for them like you did for Tara in #904?
Otherwise, i think this is good to go!
Co-authored-by: melange396 <george.haff@gmail.com>
Great catch- just added in those descriptions! |
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 good!
This post introduces the team and will be part of a series (likely 4 more parts right now- one for the univariate method (IJCAI paper) one for the volume correction method (AAAI paper) one for engineering challenges from the start until the dashboard, covering the VIS paper, and one on how to use the public-facing alerts (once it has been built out).
@nolangormley @richagadgil