Closed idekkerkza closed 11 months ago
Beste I Dekker, Dank voor de melding en het meedenken. Het klopt dat de Github pagina open staat voor iedereen, maar we hebben liever dat als men iets wil melden dat dat dan via de mail gaat naar IMEV@Geonovum.nl. Geonovum zet daarna het wijzigingsverzoek of de bug op Github. Daarna is het juist wel de bedoeling dat er gereageerd wordt via Github zodat iedereen er over mee kan discussiëren.
Taalkundig ben ik het niet eens met deze gemelde bug. Je zegt niet "het aantal is meer geworden" maar wel "het aantal is groter geworden". Het gaat hier over een Aantal. Als er niet zo specifiek "Aantal" in de naam van het objecttype had gestaan was "meer" wel beter geweest. Je zegt wel "er zijn meer bevoorradingen".
Ik hoop dat je na deze motivatie het hiermee eens bent en voel je vrij om er weer op te reageren via Github. Dan geef ik hem daarna het label "won't fix".
Beste Pieter,
Bedankt voor jouw reactie. Wijzigingsverzoeken zal ik in het vervolg via het opgegeven e-mailadres doorzetten. Ik mis dit echter in de readme.md en was hierdoor niet op de hoogte. Zou dit in de readme.md verwerkt kunnen worden?
Bedankt ook voor de uitleg, ik begrijp nu waarom hiervoor is gekozen. De bewoording: 'Kleiner of gelijk vijf' is alleen, als gebruiker bij het invoeren, niet helemaal logisch. Hier zou je 'Kleiner of gelijk aan vijf' verwachten. Misschien kan deze wijziging meegenomen worden, anders houden wij het op 'won't fix'.
Vriendelijke groeten, Isabeau Dekker
Hoi Isabeau,
Dan zou het moeten worden "Kleiner dan of gelijk aan vijf". Hier gaan we nog over nadenken.
Omdat je niet de eerste bent die zelfstandig issues heeft aangemaakt, heb ik het nu in de templates verwerkt die je te zien krijgt wanneer je een nieuw issue probeert aan te maken. Helaas is er op de issue pagina geen manier om een readme.md op te nemen. Op de home pagina https://github.com/Geonovum/imev-werkomgeving/ kan dit wel. Dus daar ga ik het ook nog opnemen. Bedankt voor deze tip.
Hoi Pieter,
Dat is goed. Super dat het nu verwerkt is in de templates en in de readme.md op de home pagina.
Vriendelijke groeten, Isabeau Dekker
Misschien is het het beste om een zo consistent mogelijke schrijfwijze toe te passen binnen het IMEV? Mijn voorkeur zou bijvoorbeeld zijn:
Voordruk
Categorie aantal bevoorradingen
reactietijdNoodstop
hoeveelheidsklasse stikstofgehalte
Doorzet categorie
Daarmee wordt het issue groter en daarom ververander ik dit ook van bug naar wijzigingsverzoek. Ik vertaal hem daarom ook in de volgende opmerking naar het template van wijzigingsverzoek.
De waardes in de enumeraties voor hoeveelheden en grootheden zijn niet consistent.
Voordruk
Categorie aantal bevoorradingen
reactietijdNoodstop
hoeveelheidsklasse stikstofgehalte
laag
Een volledige inventarisatie waar dit voorkomt is nog niet uitgevoerd
De < en > dan tekens geven een probleem in de modelleertooling. Op zich niet zo netjes en ik zal eens kijken wat we daar aan kunnen doen.
In de expertgroep van 17 mei is besloten om te adviseren om dit wijzigingsverzoek niet uit te voeren, omdat de impact groot is en het voordeel minimaal. Tevens voorkomt dat problemen zoals in bovenstaande comment genoemd.
Tijdens het softwareleveranciersoverleg van 7 juni is er geen bezwaar geuit tegen het niet doorvoeren van deze wijziging.
Adviesgroep 13-9-2023 heeft besloten deze wijziging niet door te voeren.
Waar zit de fout?
6.1.10 VII-A.7 Opslaan van propaan of propeen in opslagtanks. Bij enumeratie: CategorieAantalBevoorradingen![image](https://user-images.githubusercontent.com/126789579/222692657-3d44dff6-160b-4e31-a0c2-b8e061693528.png)
Wat is de fout?
Aantal bevoorradingen bij A7 optie keuze: ‘groter dan vijf’, ‘kleiner of gelijk vijf’. De bewoording is hier niet helemaal logisch. Dit zou iets netter kunnen.
Wat moet het worden? [optioneel]
Bijvoorbeeld: ‘meer dan 5’, ‘minder of gelijk aan 5’