biodiversitydata-se / biocollect

Biocollect front end application for Bioatlas
https://biocollect.biodiversitydata.se/
Other
0 stars 0 forks source link

wrong total for mammals count in Nattrutterna #220

Closed mathieuLU closed 2 years ago

mathieuLU commented 2 years ago

example https://biocollect.biodiversitydata.se/sft/bioActivity/index/4a38d79a-afa9-4aeb-a404-2a7f27601432

pastedImage

mathieuLU commented 2 years ago

fixed ! as expected, it was a copy-paste error in the template of the nattrutter....

"dataType": "number", "name": "totalP13", "decimalPlaces": 0, "computed": { "expression": "countNonZeroValuesInColumn(**mammalObservations**, \"P13\")"

it was written observations instead of mammalObservations, for all the points and the total => the total line for mammals was then the same as the number of birds :)

Now I must check how many existing surveyx are wrong, and how to fix them....

ekol-ali commented 2 years ago

Tack Mathieu,

för alla goda insatser!

Åke

Från: mathieuLU @.> Skickat: Thursday, 24 March 2022 16:54 Till: biodiversitydata-se/biocollect @.> Kopia: Åke Lindström @.>; Assign @.> Ämne: Re: [biodiversitydata-se/biocollect] wrong total for mammals count in Nattrutterna (Issue #220)

fixed ! as expected, it was a copy-paste error in the template of the nattrutter....

"dataType": "number", "name": "totalP13", "decimalPlaces": 0, "computed": { "expression": "countNonZeroValuesInColumn(mammalObservations, \"P13\")"

it was written observations instead of mammalObservations, for all the points and the total => the total line for mammals was then the same as the number of birds :)

Now I must check how many existing surveyx are wrong, and how to fix them....

— Reply to this email directly, view it on GitHubhttps://github.com/biodiversitydata-se/biocollect/issues/220#issuecomment-1077780689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXBWL5WMWJT3A7WX6X2JBBLVBSF3LANCNFSM5RQR677Q. You are receiving this because you were assigned.Message ID: @.***>

mathieuLU commented 2 years ago

Actually the old protocols are fixed as well by that modification ! If we compare with the very first screenshot of this issue, we now have : Screenshot from 2022-03-25 09-37-44

But in the database that total-line data is stored (but why ?). Not a big deal since this elements are not used anyway. A way to fix these lines could be to re-save them? no urgent at all