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

Op welke model elementen zit historie #224 #327

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

lennartvanbergen
Copy link
Collaborator

@lennartvanbergen lennartvanbergen commented Aug 16, 2023

Zie #224

Korte beschrijving voor changelog bijlage bij uitbrengen versie: De metagegevens voor historie kunnen niet (meer) bijgehouden worden op gegevensgroep (dit bijhouden gebeurd bij de specifieke modelelementen binnen het gegevensgroeptype dat bij de gegevensgroep hoort).

Verwerking nodig in H2, H3, H4 en H6. In H2 een toelichting, in de rest de uitwerking ervan.

Heeft een sterke relatie met #326 en het H2 deel is al verwerkt in een eigen pull request (stap 1). Dit had ook binnen deze getrokken kunnen worden, maar is in een eigen gebeurd).

Stap 2: in H3 UML. Deze is gedaan in deze branch: verwijderd dat bij een gegevensgroep indicatie materiele en formele historie bijhouden kan worden. Let op dat deze verwerking niet teruggedraaid wordt bij de verwerking van #151 Historie naamgeving aanpassen

Stap 3: in H4 LD.
In de tekst bij https://docs.geostandaarden.nl/mim/mim/#modellering-metagegevens-voor-objecten-en-attributen-in-ld staat het nog wel, echter in de tekst on de source file https://github.com/Geonovum/MIM-Werkomgeving/blob/master/MetamodelLD.md staat het niet meer. Mogelijk heeft iemand dit al verwerkt.

In de diagrammen is het nog niet verwerkt! @pmaria kan jij de diagrammen aanpassen?

H6 lijkt niet geraakt te worden door deze wijziging, maar dit kan ik niet met zekerheid zeggen. @pmaria kan jij hier ook naar kijken? Kan samen.

Zie #224 

Korte beschrijving voor changelog bijlage bij uitbrengen versie: 
De metagegevens voor historie kunnen niet (meer) bijgehouden worden op gegevensgroep (dit bijhouden gebeurd bij de specifieke modelelementen binnen het gegevensgroeptype dat bij de gegevensgroep hoort). 

Verwerking nodig in H2, H3, H4 en H6. In H2 een toelichting, in de rest de uitwerking ervan. 

Heeft een sterke relatie met #326 en het H2 deel is al verwerkt in een eigen pull request (stap 1). Dit had ook binnen deze getrokken kunnen worden, maar is in een eigen gebeurd). 

Stap 2: in H3. Deze is gedaan in deze branch: verwijderd dat bij een gegevensgroep indicatie materiele en formele historie bijhouden kan worden. Let op dat deze verwerking niet teruggedraaid wordt bij de verwerking van #151 Historie naamgeving aanpassen
@lennartvanbergen
Copy link
Collaborator Author

Het LD deel moet nog bekeken worden. Deels verwerkt, maar in de diagrammen nog niet (kan ik niet) en in bijlage 6.4 lijk er geen impact, maar dat is niet zeker.

@pmaria
Copy link
Contributor

pmaria commented Aug 17, 2023

Op zich kan ik me hierin vinden. Alleen vraag ik me wel af of je dit issue kunt afhandelen zonder issue #170 af te handelen.

Wanneer ik nog passages zie als:

De gegevensgroep is een betekenisvol kenmerk van een objecttype.

Kenmerken van een objecttype hebben een unieke naam binnen het objecttype (attribuutsoort, gegevensgroep, relatiesoort et cetera)

en ik kijk naar het huidige gebruik van gegevensgroepen in veel modellen, dan wordt de gegevensgroep vaak gezien als een kenmerk die verwijst naar een object zonder identificatie in de vorm van een gegevensgroeptype.

Als dat laatste de bedoeling is, dan is deze aanpassing onterecht. Als dat laatste niet de bedoeling is, dan moeten we dat ook beschrijven.

@Gtrouborst
Copy link
Contributor

Gtrouborst commented Aug 22, 2023

@lennartvanbergen, @pmaria: ik voer deze request nog niet door i.v.m. laatste commentaar van Pano. Als het wel gemerged kan worden ondanks het commentaar (omdat dit in een ander issue wordt opgepakt?) dan hoor ik het graag.

@Gtrouborst Gtrouborst merged commit 7c5a962 into master Sep 5, 2023
@Gtrouborst
Copy link
Contributor

De metagegevens voor historie kunnen niet (meer) bijgehouden worden op gegevensgroep (dit bijhouden gebeurd bij de specifieke modelelementen binnen het gegevensgroeptype dat bij de gegevensgroep hoort).

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.

3 participants