search

inbo / dhcurve

An R package for automated modelling of diameter-height relations for trees
https://inbo.github.io/dhcurve
GNU General Public License v3.0
0 stars 0 forks source link

Functie AfwijkendeCurves uitbreiden #31

Closed leymanan closed 3 years ago

leymanan commented 3 years ago

uitbreiden met:

ElsLommelen commented 3 years ago

In hoeverre is het belangrijk dat RMSE erbij staat? Ik merk dat die niet berekend wordt in afwijkendeCurves, en vermits deze berekening wat tijd vraagt, lijkt het me niet aangewezen om deze toe te voegen aan de functie afwijkendeCurves.

Ik zie wel dat BOS_BHI, nBomenInterval en nBomenOmtrek05 verschijnen bij de berekening van de RMSE, dus in dat geval lijkt me een nieuw functietje overzichtAfwijkendeCurves() (naam nog overeen te komen) dat de berekeningen van RMSE en de functie afwijkendeCurves() combineert, de meest elegante oplossing.

leymanan commented 3 years ago

Voor mij hoeft die inderdaad niet in functie afwijkendeCurves. Uw voorstel om die er eventueel achteraf aan te koppelen vind ik OK!

Op di 27 okt. 2020 om 13:55 schreef ElsLommelen notifications@github.com:

In hoeverre is het belangrijk dat RMSE erbij staat? Ik merk dat die niet berekend wordt in afwijkendeCurves, en vermits deze berekening wat tijd vraagt, lijkt het me niet aangewezen om deze toe te voegen aan de functie afwijkendeCurves.

Ik zie wel dat BOS_BHI, nBomenInterval en nBomenOmtrek05 verschijnen bij de berekening van de RMSE, dus in dat geval lijkt me een nieuw functietje overzichtAfwijkendeCurves() (naam nog overeen te komen) dat de berekeningen van RMSE en de functie afwijkendeCurves() combineert, de meest elegante oplossing.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/inbo/dhcurve/issues/31#issuecomment-717223471, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGKXN5CJK3YKE557LROE3KTSM27M3ANCNFSM4SACTRTQ .

--

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

ElsLommelen commented 3 years ago

BOS_BHI, nBomenInterval en nBomenOmtrek05 toegevoegd als output van functie afwijkendeCurves().

Achteraf bekeken is die toevoeging misschien toch overbodig: als je RMSE hier toch aan wil koppelen, bv. door de output van resultaat() eraan te koppelen, zitten BOS_BHI, nBomenInterval en nBomenOmtrek05 er sowieso bij omdat ze in de output van resultaat() zitten. Daarom heb ik de aanpassingen in een aparte PR (#43) gestoken, zodat je ervoor kan kiezen om deze niet te mergen als je de aanpassingen niet nodig vindt.

leymanan commented 3 years ago

je hebt gelijk, is overbodig. Sorry.