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.

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

Missing visit end date #127

Open
solmazeradat opened this issue Apr 12, 2024 · 4 comments
Open

Missing visit end date #127

solmazeradat opened this issue Apr 12, 2024 · 4 comments
Assignees

Comments

@solmazeradat
Copy link

Missing visit end date

CDM or THEMIS convention?

THEMIS

Table or Field level?

Field

Is this a general convention?

No

Summary of issues

  • How to handle missing visit_end_date in the VISIT_OCCURRENCE table?
  • How to address the missing visit_end_date with the non-standard concept_id = 32220 for “Still patient”?

Summary of answer

  • Suggested solution:

Visit_end_date is mandatory. Use the best information to infer a visit end date. If end dates are not provided, possible ways to derive them include:

  1. Outpatient Visit: end_date=start_date
  2. Emergency Room Visit: end_date=start_date
  3. Inpatient Visit: usually there is information about discharge. If not you should be able to derive from the sudden decline of activity, or from the absence of inpatient procedures/drugs. But the latter would be not obvious.
  4. Long Term Care Visits: particularly for claims data, if end dates are not provided assume the visit is for the duration of month that it occurs.
  5. For inpatient visits ongoing at the date of ETL, put date of processing the data as mandatory visit_end_date and VISIT_TYPE_CONCEPT_ID with 32220-"Still patient" to identify the visit as incomplete.

On the subject of address the missing visit_end_date with the non-standard concept_id = 32220 for “Still patient”"

... need to update concept_id = 32220 to standard status since only standard concept_ids are allowed in the visit_type_concept_id field.

Related links

Other comments/notes

  • Please use this section to add any other relevant notes or comments. (i.e. if links are not working, something is out of date or deprecated)
@solmazeradat solmazeradat converted this from a draft issue Apr 12, 2024
@solmazeradat solmazeradat moved this from Hunters - In Progress to Hunters - Needs Docs in OMOP CDM & THEMIS Conventions Apr 12, 2024
@MelaniePhilofsky
Copy link
Collaborator

This issue needs to be addressed by Themis. Current guidance (use type_concept_id field to flag an ongoing visit) conflicts with current guidance (end_date is required) and breaks general CDM guidance (*_type_concept_Id fields must contain standard concept_ids). I will move it to the "needs work" column.

@MelaniePhilofsky
Copy link
Collaborator

This issue is for a missing visit_end_date. The "still a patient" issue, is issue OHDSI/CommonDataModel#698 in Themis.

There is already guidance on how to populate the mandatory visit_occurrence.visit_end_date when the source doesn't supply an end date. The following is the heuristic found on the CDM v5.4 website. This needs to be documented as a Themis issue, posted to the Themis convention library and then the wording should be removed from the CDM website.

"If end dates are not provided in the source there are three ways in which to derive them: - Outpatient Visit: visit_end_datetime = visit_start_datetime - Emergency Room Visit: visit_end_datetime = visit_start_datetime - Inpatient Visit: Usually there is information about discharge. If not, you should be able to derive the end date from the sudden decline of activity or from the absence of inpatient procedures/drugs. - Non-hospital institution Visits: Particularly for claims data, if end dates are not provided assume the visit is for the duration of month that it occurs."

@cgreich
Copy link
Contributor

cgreich commented May 10, 2024

Non-hospital institution Visits: Particularly for claims data, if end dates are not provided assume the visit is for the duration of month that it occurs.

Apart from the clumsy English: Is that true? Skilled Nursing Facilities are claimed per month? And if so, shouldn't they then be cobbled together into a lengthy overall stay?

@MelaniePhilofsky MelaniePhilofsky moved this from Ratified to Ready for agenda in Themis Kanban board May 13, 2024
@MelaniePhilofsky
Copy link
Collaborator

Agree the wording needs improvement, @cgreich. Also, if the guidance is inaccurate, we need to correct it. Themis will add it to the agenda for discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Ready for agenda
Development

No branches or pull requests

3 participants