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

Map sensors to display flights on a map card #10

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

Conversation

jesmak
Copy link

@jesmak jesmak commented Jan 4, 2024

  • Added a new configuration option (Amount of map sensors to create)
  • Added code to create and update map sensors
  • Added Finnish localization
  • Updated readme to reflect the changes, with an example map card configuration

@AlexandrErohin
Copy link
Owner

@jesmak Thank you for this work! I have tested it and I like that as it is a clear map with flight direction. But it has 2 problems

  1. You if there are more flights then amount of sensors - some flights will not be displayed
  2. These sensors consume memory

I have played with maps and what I have got:
map

Unfortunately, there are commercial labels that interfere (Your map more clean), but it doesnt require additional sensors

@jesmak
Copy link
Author

jesmak commented Jan 10, 2024

Yup, that is a clear downside. Need to add a big number of sensors in a busy area, to avoid flights not being shown. It is in no way a perfect solution, but it works quite well in my less busy area.

By the way, you could probably get rid of the commercial tags with an adblocker. You can just block the annoying elements. Won't work with a mobile, but if you mainly use Home Assistant with a computer, that might make it pretty ok.

# Conflicts:
#	custom_components/flightradar24/config_flow.py
#	custom_components/flightradar24/sensor.py
#	custom_components/flightradar24/translations/de.json
#	custom_components/flightradar24/translations/en.json
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.

None yet

2 participants