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.

Sign up for GitHub

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

Jump to bottom

[BUG] [feasibility_assessment] The last_year variable in stratified_person.R being set to 9999 causes age_group to be NA in person_stratified_breakdown table #39

Open
kzollove opened this issue Mar 17, 2023 · 1 comment · May be fixed by #40
Open

[BUG] [feasibility_assessment] The last_year variable in stratified_person.R being set to 9999 causes age_group to be NA in person_stratified_breakdown table #39

kzollove opened this issue Mar 17, 2023 · 1 comment · May be fixed by #40
Labels
bug Something isn't working

Comments

@kzollove
Copy link
Contributor

Describe the bug

The person_stratified_breakdown table's variable age_group has all values set to NA. This is the result of the last_year variable in the stratified_person.R being set to 9999.

The root cause of this is that many CDMs assign the value 9999-01-01 as the end date in the OMOP observation_period table when there is no end date.

To Reproduce

  1. A value of 9999-01-01 in the observation_period table.
  2. Run the stratified_person.R script

Screenshots (If Applicable)

image
@kzollove kzollove added the bug Something isn't working label Mar 17, 2023
kzollove added a commit to kzollove/MentalHealthEquity that referenced this issue Mar 17, 2023
@kzollove
Limit observation_period_end_date to current date
1c8922d 
Fixes ohdsi-studies#39 
By limiting the end_date to current date, age groups are created without NAs
@kzollove kzollove linked a pull request Mar 17, 2023 that will close this issue
Open
4 tasks
@kzollove
Copy link
Contributor Author

The same issue exists in the stratified_person.R associated with the baseline study

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Limit observation_period_end_date to current date kzollove/MentalHealthEquity
1 participant
@kzollove