Nelos-VZW / dives-2

Repository waar we de issues van de testers zullen opvangen
1 stars 1 forks source link

[BUG] Verkeerd aantal gelogde duiken in opleiding vs. statistieken #329

Closed Calinatus closed 1 year ago

Calinatus commented 1 year ago

Aantal duiken 10-20 m onder "Opleiding" (voor 2*D althans) stemt niet overeen met het aantal bij de statistieken.

Overigens worden duiken gelogd op 20 m hier niet bij meegeteld.

FIG1

FIG2

ndsmyter commented 1 year ago

@Calinatus , Ik zal dit nog wel eens verder bekijken in de code ook, maar zou het kunnen zijn dat het verschil te maken heeft met de "Open water" voorwaarde? Want een duik in bijvoorbeeld Nemo33 telt niet mee als een "Open water" duikplaats. Dus in de eerste screenshot zou die niet meetellen, maar in de 2e wel.

Kan zijn dat het effectief een fout is in de berekening, maar ik denk dat we dit nochtans goed getest hebben :-)

Calinatus commented 1 year ago

@Calinatus , Ik zal dit nog wel eens verder bekijken in de code ook, maar zou het kunnen zijn dat het verschil te maken heeft met de "Open water" voorwaarde? Want een duik in bijvoorbeeld Nemo33 telt niet mee als een "Open water" duikplaats. Dus in de eerste screenshot zou die niet meetellen, maar in de 2e wel.

Kan zijn dat het effectief een fout is in de berekening, maar ik denk dat we dit nochtans goed getest hebben :-)

Hoi Nicolas, bedankt alvast voor de input. Nee hoor, alle openwaterduiken waren wel degelijk in open water, dus er moet ergens op een of andere manier toch een foutje in geslopen zijn.

Kunnen de intervallen van de dieptezones overigens aangepast worden zodat de grenswaarden geïncludeerd worden? Zo heb ik bv. 2 duiken gelogd op 20 m diepte, die dan in 20-30 m terechtkomen, en niet meetellen voor de vereiste van 15 duiken op 10-20 m. Op zich geen probleem met het aantal duiken dat nog komen zal, maar kan wel vervelend worden naarmate de vereisten strenger worden bij hogere brevetten.

DIVEScoordinator commented 1 year ago

we sluiten deze af. twee redenen dat dit afwijkend is: