opensafely / os-sch-children-2021

update of risk of covid outcome associated with living with kids
MIT License
0 stars 2 forks source link

Nit: Index Date #7

Open CarolineMorton opened 3 years ago

CarolineMorton commented 3 years ago

Thoughtout study_definition,py the date used is 2020-12-20. This is in multiple places in the code. It would be better to take advantage of index_dates where you set the date once at the top of your study definition and then refer to it as index_date. This has the advantage that if you want to change the dates in future, you can just change it in one place.

hjforbes commented 3 years ago

Good tip! Thanks