inbo / LSVI

Rekenmodule Lokale Staat Van Instandhouding van habitattypen
https://inbo.github.io/LSVI/
GNU General Public License v3.0
1 stars 0 forks source link

aanpassing indicator éénjarige bij habitat 2330_dw (Versie 3) #231

Open ToonHub opened 2 months ago

ToonHub commented 2 months ago

De indicator éénjarigen is gunstig als het 'aandeel éénjarigen en kaal zand > aandeel meerjarige kruiden'. De indicator bestaat uit twee voorwaarden: 'aandeel éénjarigen + open zand' en 'aandeel meerjarigen'.

De voorwaarde 'aandeel meerjarigen' wordt berekend door de som van de bedekking te nemen van alle soorten behalve die van de éénjarigen (AnalyseVariabele = 'bedekkingExcl' en TaxongroepNaam = 'éénjarigen'). Maar op die manier worden ook de bedekking van de mossoorten, struiken en bomen meegerekend en dat is niet de bedoeling. Dus zou aangepast moeten worden zodat enkel de soorten uit de kruidlaag meegerekend worden.

@ElsLommelen Bestaat daar al een type AnalyseVariabele voor die dat kan berekenen?

Voorlopig heb ik het opgelost door voor deze indicator de LSVI-analyse te doen enkel met de soorten uit de kruidlaag. Dus niet super dringend dat deze issue wordt opgelost.

ElsLommelen commented 2 months ago

Probleem hierbij is dat bedekkingExcl de functie deselecteerKenmerkenInOpname() oproept, en in tegenstelling tot de gelijkaardige selecteerKenmerkenInOpname() lijkt in deze eerste het geval niet te ondersteunen dat een soortengroep en studiegroep opgegeven wordt (omdat dit nog niet eerder nodig was). Dus op zich kunnen we gewoon de analysevariabele bedekkingExcl hiervoor blijven gebruiken, maar dan moeten we deselecteerKenmerkenInOpname() aanpassen zodat bij opgave van een soortengroep en studiegroep de records worden geselecteerd die niet in de opgegeven soortengroep vallen en wel in de opgegeven studiegroep. Als ik het goed begrepen heb?

@Patrikoosterlynck Hoe zat dit weer precies? Kan jij een soortengroep en een studiegroep gelijktijdig opgeven bij de invoer, of moesten we daarvoor een trucje via een csv gebruiken? (In dat geval kan het nog nodig zijn om er toch een aparte analysevariabele van te maken. Ik zal dat straks bekijken.)

ElsLommelen commented 2 months ago

Ok, blijkbaar kan dit in de databank opgelost worden door een van de csv's aan te passen, ik zal dit in orde brengen (dus @Patrikoosterlynck hoeft niks te doen). En ik zal ook die functie aanpassen zodat het package geen rare dingen gaat doen als die databankaanpassing gebeurd is. ;-) Vraagje voor de zekerheid: die regel om enkel soorten in de kruidlaag in beschouwing te nemen, die geldt toch enkel voor aandeel meerjarigen en niet voor aandeel éénjarigen, hoop ik? (Anders zouden we wel een probleem hebben, want bij die éénjarigen zit het open zand al als studiegroep...)

ToonHub commented 2 months ago

voor de éénjarigen moeten we inderdaag geen rekening houden met de vegetatielaag

ElsLommelen commented 1 month ago

Ok, die studiegroep is toegevoegd in de brondatabank en zal bij een update van de SQLite-databank in het package aangepast worden.