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

UI : show new and rare species on top in overview.php #213

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

Conversation

alexbelgium
Copy link

@alexbelgium alexbelgium commented Oct 4, 2024

Proposal : highlight new species in a given day

How :

  • when a new species, or rare (not seen in last 14 days) is detected, put a banner on top to show them showing the species name and max confidence.
  • If there is more than 5, just add a link to Today's table
  • If no new species, same layout as before

Another solution for this proposal : #143

Don't mind the 0%, I faked a new species for my tests

image image

@alexbelgium alexbelgium marked this pull request as draft October 4, 2024 11:33
@alexbelgium alexbelgium marked this pull request as ready for review October 4, 2024 14:00
@alexbelgium
Copy link
Author

Additional ideas :

  • Put a "x" next to the species, so that if we click it, it gets ignored for the rest of the day and disappears from the table. This would be done with a temporary file that would store the names of such species
  • Instead of showing only new species, show also "uncommon" species : species with less than 5 detections over the past 7 days (as current logic for apprise). This helps for example see when a birds gets back from its winter places. Of course there would be a field to show if it's a new, or an uncommon species. I fear there would be too many at the same time, but then that's where the x comes in play.

@alexbelgium
Copy link
Author

Hi @lloydbayley, if you're still around and are interested by the idea I would be really interested by your feedback ! From a logic & coding POV

@lloydbayley
Copy link

If I'm still around? Now I really feel included. The last time I tried an opinion, it resulted in a public telling off, which I don't abide. Pass.

@alexbelgium alexbelgium changed the title UI : show new_species on top in overview.php UI : show new and rare species on top in overview.php Nov 12, 2024
@alexbelgium
Copy link
Author

I've added also rare species - for example species not seen since x days, and it says the last time it was seen (for example : Last seen : 3m ago)

@alexbelgium
Copy link
Author

Code should be final now, including configuration for the duration before which a species becomes "rare" :) I see that as a fast overview of new species, and species not encountered in a long time (like seasonal species, or just ones rarely seen). Default for rare species is 30 days to avoid overload. I've already implemented it in the homeassistant add-on and currently it runs well

@alexbelgium
Copy link
Author

Makes me think that currently apprises notifications considers that a "rare" species is a species not seen in 5 days. We could make that linked to the same new option - honestly a specie not seen in 5 days is not that rare. For example after a bout of rains (quite lengthy in Belgium) I get 5 common "rare species" that just don't like their feathers wet

@jmtmp
Copy link

jmtmp commented Nov 20, 2024

Nice and smart, but what slightly different/cleaner layout?
recent

@alexbelgium
Copy link
Author

alexbelgium commented Nov 26, 2024

Updated proposal :

  • Max confidence
  • Last seen
  • Hour of first seen today (number of occurences today)
image

@alexbelgium alexbelgium reopened this Nov 26, 2024
@alexbelgium
Copy link
Author

Cleaned the history

@alexbelgium
Copy link
Author

alexbelgium commented Nov 27, 2024

Update : layout is now depending on screen size :

800px

image

< 800px:
IMG_6621

@jmtmp
Copy link

jmtmp commented Dec 13, 2024

This is really nice!
How about using these unified "bird cards" elsewhere on the site, such as Top10/New birds in the weekly overview?

@alexbelgium
Copy link
Author

Thanks for the feedback!

Well those bird cards were initially globally aligned with the 5 latest but I see your point for the weekly overview. It should be feasible but should be managed through a specific feature request & PR to avoid complexifying this PR

@jkrauska
Copy link

love this!

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