-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support multiple field lookups based on year #97
Comments
As a frame of reference, here is what {
"key": "grade_clin",
"name": "Grade Clinical",
"naaccr_item": 3843,
"naaccr_xml_id": "gradeClinical",
"default": "9",
"table": "grade_clinical_standard_non_ajcc_32473",
"used_for_staging": false,
"metadata": [
{ "name": "COC_REQUIRED" },
{ "name": "NPCR_REQUIRED" },
{ "name": "SSDI" },
{ "name": "CCCR_REQUIRED" },
{ "name": "SEER_REQUIRED" }
]
} It makes a big difference whether any of these fields that will require multiple tables based on year are ever used in the staging process. This one is an SSDI that is not used in staging. If we can assume that the fields are only non-staging fields, we could add a {
"key": "grade_clin",
"name": "Grade Clinical",
"naaccr_item": 3843,
"naaccr_xml_id": "gradeClinical",
"default": "9",
"table": "grade_clinical_standard_lookup_2018_2024",
"used_for_staging": false,
"metadata": [
{
"name": "COC_REQUIRED",
"start": 2018,
"end": 2024
},
{
"name": "COC_REQUIRED",
"start": 2025,
"table": "grade_clinical_standard_lookup_2025"
},
{ "name": "NPCR_REQUIRED" },
{ "name": "SSDI" },
{ "name": "CCCR_REQUIRED" },
{
"name": "SEER_REQUIRED",
"start": 2018,
"end": 2024
},
{
"name": "SEER_REQUIRED",
"start": 2025,
"table": "grade_clinical_standard_lookup_2025"
}
]
} @schusslern thoughts? |
yes, grade is sometimes used in staging. The summary table uses the superset of grade values, and since they aren't adding new potential values, it should not have any issues with the possible values vs values in the calculation table. regarding your potential solution, i could live with that. will that mean we would have to adjust all the existing meta data? since they are still voting on what to do, i wouldn't do anything right now, but if you could give me an idea if this will be time consuming? |
The good thing about this solution is that it would only require a single change to the JSON structure. I would add Of course applications, like SEER*DMS will need to be adjusted to show the correct lookup to the users. However I do not think that will be a lot of work. Helios would need to support the new field as well but I assume that would not be much work. |
that sounds like a good working theory. i'll let you know once they start making some decisions. |
From @schusslern
So right now there are 4 grade fields defined as SSDIs
(Grade Clin, Grade Path, Grade Post Tx Clin, Grade Post Tx Path)
They are discussing changing the lookups in 2025 and adding a new grade (Grade 2025).
They will then have:
So while for any given field I can say any one of these things:
What I can’t do currently is
Because these are the same field.
I don’t want to make an entirely new EOD library (Where 2018-2024 is in EOD 2018; and the 2025+ is in EOD 2018b) and I don’t want to make a second set of schemas for all schemas so these 4 fields can have 2 different lookups.
As you can see, we have a year before we need to be able to do this, if they do in fact choose to NOT convert the old values into the new coding system.
I think they should convert it, personally, but I’m not sure I’ll be able to convince them. So I want to think about the best way to do this now so if we need to make changes to the Helios, API, etc; we have a plan.
The text was updated successfully, but these errors were encountered: