Aanvullingen correcties al

[leymanan]

kleine aanvullingen aan develop

Zie ook issues #31, #32 en #33 voor aanpassingen die nog moeten gebeuren

[ElsLommelen]

Een kleine opmerking, verder zijn de aanpassingen in orde.

[leymanan]

ok, was voor mezelf ...

Op do 1 okt. 2020 om 17:25 schreef ElsLommelen notifications@github.com:

@ElsLommelen commented on this pull request.

In R/initiatie.R https://github.com/inbo/dhcurve/pull/35#discussion_r498330781:

@@ -195,7 +198,7 @@ initiatie <-

   logOmtrek2 = ~logOmtrek ^ 2

 ) %>%

 filter_(

• ~Omtrek < 2.40

• ~Omtrek < 2.40 # er werden geen te dunne bomen opgemeten (< 0.2 )

⬇️ Suggested change

• ~Omtrek < 2.40 # er werden geen te dunne bomen opgemeten (< 0.2 )

• ~Omtrek < 2.40

Voor uw dataset is dit zo, maar het lijkt me geen relevante info voor iemand die verbeteringen in de code zou willen aanbrengen of die om een andere reden de code bekijkt, dus ik zou het hier weglaten. Ook ben ik niet zeker of het wel nodig is om in de functie initiatie in de code zelf nog eens de voorwaarden voor selectie van basisdata, afgeleide data en lokale data expliciet te vermelden. Dit lijkt me duidelijk vermeld te zijn in de documentatie, de relatief korte codeblokken lijken me voor zich te spreken, en iemand die in het stadium is dat hij code in het package wil verbeteren, kent meestal de werking van het package al voldoende als gebruiker om zoiets te weten (en z'n weg te vinden in de code), lijkt mij. Enfin, ik heb de gewoonte om commentaren in code van package-functies te beperken tot korte functionele opmerkingen om het overzichtelijk te houden. Enkele woordjes commentaar hier en daar vallen op om de workflow te verduidelijken en gemakkelijk iets terug te vinden om aan te passen, maar teveel commentaar maakt het naar mijn gevoel soms onoverzichtelijk (teveel commentaar-kleur die de aandacht trekt bij het overlopen van de code, en de code wordt langer).

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/inbo/dhcurve/pull/35#pullrequestreview-500464524, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGKXN5G5ES455CENKOC27UDSISNNDANCNFSM4SALL4YQ .

--

Anja Leyman

Expert Cel Beheerplanning en Monitoring

---

Ik werk tijdelijk niet op woensdag- en vrijdagnamiddag.

//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

Vlaamse overheid

AGENTSCHAP NATUUR & BOS

Standplaats Instituut voor Natuur- en Bosonderzoek (INBO) Gaverstraat 4, 9500 Geraardsbergen T: 054 436 182 M: 0495 14 90 60 E-mail: anja.leyman@vlaanderen.be anja.leyman@lne.vlaanderen.be

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

[codecov[bot]]

Codecov Report

Merging #35 into develop will not change coverage. The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #35   +/-   ##
========================================
  Coverage    97.49%   97.49%           
========================================
  Files           25       25           
  Lines         1236     1236           
========================================
  Hits          1205     1205           
  Misses          31       31           

Impacted Files	Coverage Δ
R/initiatie.R	100.00% <ø> (ø)