Skip to content

Conversation

@LilianMinne
Copy link
Contributor

@VGiakoumi deze branch is getest in de Nictiz-GZ repo, maar voor de zekerheid nog even testen in Nictiz-Testscripts voordat deze gemerged gaat worden naar de patch release branch. Het uploaden liep vast bij mij, dus ik ga dit helaas niet meer op tijd redden. Wil jij de links naar de uitgevoerde testscripts toevoegen in deze PR?

@VGiakoumi
Copy link
Contributor

Touchstone tests:
load: https://touchstone.aegis.net/touchstone/execution?exec=202502170955186592039577
PHR: https://touchstone.aegis.net/touchstone/execution?exec=202502170958073044188812
XIS: https://touchstone.aegis.net/touchstone/execution?exec=202502170959339885811414
op de XIS tests is nog wel een error, in scenario 1.4, test 5a, serve observations.
Twee terminologie fouten:
`
Validation of response body against profile 'http://hl7.org/fhir/StructureDefinition/Bundle' by FHIR specification's Validation Engine produced the following results:

ERROR: (TerminologyEngine) Unknown code '3278292003' in the CodeSystem 'http://snomed.info/sct' version 'http://snomed.info/sct/11000146104/version/20240930'. Location: Bundle.entry[2].resource/*Observation/46aa440f-2064-11ec-5265-020000000000*/.method.coding[0].code (line 227, col 17).

ERROR: (TerminologyEngine) Unknown code 'k[arb'U]/L' in the CodeSystem 'http://unitsofmeasure.org' version '2.2'. Location: Bundle.entry[15].resource/*Observation/a49071d0-2291-11ec-8669-020000000000*/.value.ofType(Quantity).code (line 1387, col 24).

`
De fout met k[arb'U] is bekend, de andere fout herken ik niet, nader uit te zoeken

@VGiakoumi
Copy link
Contributor

Wat de terminologiefout betreft; inderdaad 3278292003 bestaat niet als code (ook niet als je zoekt naar active en inactive codes in de SNOMED browser), maar 278292003 |echografie (kwalificatiewaarde)| bestaat wel, met alternatieve naam 'echografische beeldvorming'. Ik zal kijken of ik dit kan corrigeren.

@VGiakoumi
Copy link
Contributor

VGiakoumi commented Feb 17, 2025

zie: PR#110.
Edit: de wijziging meegenomen in de patchrelease via deze PR betreft echter niet de fixture van de foutmelding, de error heeft betrekking op medmij-gbz-bc-PregnancyObservation65147003-46aa440f-2064-11ec-5265-020000000000.xml uit Vrouw2-Kaart3-1elijn

@VGiakoumi
Copy link
Contributor

VGiakoumi commented Feb 17, 2025

De error wordt verholpen als er in fixture Vrouw2-Kaart3-1elijn\medmij-gbz-bc-PregnancyObservation65147003-46aa440f-2064-11ec-5265-020000000000.xml in plaats van "echografische beeldvorming", gebruik wordt gemaakt van <display value="echografie (kwalificatiewaarde)" />, dus de primaire concept ID uit SNOMED.

Dan blijft alleen de k[arb'U] error over.

load: TS link
PHR: TS link
XIS: TS link

@pieter-edelman-nictiz pieter-edelman-nictiz self-assigned this Feb 20, 2025
@VGiakoumi
Copy link
Contributor

VGiakoumi commented Feb 20, 2025

..een andere oplossing is bij Vrouw2-Kaart3-1elijn\medmij-gbz-bc-PregnancyObservation65147003-46aa440f-2064-11ec-5265-020000000000.xml om <display value="echografische beeldvorming (kwalificatiewaarde)" /> te gebruiken.
Dit zorgt er ook voor dat de eerste foutmelding er niet is - wat eerder ontbrak was (kwalificatiewaarde)
Misschien is deze optie zelfs een geschiktere oplossing, aangezien de ValueSet (link) juist deze display verwacht.
Dan zou ik dezelfde waarde ook wijzigen in fixture Vrouw2-Kaart3-2elijn\medmij-gbz-bc-PregnancyObservation65147003-a4907acc-2291-11ec-8669-020000000000.xml , voor uniformiteit (in die tweede fixture stond nu echografie (kwalificatiewaarde) )

load: https://touchstone.aegis.net/touchstone/execution?exec=202502200603474117083926
PHR: https://touchstone.aegis.net/touchstone/execution?exec=202502200607572211650526
XIS: https://touchstone.aegis.net/touchstone/execution?exec=202502200609208939402933

alleen de fout met k[arb'U] zie ik tussen de XIS testen

@pieter-edelman-nictiz pieter-edelman-nictiz merged commit 74027fe into patchrelease-february-2025 Feb 21, 2025
1 check passed
@pieter-edelman-nictiz pieter-edelman-nictiz deleted the release/medmij-patch-1.0.6 branch February 21, 2025 07:07
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.

4 participants