Skip to content
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

Feedback Doodsoorzaak #6

Open
jitsedc opened this issue Jun 20, 2024 · 5 comments
Open

Feedback Doodsoorzaak #6

jitsedc opened this issue Jun 20, 2024 · 5 comments
Labels
feedback-implementatie Feedback op basis van implementatie question Further information is requested

Comments

@jitsedc
Copy link
Contributor

jitsedc commented Jun 20, 2024

  • Doodsoorzaak.duur | Veld “duur” ontbreekt op doodsoorzaak. Dit moet een Periode zijn (waarde + eenheidcode) zonder begin- en einddatum.
  • Doodsoorzaak.volgtOp | Geen lijst nodig. Cardinaliteit 0..1.
  • Doodsoorzaak.geassocieerdeOorzaak | Cardinaliteit veranderen naar 0..1. Een geassocieerde oorzaak moet ook een duur ondersteunen.
  • Doodsoorzaak.geassocieerdeOorzaak | Geassocieerde oorzaken zijn conceptueel anders dan de doodsoorzaken. Het zou best ook als apart veld beschouwd worden voor de duidelijkheid.
  • Doodsoorzaak.relatie? | Model III D maakt het mogelijk om een doodsoorzaak te linken aan de moeder/foetus/kind. (Bv. doodsoorzaak kind lag aan een aandoening van de moeder.) Dit veld ontbreekt in Doodsoorzaak. Hoe kunnen we dit modelleren?
@jitsedc jitsedc added the feedback-implementatie Feedback op basis van implementatie label Jun 20, 2024
@jitsedc
Copy link
Contributor Author

jitsedc commented Jun 20, 2024

  • duur werd toegevoegd. Periode is echter aangewezen om gebruikt te worden. Hierbij is het echter niet nodig om een begin- of einddatum te geven. De twee waarden binnen een Periode kunnen uren zijn die van elkaar kunnen afgetrokken worden om de duur in uren te krijgen.
  • Kardinaliteit van Doodsoorzaak.volgtOp werd op 0..1 gezet.
  • Geassocieerde Oorzaak is een specialisatie gemaakt van Doodsoorzaak waarbij er kan overgeërfd worden voor attribuering en associaties. Het attribuut van Doodsoorzaak.doodsoorzaak werd gewijzigd naar 'beschrijving' met als verwachte waarde TaalString.
  • Er werd een nieuw attribuut toegevoegd: Doodsoorzaak.gelinktAan:Gezondheidstoestand waarmee kan verwezen worden naar een Gezondheidstoestand van een andere Persoon.

@jitsedc jitsedc added the fixed issues die gefixt zijn in laatste publicatie label Jun 20, 2024
@tdekeyser
Copy link

Volgens mij zit er conceptueel nog iets niet goed tussen de doodsoorzaken en geassocieerde oorzaken. In het formulier zijn geassocieerde oorzaken (sectie II) ziekten die bijgedragen hebben tot de dood, maar niet in verband staan met de logische samenhang van doodsoorzaken (sectie I).

image

Is het haalbaar om dit als twee verschillende velden te modelleren (doodsoorzaken en geassocieerdeOorzaken), waarbij je sectie I modelleert als een linked list van Doodsoorzaak (via volgtOp), en sectie II als een lijst van Doodsoorzaken? Zo hoeft er eigenlijk geen inheritance gebruikt te worden.

Het gelinktAan veld is een mogelijke oplossing, maar dan moet dit wel op Doodsoorzaak staan, aangezien daarvoor ook moet kunnen aangeduid worden wie gelinkt is. Gelinkt aan een Gezondheidstoestand lijkt me daarentegen problematischer: de doodsoorzaak zelf is reeds een soort medische toestand. Ook willen we niet te veel hops introduceren tot de Persoon.

@jitsedc
Copy link
Contributor Author

jitsedc commented Jul 2, 2024

image

Dag Tom, heb ik het zo juist begrepen? De link met Gezondheidstoestand was voor mij wat toekomstgericht denken maar via Overlijdenstoestand>doodsoorzaak heb jij eigenlijk ook al de link. Indien oké zo pas ik het aan. Anders hoor ik graag je feedback.

Eventueel kunnen we er nog een codelijst aan koppelen maar dacht dit expliciet dan de vraag was om niet te doen.

@tdekeyser
Copy link

We hebben hier wel een codelijst aan gekoppeld, dus: gelinktAan: DoodsoorzaakGelinktAan [0..1]. En hierin zit dan [Moeder, Kind, Foetus]. Dit leek ons het makkelijkste om zo weinig mogelijk hops te moeten maken. (We hebben momenteel het "gerelateerdAan" genoemd, maar dat is hetzelfde.)

@jitsedc
Copy link
Contributor Author

jitsedc commented Jul 5, 2024

Heb je liever dat we dan gerelateerdAan:Persoon gebruiken? Dan kan je altijd nog vertakken naar de moeder, kind of foetus via de persoonsrelaties?

@jitsedc jitsedc added question Further information is requested and removed fixed issues die gefixt zijn in laatste publicatie labels Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback-implementatie Feedback op basis van implementatie question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants