Skip to content

TheChristie-Manchester: Add missing T-stage mappings - #51

Open
JHogenboom wants to merge 1 commit into
TheChristie-Manchesterfrom
vibe/add-t-stage-mappings-5164a5
Open

TheChristie-Manchester: Add missing T-stage mappings#51
JHogenboom wants to merge 1 commit into
TheChristie-Manchesterfrom
vibe/add-t-stage-mappings-5164a5

Conversation

@JHogenboom

Copy link
Copy Markdown
Contributor

Summary

  • Add missing T-stage mappings to pathological_t_stage for TheChristie-Manchester branch
  • Include values: T1a, T1b, T2a, T3a, Tis, 0, 1, 2, 3, 4, x

Details

These T-stage values were identified in the Manchester data analysis but were not previously mapped in the schema. All values are added to the existing pathological_t_stage localMappings since there is only one T-stage column in the data.

Note: The user will verify with the hospital whether this should be labeled as clinical_t_stage or pathological_t_stage.

Add the following T-stage values to pathological_t_stage localMappings:
- T1a, T1b (to stage 1)
- T2a (to stage 2)
- T3a (to stage 3)
- Tis (to stage 0)
- 0, 1, 2, 3, 4, x (numeric and x values)

These values were found in the Manchester data analysis but were not
previously mapped in the schema.

Co-authored-by: JHogenboom <JHogenboom@users.noreply.github.com>
@JHogenboom JHogenboom changed the title Add missing T-stage mappings for TheChristie-Manchester TheChristie-Manchester: Add missing T-stage mappings Aug 20, 2026
@CharlieJHeaven

Copy link
Copy Markdown

T stage is pulled from a combined table. It will come primarily from path_stage, if path_stage does not exist then clinical_stage is used. Given this, it will be a combination of clinical and pathological staging.

@JHogenboom

Copy link
Copy Markdown
Contributor Author

@CharlieJHeaven Thanks for clearing that up!
My take is then that it is probably best to map everything to clinical staging, and accept that some of them might be confirmed pathologically?

@CharlieJHeaven

Copy link
Copy Markdown

Yes that sounds reasonable to me.

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.

3 participants