toevoegen van functie om status metingen aan te passen

[leymanan]

ik heb zelf enkele hulpfuncties geschreven om status van enerzijds metingen, en anderzijds van dhcurves aan te passen.
(zie github leymanan/dhcurves_ANB: develop-branch)

Misschien kunnen die meegenomen worden in het package?
Moet niet persé, ik roep ze nu zelf aan via source(file = here::here("src/hulpfuncties.R")), dus ik kan dat ook zo blijven doen.

[ElsLommelen]

Vermits het aanspreken van de databank volledig buiten dit package gelaten is, lijkt het me properder om ook deze functies erbuiten te laten. Nadeel is vooral dat die functies toch wat extra ballast met zich meebrengen in het package:

• een extra dependency, nl. RODBC, dus een extra package dat bij gebruikers geïnstalleerd wordt als ze het package willen gebruiken (kan wel onder suggests, zodat het niet bij elk gebruik van dhcurve geladen wordt)
• om unittests voor deze functies te maken, is er een testdatabank in Access nodig (intern in het package opgeslagen, of eentje dat voor de test codematig aangemaakt wordt als dat al kan)

Bijkomend gaat het ook lastig zijn om een unittest te ontwikkelen die ook op andere OS werkt (omdat Access enkel op Windows draait). Ik zou uiteraard ook de tests kunnen aanpassen zodat ze enkel in Windows uitgevoerd worden, maar eigenlijk test ik bij voorkeur toch op alle OS om de rest van het package ook voor niet-Windows-gebruikers bruikbaar te houden.

Enfin, toch wel wat moeilijkheden te overwinnen om die functies toe te voegen. Wat ik me afvraag: zou het evt. interessant kunnen zijn om een apart package te maken met alle functies voor het aanspreken van de databank, waarin behalve die hulpfuncties ook functies zitten om de gegevens uit de databank op te vragen? Of is dat meer werk dan dat het voordelen biedt t.o.v. de functies in je eigen repo te hebben? (Reken dat het meeste werk van een package kruipt in het schrijven van de functies en de documentatie erbij, of toch als het voor een beperkte gebruikersgroep is en je niet teveel tijd wil steken in unittests en het automatisch testen van code. Voor dhcurve zit dat er dus wel allemaal in, inbodb is een voorbeeld waar dit (nog) niet het geval is.) Een groot voordeel van de functies in package-vorm lijkt mij dat ze gemakkelijk te gebruiken zijn na installatie van het package. Nadeel is een extra repo, dus mss een beetje overdreven als het maar over enkele functies gaat, en een klein beetje code in een andere repo?

[leymanan]

als ik het zo hoor: laat maar zitten ;-)

Denk dat we allebei genoeg werk hebben, niet?
Ik hou het in een apart scriptje met extra "hulpfuncties"!
En inladen doe ik ook in een "min" script.
Voordeel is ook dat het veel flexibeler is, dan functies in een package ...

[ElsLommelen]

Da's wel waar, losse scripts zijn flexibeler, zeker in vergelijking met een package dat je uitgebreid test zodat de functies 'stabiel' blijven voor gebruikers (waar je dus eigenlijk het omgekeerde mee wil bereiken).
Ok, dan laten we de situatie zoals ze is en kan dit issue weg?

[leymanan]

OK! Op wo 24 mrt. 2021 om 15:07 schreef ElsLommelen ***@***.***>:

…

Da's wel waar, losse scripts zijn flexibeler, zeker in vergelijking met een package dat je uitgebreid test zodat de functies 'stabiel' blijven voor gebruikers (waar je dus eigenlijk het omgekeerde mee wil bereiken). Ok, dan laten we de situatie zoals ze is en kan dit issue weg? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGKXN5DJZWWXYYKWBVDK6MDTFHW3ZANCNFSM4WHMHR4Q> .

-- *Anja Leyman * Expert Cel Beheerplanning en Monitoring ////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// Vlaamse overheid AGENTSCHAP *NATUUR & BOS* Standplaats Instituut voor Natuur- en Bosonderzoek (INBO) Gaverstraat 4, 9500 Geraardsbergen T: +32 495 14 90 60 E-mail: ***@***.*** ***@***.***>* *www.natuurenbos.be <http://www.natuurenbos.be/>* --- De inhoud van dit bericht en eventuele bijlage(n) verbinden het Agentschap voor Natuur en Bos niet, zolang niet bevestigd door een geldig ondertekend document