pyronear / pyro-platform

Detection & monitoring platform of wildfires
https://platform.pyronear.org/
Apache License 2.0
10 stars 11 forks source link

Alert erasing button #54

Closed pechouc closed 3 years ago

pechouc commented 3 years ago

Hello everyone!

Brief description of the PR

This PR aims at introducing two new features in the overview area of each alert/event:

These simple interactions required a bit of work, essentially in main.py, as several outputs are now determined by a multitude of inputs (not only the center and zoom attributes of the map anymore). It also required to modify the build_individual_alert_components in alerts.py, some of its outputs being transferred to a new function (display_alert_selection_area).

I have tried to comment the code as much as possible but let me know if you have any question of course!

Screenshots

Capture d’écran 2021-05-24 à 08 53 46

Capture d’écran 2021-05-24 à 08 54 16

Capture d’écran 2021-05-24 à 08 55 11

Capture d’écran 2021-05-24 à 08 56 05

Capture d’écran 2021-05-24 à 08 56 57

Capture d’écran 2021-05-24 à 08 57 38

Question

I don't know if you have the same issue but as you can see on the screenshots, the map doesn't load properly when I build the app locally with Docker. Using `python3 app/main.py" works (slightly) better and the deployed platform seems to display the map correctly, so it should not be too big an issue. But still, it does not help with the debugging...