Skip to content

Conversation

@rumackaaron
Copy link
Contributor

Describes cmu-delphi/covidcast-indicators#1289 in the docs.

Copy link
Contributor

@capnrefsmmat capnrefsmmat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks okay (one suggestion), but @krivard given the additional issues discussed on Slack yesterday, does this need to be more expansive? It can cover the temporary halt and serve as a reference for any email to the users list

@krivard krivard requested a review from capnrefsmmat October 8, 2021 21:36
@krivard
Copy link
Contributor

krivard commented Oct 8, 2021

@capnrefsmmat good idea -- amended to include the same details I sent out to the list.

Copy link
Contributor

@capnrefsmmat capnrefsmmat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some additional suggestions

krivard and others added 2 commits October 11, 2021 08:58
Co-authored-by: Alex Reinhart <[email protected]>
Copy link
Contributor

@krivard krivard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@krivard krivard merged commit 98561e9 into dev Oct 11, 2021
@krivard krivard deleted the chng_backfill_estimates branch October 11, 2021 21:28
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.

4 participants