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

Toelichting niet overal mogelijk waar die verwacht wordt #199

Closed
RuudKathmann opened this issue Dec 13, 2021 · 12 comments
Closed

Toelichting niet overal mogelijk waar die verwacht wordt #199

RuudKathmann opened this issue Dec 13, 2021 · 12 comments
Assignees
Labels
1.1.1 Issue meenemen in versie 1.1.1 prio2 Belangrijk, maar niet urgent verwerkt (geheel) z-issue PATCH-versie: achterwaarts compatible bugfix

Comments

@RuudKathmann
Copy link

Bijvoorbeeld het metagegeven “Toelichting” staat als volgt gedefinieerd:

image

Nu willen wij binnen het IMWOZ keuzes zowel voorzien van een definitie als een toelichting. Volgens de beschreven toepassing zou dat moeten kunnen.

Maar bij de opsomming van de tagged values van Keuze staat wel de definitie, maar niet de toelichting.

image

We kunnen onze toelichtingen dus niet kwijt.

Datzelfde probleem hebben we bij relaties:

image

image

Het zou mooi zijn als we op meer plaatsen een toelichting zouden kunnen geven.

@lennartvanbergen
Copy link
Collaborator

Toelichting is in principe bedoeld op alle plekken waar een definitie ook aan de orde is. Mogelijk dat dit ergens vergeten is om door te voeren. Je kan altijd met de hand overigens een tagged value Toelichting toevoegen, als tijdelijke workaround.

@lennartvanbergen
Copy link
Collaborator

lennartvanbergen commented Dec 14, 2021

Verwacht wordt:
per modelelement waarbij de betekenis van data wordt gedefinieerd:

  • een definitie
  • een toelichting op de definitie
  • een herkomst van de definitie

(herkomst zelf heeft meer te maken met een domein aanduiding en niet met de betekenis van data dus deze laat ik hier even buiten beschouwing)

Modelelementen met een definitie, maar zonder een toelichting:

3.2.2.1 Relatiesoort is leidend (alternatief 1)
Relatierol

  • toelichting
  • herkomst definitie

3.2.2.2 Relatierol is leidend (alternatief 2)
Relatiesoort

  • toelichting
  • herkomst definitie

Relatierol

  • toelichting

Relatieklasse

  • toelichting
  • herkomst definitie

Externe koppeling

  • definitie
  • toelichting
  • herkomst definitie

Referentielijst

  • herkomst definitie

Referentie element

  • herkomst definitie

Codelijst --> heet nog codelist, moet codelijst zijn

  • toelichting
  • herkomst definitie

Primitief datatype, gestructureerd datatype, data element, keuze,

  • toelichting
  • herkomst definitie

Enumeratie

  • herkomst definitie
  • toelichting

Enumeratiewaarde

  • herkomst definitie
  • toelichting

--
Bij packages heb ik eigenlijk het idee dat definitie hier niet de juiste match is. Liever zie ik hier alleen: omschrijving.

Voor een z-versie kunnen we dit niet gaan aanpassen. Ik stel voor dat we een keuze maken tussen:

  • nu even niks aanpassen --> hier kiezen we voor
  • vanwege consistentie hier definitie laten staan en toelichting en herkomst definitie en herkomst verwachten
  • we hier omschrijving en herkomst verwachten en dit toevoegen en als tip geven: gebruik de definitie hiervoor maar niet meer.

@lennartvanbergen lennartvanbergen added 1.1.1 Issue meenemen in versie 1.1.1 prio2 Belangrijk, maar niet urgent voorstel oplossing z-issue PATCH-versie: achterwaarts compatible bugfix labels Dec 14, 2021
@lennartvanbergen lennartvanbergen self-assigned this Dec 14, 2021
@lennartvanbergen lennartvanbergen added this to Te beoordelen in MIM Versie 1.2 via automation Dec 14, 2021
@lennartvanbergen lennartvanbergen moved this from Te beoordelen to Review in MIM Versie 1.2 Dec 14, 2021
@lennartvanbergen
Copy link
Collaborator

@PalmJanssen eens? Dan komt dit er ook bij in het diagram van de metadata gegevens.

@PalmJanssen
Copy link
Contributor

Lijkt me prima. Omdat Imvertor een standaard catalogus genereert kan met de Tagged Value Toelichting bij al deze elementen het gebruik worden toegelicht.

@PalmJanssen
Copy link
Contributor

Proces informatie:

  • toegevoegd aan UML diagrammen

@ArjanLoeffen
Copy link

ArjanLoeffen commented Jan 4, 2022

Toelichting komt dus op alle modelelementen hieronder, net zoals deze Definitie hebben, maar overal optioneel: Graag jullie okay zodat we dat kunnen overnemen in Imvertor OS.

  • Objecttype
  • Attribuutsoort
  • Gegevensgroeptype
  • Relatiesoort
  • Referentielijst
  • Referentie element
  • Codelijst
  • Gegevensgroep
  • Relatierol
  • Relatieklasse
  • Enumeratie
  • Primitief datatype
  • Gestructureerd datatype
  • Keuze
  • Data element
  • Datatype
  • Enumeratiewaarde
  • Externe koppeling (toegevoegd nav. @PalmJanssen)

@RuudKathmann
Copy link
Author

Lijkt mij een mooie oplossing.

Wat ons betreft Okay

@PalmJanssen
Copy link
Contributor

Toelichting komt dus op alle modelelementen hieronder, net zoals deze Definitie hebben, maar overal optioneel: Graag jullie okay zodat we dat kunnen overnemen in Imvertor OS.

Hier wordt aan gewerkt. Je vergeet nog externe koppeling

@lennartvanbergen
Copy link
Collaborator

lennartvanbergen commented Jan 6, 2022

@ArjanLoeffen niet op alles wat een definitie heeft, voor packages namelijk niet. De rest wel inderdaad.

Zie ook #205

Maar ik kan er goed mee leven om het ook daar wel te doen ...

lennartvanbergen pushed a commit that referenced this issue Jan 6, 2022
Verwerking van #199 in H2 Algemeen.
lennartvanbergen pushed a commit that referenced this issue Jan 6, 2022
Vanwege #199

Voor alle modelelementen 
| **Naam**            | 1            | Algemeen metagegeven. | *name van de metaclass Named element* |    | *Name*         |     |
| **Alias**            | 0..1         | Algemeen metagegeven. | *UML-Property*                        |    | *Alias*        |     |
| **Herkomst**         | 1            | Algemeen metagegeven. |                                       |    | *tagged value* |     |
| **Begrip**           | 0..\*        | Algemeen metagegeven. |                                       |    | *Tagged value* |     |
| **Definitie**       | 1            | Algemeen metagegeven. | *Body van de metaclass Comment*       |    | *Notes*        |     |
| **Herkomst definitie** | 1         | Algemeen metagegeven. |                                       |    | *Tagged value* |     |
| **Toelichting**     | 0..1         | Algemeen metagegeven. |                                       |    | *Tagged value* |     |
| **Datum opname**     | 1            | Algemeen metagegeven. |                                       |    | *Tagged value* |     |

Deze zijn optioneel voor alle datatypen en keuze, behalve voor waardelijsten.
lennartvanbergen pushed a commit that referenced this issue Jan 6, 2022
#199
Toelichting en definitie wat aangescherpt ook.
@ArjanLoeffen
Copy link

Ik heb nu twee dingen doorgevoerd.

1/
In de MIM 1.1 standaard is nu de toelichting toegestaan op alle elementen zoals eerder in dit issue beschreven. Nergens verplicht.

2/
In MIM 1.1 documentatie is de toelichting op alle elementen getoond waar ook een definitie kan worden getoond.

Achtergrond:
De documentatie voor Waarderingskamer volgt een "standaard" MIM 1.1 documentatievorm. Deze is ook onderdeel gemaakt van MIM 1.1 configuratie. Dit is meer een "afspraak" die we maken mbt. hoe je MIM 1.1 modellen in documentatie kunt omzetten.

In deze documentatievorm heb ik nu te toelichting opgenomen op alle constructies die ook een definitie kunnen hebben. dat kunnen we fijner uitwerken, maar daarmee is nu de WK geholpen. Als iemand ergens géén toelichting opneemt wordt deze ook niet getoond (het veld is ook niet "leeg") in de documentatie.

@architolk
Copy link
Contributor

Nog wel een klein dingetje: dit issue lijkt iets meer op te pakken dan alleen toelichting waar ook een definitie was. Er zijn (lijken?) ook definities bij te zijn gekomen waar ze er eerst niet waren. Aangezien definitie (normaal gesproken) verplicht is, lijkt me dat een breaking change? Ik zou voorstellen om het daar OF niet te doen, OF overal optioneel. Zelfde geldt overigens ook voor herkomst en herkomst-definitie

@ArjanLoeffen
Copy link

Nog wel een klein dingetje: dit issue lijkt iets meer op te pakken dan alleen toelichting waar ook een definitie was. Er zijn (lijken?) ook definities bij te zijn gekomen waar ze er eerst niet waren. Aangezien definitie (normaal gesproken) verplicht is, lijkt me dat een breaking change? Ik zou voorstellen om het daar OF niet te doen, OF overal optioneel. Zelfde geldt overigens ook voor herkomst en herkomst-definitie

@architolk Dat is wel iets, als we in Imvertor een (herkomst) definitie opnemen waar die niet thuishoort. Als ik het goed zie gebeurt dat niet. Geef anders even aan waar dat gebeurt. Ik heb de configuratie gehouden tegen MIM 1.2, omdat dat toch de variant wordt die we gaan hanteren (?) en we geen klachten krijgen op huidige 1.1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.1.1 Issue meenemen in versie 1.1.1 prio2 Belangrijk, maar niet urgent verwerkt (geheel) z-issue PATCH-versie: achterwaarts compatible bugfix
Projects
None yet
Development

No branches or pull requests

6 participants