icgc-argo / roadmap

Place to review/request new features and new tools on ICGC-ARGO's roadmap
1 stars 0 forks source link

Investigation: changing data type for Specimen.percent_tumour_cells #907

Open Buwujiu opened 2 years ago

Buwujiu commented 2 years ago

The Problem

As part of the clinical exceptions requests, we want to add two text values to Specimen schema's percent_tumour_cells. The new values are:

See complete requirement here: https://wiki.oicr.on.ca/display/icgcargotech/Core+Clinical+Data+Exceptions#CoreClinicalDataExceptions-UserStories

Currently in argo dictionary, percent_tumour_cells only accepts numbers: https://docs.icgc-argo.org/dictionary

Changing the data type will affect downstream services, for instance the clinical index mapping, donor aggregator, lectern, clinical service.

The solution

Investigate what will be affected by the data type change. Questions to start with:

Exit Criteria:

Buwujiu commented 2 years ago

there is a potential change to percent_tumour_cells data type to change to text. If this goes through, there is no need for exception. Talk to @joneubank about potential data migration.

Buwujiu commented 2 years ago

@hknahal to investigate if we can add 2 more fields to describe percent_tumour_cell range. Instead of changing the data type.