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

fix api blog error #253

Closed
sgratzl opened this issue Mar 5, 2021 · 4 comments · Fixed by #303
Closed

fix api blog error #253

sgratzl opened this issue Mar 5, 2021 · 4 comments · Fixed by #303
Assignees
Labels

Comments

@sgratzl
Copy link
Member

sgratzl commented Mar 5, 2021

see https://github.com/cmu-delphi/www-main/runs/2040846791?check_suite_focus=true

@sgratzl
Copy link
Member Author

sgratzl commented Mar 8, 2021

I assume it is because:

Fetching covid_ag_raw_pct_positive from quidel for 2020-08-01 to 2020-08-01 in geography 'ny': no results

@krivard can you update the code to work again?

related code chunk:

```{r q-versioning, warning=FALSE, message=FALSE, cache=TRUE}
library(covidcast)
library(dplyr)
query_date <- "2020-08-01"
covidcast_signal(
data_source = "quidel",
signal = "covid_ag_raw_pct_positive",
start_day = query_date,
end_day = query_date,
geo_type = "state",
geo_value = "ny",
issues = c(query_date, "2020-09-04")) %>%
select(time_value, value, sample_size, issue, lag) %>%
distinct(value, .keep_all=TRUE) %>%
knitr::kable("html", digits = 2,
col.names = c("Test date", "Positivity rate (%)", "Sample size",
"Issued on", "Lag (days)"))
```

@sgratzl sgratzl added the bug label Mar 8, 2021
@krivard
Copy link
Contributor

krivard commented Mar 8, 2021

This is probably due to BIGINT -- the only issues we have for 2020 are the latest issue for each day, so if that falls outside of the queried range it won't return anything.

We're still looking for a way to restore the 2020 data history that will complete before May but doesn't require taking down COVIDcast for 2 days.

@sgratzl
Copy link
Member Author

sgratzl commented Mar 30, 2021

are there any workaround in which we could change the blog post? it is getting annoying that half my CI build fail

@sgratzl
Copy link
Member Author

sgratzl commented Mar 31, 2021

see also cmu-delphi/covidcast#519

#295 is disabling the problematic line for now. However, I won't update the rendered HTML page. Thus, the published blog looks the same. Once either the big int issue has been resolved we can enable it again

@sgratzl sgratzl self-assigned this Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants