Skip to content

Conversation

@nmdefries
Copy link
Contributor

Summary

Update dates from #674 after backfilling indicators to Wave 11 release.

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.

So the API signals say May 20 and the contingency tables say May 16; is that just because how the weeks fall?

One question and then I think this is good to go

Comment on lines -69 to -70
CSVs for the month of January 2021 only use data from January 6-31 due to a
[definitional change in a major vaccine item on January 6](./coding.md#new-items-2).
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this no longer true?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's correct.

Display logic for V3 was changed on Jan 6, mid-wave 6, to be shown only to respondents indicating that they had not been vaccinated. For the purposes of the contingency tables, we ignore responses to V3 from before the change. During the initial runs we enforced this by only including input data for Jan 6 or later, resulting in non-V3-derived indicators also only using data after Jan 6, hence the note in the docs.

The logic to ignore pre-Jan 6 V3 data is now included in the variable definitions, so I backfilled to the beginning of January so we don't have any partial weeks or months.

Copy link
Contributor

Choose a reason for hiding this comment

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

Makes sense, thanks.

@nmdefries
Copy link
Contributor Author

So the API signals say May 20 and the contingency tables say May 16; is that just because how the weeks fall?

Yep.

@krivard krivard merged commit a8138c6 into dev Sep 9, 2021
@krivard krivard deleted the update-wave11-indicators-dates branch September 9, 2021 14:23
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