Skip to content

docs(olids): add Compass Equivalent column to schema docs#92

Open
EddieDavison92 wants to merge 2 commits into
mainfrom
docs/add-compass-equivalents
Open

docs(olids): add Compass Equivalent column to schema docs#92
EddieDavison92 wants to merge 2 commits into
mainfrom
docs/add-compass-equivalents

Conversation

@EddieDavison92

@EddieDavison92 EddieDavison92 commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

What

Adds a Compass Equivalent column to each table in OLIDS/Documentation/Schema/*.md (22 files), restoring the mappings that were present in the archived Synapse schema dictionary but omitted from the new dbt schema docs in #89.

How

  • Values are taken from Synapse-Platform-Archive/OLIDS-schema-dictionary.md, matched by column name per table. Compass itself has not changed, so rows carry the new OLIDS column name with the old Compass value.
  • Where columns were renamed in the rewrite, mappings were carried across manually where the equivalence is unambiguous (e.g. SUBTYPE <- SUB_TYPE, RECORDED_DATE <- DATE_RECORDED, USUAL_GP_PRACTITIONER_IN_ROLE_ID <- CARE_MANAGER_PRACTITIONER_IN_ROLE_ID, plus two archive typos).
  • Columns new to the dbt build (SOURCE_EXTRACTION_DATE, LDS_TRANSFORM_DATETIME, SLOT_ID, etc.) show --, following the archive convention.
  • Every mapping was then validated against the actual Compass source schema. A handful of archive values referenced columns that do not exist in Compass and were corrected: result units columns are result_value_units (not a concept id), supplier codings are non_core_concept_id (not raw_concept_id), organisation/organization spelling on two encounter columns, and Compass has no age_at_event_baby/age_at_event_neonate, birth date part, clinical_status or referral specialty columns (now --). APPOINTMENT_PRACTITIONER.APPOINTMENT_ID had an evident copy-paste value in the archive and is --.

Notes

Four table rows in Patient_Address.md and Patient_Contact.md had fewer cells than their headers; they are padded so the new column aligns (also fixes four pre-existing MD056 lint errors). No other pre-existing content is modified, and markdownlint reports no new issues.

Restores the Compass mappings from the archived Synapse schema
dictionary, matched to the renamed dbt/Snowflake columns. Pads four
ragged table rows so the new column aligns.
Corrects values carried over from the archived dictionary that do not
exist in the Compass schema: result units columns are
result_value_units (not a concept id), supplier codings are
non_core_concept_id (not raw_concept_id), and Compass has no
age_at_event_baby/neonate, birth date part, clinical_status or
referral specialty columns. Fixes organization spelling for two
encounter columns.
@EddieDavison92

Copy link
Copy Markdown
Collaborator Author

Heads up: after raising this I cross-checked every mapping against the actual Compass source schema and pushed a correction commit - the archived dictionary itself had a few values that don't exist in Compass (details in the description). All remaining mappings now validate against real Compass columns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant