Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functie Initiatie - extra output #34

Closed
leymanan opened this issue Oct 1, 2020 · 6 comments
Closed

Functie Initiatie - extra output #34

leymanan opened this issue Oct 1, 2020 · 6 comments
Assignees

Comments

@leymanan
Copy link
Collaborator

leymanan commented Oct 1, 2020

Eventueel toch nog DataRapport als output (csv) meegeven?
(bevat overzicht van bms-domein-combinaties met aantal en percentage van te dikke en te dunne bomen)

Dit wordt nergens anders gecreëerd en kan hulp zijn bij foutopsporing (dan moeten we niet enkel op html voortgaan)

@ElsLommelen
Copy link
Collaborator

Misschien eventjes voor de duidelijkheid: in de html gaat het over bomen dunner of dikker dan hetgeen we nodig hadden voor het opstellen van de modellen en die dus weggelaten worden voor het opstellen van de modellen. Dus het gaat hier niet over bomen die een slechte diameter-hoogte-verhouding hebben of dergelijke. Die cijfers op zich worden inderdaad verder niet meer weergegeven, maar je kan hun totaal afleiden uit nBomen en nBomenOmtrek05.

Als je deze nodig hebt bij de foutopsporing, in welke fase is dat dan? Kwestie van het meteen efficiënt aan te pakken: als je er de cijfers graag bij hebt bij de grafieken, is het mss handiger om ze meteen bij de grafiek te zetten, zodat je niet moet gaan zoeken? (Als ik het me goed herinner, was het idee van die html vooral om te vermijden dat de alle metingen van een domein in m opgegeven waren i.p.v. in cm en dit soort problemen er snel uit te halen. Dit was bewust in het begin gezet zodat dit bekeken kon worden voor de lang durende validatiestap.)

Enfin, ik wil maar helpen om het op een voor u zo gemakkelijk mogelijke manier in te bouwen in de workflow, zodat je niet met meerdere tabellen naast elkaar moet werken (onhandig en vraagt veel tijd).

@leymanan
Copy link
Collaborator Author

leymanan commented Oct 1, 2020 via email

@ElsLommelen
Copy link
Collaborator

@leymanan
Ik merk dat in het overzichtstrapport (en ook in de toegevoegde dataframe) alle domein-BMS-combinaties met meer dan 10 bomen toegevoegd worden, ook die waar geen bomen verwijderd worden wegens te dun of te dik. Laat ik deze erin of niet? Vermits maar ongeveer 10 % van de domein-BMS-combinaties te dikke of te dunne bomen heeft, gaat dit de lijsten 10x korter maken. En vermits de functie resultaat() ook nBomen (=nTotaal) en nBomenInterval (=nInterval) weergeeft, kan je hieruit perfect afleiden welke domein-BMS-combinaties wel/geen te dikke of te dunne bomen hebben.

@leymanan
Copy link
Collaborator Author

leymanan commented Nov 23, 2020 via email

@ElsLommelen
Copy link
Collaborator

opgelost in PR #42

@leymanan
Copy link
Collaborator Author

dit iis volgens mij goed opgelost op deze manier, maar ik wacht nog met afsluiten issue tot ik de pull request #42 aanvaard heb (maar daarvoor moet ik eerst nog de andere issues doorlopen)

@leymanan leymanan closed this as completed Dec 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants