search

Geonovum / MIM-Werkomgeving

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

Voorstel toevoeging 'alternatieve' om breaking change te voorkomen, issue #198 #226

Closed architolk closed 2 years ago

architolk commented 2 years ago

Gelinkt aan #198

pmaria commented 2 years ago

In mijn optiek leidt de aanscherping van Lennart in #210 automatisch tot een breaking change in de Linked Data transformatie. Op zich prima om het voor deze versie zo te laten, maar dan moeten we dit in de Y versie wel goed aanpakken.

lennartvanbergen commented 2 years ago

Dit voorstel is juist zo opgeschreven om een breaking change te (kunnen) voorkomen.

Mogelijk zit er een bug aan de LD kant, die bij het oplossen breaking is, maar dat mag bij bugs.

Er is inderdaad wel iets nodig in de mapping:

ALS alias niet gevuld, DAN staat de normale schrijfwijze in de naam van het modelelement ALS alias wel is gevuld, DAN staat de normale schrijfwijze in de alias, en de technische schrijfwijze in de naam van het modelelement

Dit is altijd zo geweest, dus het is mijn inziens geen breaking change. Alsmede, je kan met voorgaand algoritme(tje) voorkomen dat het een breaking change wordt. Bugfixes vallen niet onder breaking changes.

lennartvanbergen commented 2 years ago

De definitie aanpassen kan als volgt: Alias; de schrijfwijze van de naam van het modelelement in natuurlijke taal. Toelichting; de alias is alleen gevuld wanneer de naam van het model element zelf niet in natuurlijke taal is beschreven, maar bijvoorbeeld een camel case of snake case schrijfwijze hanteert.

Het woord alternatieve naam kan hier niet, omdat dan nog steeds niet duidelijk is welke van de twee de natuurlijke schrijfwijze is en welke de camel case of snake case. Als A een alternatief is van B, dan is B ook een alternatief van A, en dan weet je nog steeds niet of A de natuurlijke schrijfwijze bevat of B. Het gaat hier niet om alternatieve namen maar om alternatieve schrijfwijzen.

architolk commented 2 years ago

Maar, euh... mim:naam is toch de "echte" naam? En dus is elke andere naam toch een "alternatieve naam"? Er zit geen bug in de LD kant, maar het voorstel zoals het nu wordt gedaan is onduidelijk. De huidige versie van MIM staat het toe om in mim:naam zowel een wat meer "technischere" naam op te nemen als ook de naam in natuurlijke taal. Dit is afhankelijk van welke keuze je maakt (die op dit moment dus niet formeel wordt afgedwongen).

architolk commented 2 years ago

Wat mij betreft mag deze pull request vervallen: met het voorstel van Pano (zie #198), is duidelijk hoe we er mee omgaan aan de LD kant, namelijk: niet transformeren, dan maakt het niet meer uit wat mim:alias betekent, en is er ook geen breaking change meer.

lennartvanbergen commented 2 years ago

Vanwege laatste opm @architolk dit pull request gesloten.