Geonovum / MIM-Werkomgeving

Werkomgeving van MIM. Bevat werk en alle pre-publicatieversies.
https://geonovum.github.io/MIM-Werkomgeving/
7 stars 15 forks source link

253 Definitie en toelichting codelijst aangepast #341

Closed PalmJanssen closed 10 months ago

PalmJanssen commented 11 months ago

Naar aanleiding van issue 253 is er nog een tweede aanpassing gedaan in branche 253b. Definitie en toelichting van codelijst aangepast aan nieuwe metagegevens die we nu wel opnemen.

PalmJanssen commented 11 months ago
  • Deze wijzigingen moeten nog doorgevoerd worden in het LD stuk. Ik stel voor om dat mee te nemen in dit PR.

OK maar wie gaat dat doen

  • metagegeven Waarde-item wordt hier Waarde item genoemd. Dit rechttrekken.

Is gebeurd. Nu overal Waarde-item

  • metagegeven Datatype leidt tot een naamconflict met het bestaande modelelement Datatype. Dit is niet handig. Hier moeten we een andere naam voor verzinnen.

Dit zie ik nergens terugkomen. Waar zit dat?

  • Het is mij nog niet helemaal duidelijk hoe een waarde uit een waardelijst een datatype kan hebben. Een waarde is al een datatype, volgens het metamodel. Een enumeratie heeft bijvoorbeeld ook geen datatype. Hoe werkt dit precies?

Ja dat dacht ik dat we ook bij enumeratiewaarde hadden toegevoegd op jouw verzoek. Je argument was dat het datatype betekenis toevoegde aan een waarde. Bij voorbeeld 747 als integer is iets anders dan 747 als CharacterString. Vind ik wel wat voor te zeggen, maar ik zie het nergens terugkomen in andere issues. Wat mij betrft kan het dan ook hier weg. Anders ook bij enumeratiewaarde toevoegen. Op zich kan dat in EA maar ik zie in het UML metamodel niet die mogelijkheid. Dus mijn advies geen datatype voor een waarde in een codelijst

  • Kan een datatype ook een objecttype zijn? Ofwel kun je hiermee een relatie uitdrukken?

Nee. Een datatype is geen ding, het heeft geen identiteit. Is dat wat je bedoelt?

pmaria commented 11 months ago
  • Deze wijzigingen moeten nog doorgevoerd worden in het LD stuk. Ik stel voor om dat mee te nemen in dit PR.

OK maar wie gaat dat doen

Ik wil dat wel doen, maar dat gaat mij waarschijnlijk pas volgende week of de week daarna lukken.

  • metagegeven Datatype leidt tot een naamconflict met het bestaande modelelement Datatype. Dit is niet handig. Hier moeten we een andere naam voor verzinnen.

Dit zie ik nergens terugkomen. Waar zit dat?

Zie https://geonovum.github.io/MIM-Werkomgeving/#respec-offender-this-is-defined-more-than-once-in-the-document

  • Het is mij nog niet helemaal duidelijk hoe een waarde uit een waardelijst een datatype kan hebben. Een waarde is al een datatype, volgens het metamodel. Een enumeratie heeft bijvoorbeeld ook geen datatype. Hoe werkt dit precies?

Ja dat dacht ik dat we ook bij enumeratiewaarde hadden toegevoegd op jouw verzoek. Je argument was dat het datatype betekenis toevoegde aan een waarde. Bij voorbeeld 747 als integer is iets anders dan 747 als CharacterString. Vind ik wel wat voor te zeggen, maar ik zie het nergens terugkomen in andere issues. Wat mij betrft kan het dan ook hier weg. Anders ook bij enumeratiewaarde toevoegen. Op zich kan dat in EA maar ik zie in het UML metamodel niet die mogelijkheid. Dus mijn advies geen datatype voor een waarde in een codelijst

Ha, daar heb ik geen actieve herinnering aan 😅 . Volgens mij mist er scherpte in het definiëren wat een waardelijst en de items daarin precies representeren op dit moment.

  • Kan een datatype ook een objecttype zijn? Ofwel kun je hiermee een relatie uitdrukken?

Nee. Een datatype is geen ding, het heeft geen identiteit. Is dat wat je bedoelt?

De vraag achter de vraag is meer. Als je toch een datatype kunt specificeren voor een item in een waardelijst dan ben je blijkbaar attribuutsoorten aan het definiëren (?!). Waarom zouden het dan niet mogelijk kunnen zijn om relatiesoorten te definieren waarvan de target een waardelijstitem is, door het specificeren van objecttype als type. Maar de fundamentelere vraag die eerst beantwoord moet worden is denk ik: wat representeren die waardes nu? Naar mijn idee zijn dat meestal geen waardes van attribuutsoorten.

pmaria commented 10 months ago

@Gtrouborst Deze is gemerged zonder dat er iets is gedaan aan een aantal delen van mijn review. Dit moet mijn inziens nog gebeuren