hl7-eu / pcsp

EU project PanCareSurPass aiming to conduct a robust assessment of the Cancer Survivorship Passport (SurPass)
5 stars 0 forks source link

Radiotherapy Resource #46

Closed davisart closed 2 years ago

davisart commented 2 years ago

1) We need "Not known" for "Code" http://build.fhir.org/ig/hl7-eu/pcsp/ValueSet-vs-radiotherapyType-eu-pcsp.html 2) We don't understand if "Code" in Radiotherapy Boost (http://build.fhir.org/ig/hl7-eu/pcsp/StructureDefinition-Procedure-radiotheraphyBoost-eu-pcsp.html) can have more than one type of radiotherapy (i.e. Boost radiation therapy and External beam radiotherapy). Because 'Boost Radiation Therapy' is a fixed value, we also need to add a 'Radiotherapy Type' field with the following possible values of 'External beam radiotherapy', 'Brachytherapy', or 'unknown'."

gcangioli commented 2 years ago
  1. We need "Not known" for "Code" http://build.fhir.org/ig/hl7-eu/pcsp/ValueSet-vs-radiotherapyType-eu-pcsp.html

Added

gcangioli commented 2 years ago

2. We don't understand if "Code" in Radiotherapy Boost (http://build.fhir.org/ig/hl7-eu/pcsp/StructureDefinition-Procedure-radiotheraphyBoost-eu-pcsp.html) can have more than one type of radiotherapy (i.e. Boost radiation therapy and External beam radiotherapy). Because 'Boost Radiation Therapy' is a fixed value, we also need to add a 'Radiotherapy Type' field with the following possible values of 'External beam radiotherapy', 'Brachytherapy', or 'unknown'."

My understanding is that the boost is always of the same type of the parent radiotherapy. If this assumption is correct the type is obtained from the parent radiotheraphy (i.e. Procedure.partOf.resolve().code).

In this case make more clear the mapping.

@davisart please confirm

gcangioli commented 2 years ago

https://github.com/hl7-eu/pcsp/commit/a936f8385ef4618bd7baa64704428c93da54221f

davisart commented 2 years ago
  1. ok, thank you!
  2. no, for external beam, boost can be: external beam, brachytherapy, NK. For brachytherapy and metabolic radionuclide, boost is missing
gcangioli commented 2 years ago

In this case:

obviously none of them are back compatible changes

davisart commented 2 years ago

for us both solutions are feasible, we prefer the second one, please let us know, thank you!

gcangioli commented 2 years ago

Ok let's go for the second then...

gcangioli commented 2 years ago

https://github.com/hl7-eu/pcsp/commit/d888035fd84d5d30bf587f0f22a0de748fc6a956