Merge in new cohort definitions from Denys

ohdsi-studies / PioneerWatchfulWaiting

This study is part of the joint PIONEER - EHDEN - OHDSI studyathon in March 2021, and aims to advance understanding of clinical management and outcomes of watchful waiting in prostate cancer.

Apache License 2.0

7 stars 18 forks source link

Merge in new cohort definitions from Denys #29

Closed keesvanbochove closed 3 years ago

MaximMoinat commented 3 years ago

We also need to pull in cohort changes from Atlas. Cohorts T2, T2a, and all T3x have been changed. --edit: This is done in a separate branch #45 that has to be merged in this episode branch.

bdemeulder commented 3 years ago

@MaximMoinat @denyskaduk , let me know when I can merge this into episode branch and then merge the episode branch into the master.

MaximMoinat commented 3 years ago

I see my suggested fix has been applied. @denyskaduk Have been able to test the other queries as well? I want to make sure that this issue doesn't occur elsewhere.

denyskaduk commented 3 years ago

Hi @MaximMoinat , unfortunately at the moment I do not have the opportunity to test this code

MaximMoinat commented 3 years ago

Thanks @denyskaduk. Looks good, and I think we can merge @bdemeulder