TreeMama / iSeaTree-React-Prototype

Prototype for the iSeaTree (a mobile app which creates compatible data collection for the iTree project).
MIT No Attribution
14 stars 9 forks source link

4.2+: TreeBOT values that are 0% match AND probability entries in 'other_ai' #538

Closed TreeMama closed 10 months ago

TreeMama commented 1 year ago

This looks like it is likely tied to the work done for issue #517 . There are 2 items to address in this ticket.

These screenshots are from Build 89/v 4.1.7. (9/5/2023)

1) that TreeBot sometimes comes back with a "0%" success rating. This feels like it could possibly be a rounding error? This is what it looks like in the front end.

image0 (2)

This is what it looks like in the database.

database

2) It looks like the entries being added in the database are being duplicated (both the scientific AND common name are being entered). Ideally we would JUST want the scientific name being saved. Also - we'd like the 'probability' value saved to be shown as a decimal (this is how @dceu entered it for the other_ai values). Right now it looks like the number is being rounded to an integer - 1.

We would really benefit having:

1) JUST the scientific name and the raw (unrounded) decimal value of the 'winning' match entered. 2) DO NOT enter anything for the common name. just have 1 entry (and it has the scientific name and probability as a decimal).

database

TreeMama commented 12 months ago

Hey @bestw07 I don't suppose you have a minute to check this out?

bestw07 commented 12 months ago

Hi, @TreeMama

I am checking it.

bestw07 commented 12 months ago

I fixed the confidence number issue.

Screenshot_2023-09-20-23-02-54-24_fa96579ccbffecdc18de2fe8ee545f71

bestw07 commented 12 months ago

firebase tree document

image
TreeMama commented 11 months ago

Hi @bestw07 - Christy checked this with the live build today (90 - Android) and it looks like the probability number is still being rounded to a 1 (I was expecting to see a decimal).

@christyheaton do you remember what the front-end displayed (as a percent of match) when you entered these trees on 9/21/23? still_error

TreeMama commented 10 months ago

closing! Looks fixed as of version 4.1.8/4.1.9