search

EHDEN / ETL-UK-Biobank

ETL UK-Biobank
https://ehden.github.io/ETL-UK-Biobank/
12 stars 4 forks source link

OPCS3 Inpatient procedures are mapped twice #337

Closed MaximMoinat closed 2 years ago

MaximMoinat commented 2 years ago

Both from baseline (fields 41256,41258,41273) and from hes_oper.

The data from baseline is derived from HES_OPER, which is more detailed according to portal documentation.

Record-level inpatient data A rawer form of the data is available, which consists of seven interrelated database tables: HESIN, HESIN_DIAG, HESIN_OPER, HESIN_CRITICAL, HESIN_PSYCH, HESIN_MATERNITY and HESIN_DELIVERY.

We should not map operations from the baseline data.

MaximMoinat commented 2 years ago

@vpapez Please verify.

spiros commented 2 years ago

I agree with @MaximMoinat , we should ignore these fields if we are already mapping the contents of the procedure table directly from the HES hes_oper table itself!

MaximMoinat commented 2 years ago

Thanks @spiros for checking. We will add this change to the latest release.