-
Notifications
You must be signed in to change notification settings - Fork 5
Documentation update pass #23
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
28c17bf
to
9fb8339
Compare
9fb8339
to
11dd979
Compare
Note this will need a re- |
This still needs a re- |
Closes #50. |
Looks like
Some potential causes:
|
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.
Not sure how I self-requested review. The doc changes looked good when I looked them over before. The re-document()
revealed an (inconsistent) issue with examples taking a long time / never terminating, maybe due to sometimes-long covid_hosp_facility_lookup
run times; I'm following up on this on Slack. I think it's good to either merge or to wait for some findings regarding what's causing the inconsistent long run times. @keanmingtan would it slow you down at all to continue development off a branch split off from this PR's branch?
Trying out just marking this + another long-running example |
Part of #13.
Testing the API endpoints:
Endpoints tested:
covidcast_nowcast(not an active endpoint yet)