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

Android - Digit Classification - New main color & logo #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

st-haingo
Copy link
Contributor

No description provided.

@PaulTR
Copy link
Collaborator

PaulTR commented Oct 9, 2024

Can you verify the logic behind confidence on this one? Getting 1.0 confidence on anything I put on the screen (1.0 confidence by itself should be extremely rare). Thanks.

Screenshot_20241009-090946

@st-haingo
Copy link
Contributor Author

@PaulTR I understand. I will check it

@st-nhatnguyen
Copy link
Contributor

Hi @PaulTR ,
I'm a new member and will be working with you in the near future.

I checked the result of the last frame, and found that the result always returns a confidence score of 1.0 as shown in the attached image.

Screenshot 2024-10-11 at 11 34 13 AM

@PaulTR
Copy link
Collaborator

PaulTR commented Oct 15, 2024

So I took that exact model, plugged it into our MediaPipe sample, and then added the score into the logs to make sure it's something the model can generate. Something is missing in the LiteRT app for extracting the confidences, but it should be there (potentially that first line in your log, but the order is wrong for it being 12.399% confident and classifying as '7' when there's a 14.2% earlier).

Screenshot 2024-10-15 at 8 03 25 AM

Screenshot_20241015-080254

@PaulTR
Copy link
Collaborator

PaulTR commented Oct 28, 2024

Hi @st-nhatnguyen, have you had a chance to review this?

@st-nhatnguyen
Copy link
Contributor

Hi @PaulTR , sorry for the late reply.
I am checking this code and refer to the project you sent, but i have not found the reason why it always returns 1
I will try to find out why and solve it this week 🙇

@st-nhatnguyen
Copy link
Contributor

Hi @PaulTR,
I just wanted to let you know that I have not found the cause yet and I am still investigating, it may take some more time, sorry for the delay 🙇 bow

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

Successfully merging this pull request may close these issues.

3 participants