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

Feat : sort species by max confidence #198

Merged
merged 3 commits into from
Sep 30, 2024

Conversation

alexbelgium
Copy link

Hi,

Still trying to sort through all my species, and I wanted to try to sort by max confidence. In the end this can be even more useful than sorting by count for atypical species as was demonstrated when I had a single traveling bird that came (so 3-4 occurences) but with a 99% detection value. On the other hand, I have an abnormal specie with tons of detections but none above 95% which could indicate a false positive.

Here is the code - I was wondering if this could be useful to the more global community.

Also, I made a specific button to reflect the "sort by confidence" as currently the page for individual observations of a species use the "sort by occurences" icons instead.

image

@Emmo213
Copy link

Emmo213 commented Sep 24, 2024

Oo, I like this idea.

@lloydbayley
Copy link

Works for me. We like this one. Very handy. +1 for commit!

@Nachtzuster Nachtzuster merged commit 96df8e7 into Nachtzuster:main Sep 30, 2024
@alexbelgium alexbelgium deleted the patch-8 branch September 30, 2024 19:37
@Emmo213
Copy link

Emmo213 commented Oct 22, 2024

Really good enhancement. I added it to my Best Recordings page too.

@alexbelgium
Copy link
Author

alexbelgium commented Oct 22, 2024

Thanks! Could you please provide the PR? Indeed we might as well make it there too!

Edit : or I can do it too if you prefer ;)

@Emmo213
Copy link

Emmo213 commented Oct 22, 2024

If you'd like it here's my commit.

Emmo213@671a47d

@alexbelgium
Copy link
Author

Thanks! I'll already merge it in my fork (used for the HA addon) but indeed it would be good to have it in the main app. I'm not at my computer for the moment but will test asap

@alexbelgium
Copy link
Author

@Emmo213 : thanks the PR is created !

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.

4 participants