Functie AfwijkendeCurves uitbreiden

[leymanan]

uitbreiden met:

• BOS_BHI
• nBomenInterval
• nBomenOmtrek05
• RMSE

[ElsLommelen]

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]

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 <#31 (comment)>, 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]

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]

je hebt gelijk, is overbodig. Sorry.