search

ncats / CTSA-Metrics

17 stars 16 forks source link

Create Informatics Metric OMOP Oracle.sql #9

Closed apeshansky closed 6 years ago

apeshansky commented 6 years ago

This is a rewrite of "Informatics Metric OMOP MSSQL.sql" to accommodate both Oracle SQL dialect and CDRN OMOP conventions. The comments describe the conventions difference from OMOP CDM. There are also some proposed changes to data presentation, noted in comments.

apeshansky commented 6 years ago

Please review and merge.

KenGersing commented 6 years ago

Added new file for OMOP with Oracle

KenGersing commented 6 years ago

f

KenGersing commented 6 years ago

Merge Branch

KenGersing commented 6 years ago

The OMOP Oracle Script is posted. Minor change full date of birth changed to only year of birth per request for hubs not collecting DOB

Ken

Ken Gersing, MD [E] Director of Informatics NCATS DCI Email: Kenneth.gersing@NIH.GOVmailto:Kenneth.gersing@NIH.GOV Office: 301-827-2511

From: apeshansky [mailto:notifications@github.com] Sent: Wednesday, January 10, 2018 1:18 PM To: ncats/CTSA-Metrics CTSA-Metrics@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [ncats/CTSA-Metrics] Create Informatics Metric OMOP Oracle.sql (#9)

This is a rewrite of "Informatics Metric OMOP MSSQL.sql" to accommodate both Oracle SQL dialect and CDRN OMOP conventions. The comments describe the conventions difference from OMOP CDM. There are also some proposed changes to data presentation, noted in comments.

You can view, comment on, or merge this pull request online at:

https://github.com/ncats/CTSA-Metrics/pull/9

Commit Summary

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ncats/CTSA-Metrics/pull/9, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ART4qWVvVuFDWSd-oUgsluKRpmUXqAwVks5tJP7LgaJpZM4RZvRU.

apeshansky commented 6 years ago

Did you adjust the comments correspondingly, so that hubs like ours, with one DATETIME_OF_BIRTH column (instead of OMOP's YEAR_OF_BIRTH, MONTH_OF_BIRTH and DAY_OF_BIRTH), still have the code to use in the comment block?

On Thu, Jan 11, 2018 at 8:58 AM, KenGersing notifications@github.com wrote:

The OMOP Oracle Script is posted. Minor change full date of birth changed to only year of birth per request for hubs not collecting DOB

Ken

Ken Gersing, MD [E] Director of Informatics NCATS DCI Email: Kenneth.gersing@NIH.GOVmailto:Kenneth.gersing@NIH.GOV Office: 301-827-2511

From: apeshansky [mailto:notifications@github.com] Sent: Wednesday, January 10, 2018 1:18 PM To: ncats/CTSA-Metrics CTSA-Metrics@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [ncats/CTSA-Metrics] Create Informatics Metric OMOP Oracle.sql (#9)

This is a rewrite of "Informatics Metric OMOP MSSQL.sql" to accommodate both Oracle SQL dialect and CDRN OMOP conventions. The comments describe the conventions difference from OMOP CDM. There are also some proposed changes to data presentation, noted in comments.

You can view, comment on, or merge this pull request online at:

https://github.com/ncats/CTSA-Metrics/pull/9

Commit Summary

  • Create Informatics Metric OMOP Oracle.sql

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/ ncats/CTSA-Metrics/pull/9, or mute the threadhttps://github.com/ notifications/unsubscribe-auth/ART4qWVvVuFDWSd- oUgsluKRpmUXqAwVks5tJP7LgaJpZM4RZvRU.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ncats/CTSA-Metrics/pull/9#issuecomment-356941387, or mute the thread https://github.com/notifications/unsubscribe-auth/AHp_kk_21eAKk-CNLC0-ibk62T4RoMo3ks5tJhOVgaJpZM4RZvRU .

-- Alexandre Peshansky