Skip to content

MSCI-Warsaw: Add missing database mappings - #57

Open
JHogenboom wants to merge 1 commit into
MSCI-Warsawfrom
vibe/msci-warsaw-mappings-1787160921
Open

MSCI-Warsaw: Add missing database mappings#57
JHogenboom wants to merge 1 commit into
MSCI-Warsawfrom
vibe/msci-warsaw-mappings-1787160921

Conversation

@JHogenboom

Copy link
Copy Markdown
Contributor

Summary

Add missing MSCI-Warsaw database mappings for treatment, demographic, and bespoke variables based on data analysis.

Changes Made

New Columns Added

  • Treatment variables: immunotherapy, radiotherapy, surgery, chemotherapy - mapped to yes/no
  • Demographic variables:
    • year_of_initial_diagnosis - no localMappings (values are already in correct format)
    • educational_level - mapped Polish terms (Wyzsze → ISCED-5, srednie → ISCED-3)
    • occupational - mapped Polish terms (Tak → working_full_time, unemployed → unemployed)
    • biological_sex - mapped to male/female with M/K and AO01/AO02 codes

Existing Columns Updated

  • Bespoke questions: Added AO05 to unable_to_answer for q2, q3, q5, q85
  • EORTC QLQ-AYA questions: Added N/A to not_applicable for q19, q20, q21

Notes

Verification

  • All new columns use schema terms that exist in the schema valueMapping
  • All localMappings keys match existing schema terms
  • No new schema terms were created

Closes #56

… and bespoke variables

- Add missing columns: immunotherapy, radiotherapy, surgery, chemotherapy
- Add missing columns: year_of_initial_diagnosis, educational_level, occupational, biological_sex
- Update bespoke questions (q2, q3, q5, q85) to include AO05 in unable_to_answer
- Update eortc_qlq_aya questions (q19, q20, q21) to include N/A in not_applicable

Note: Polish terms (Sporo, Tak, Wyzsze, srednie) require clarification from AnnaSikoraKoperska (see issue #56)

Closes #56

Co-authored-by: JHogenboom <JHogenboom@users.noreply.github.com>
@JHogenboom JHogenboom changed the title Add missing MSCI-Warsaw database mappings MSCI-Warsaw: Add missing database mappings Aug 20, 2026
@JHogenboom
JHogenboom marked this pull request as ready for review August 24, 2026 13:25
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.

2 participants