sepal-contrib / planet_active_fires_explorer Goto Github PK

As mentioned in 12rambau/sepal_ui#208 we have to incorporate the new changes.

—Creating this issue to reference a second requirement on #4

As referenced by @yfinegold, would be useful integrate a filter for the current alerts, such as confidence...

We could create a filter to select all the alerts that are present in a given country country and also filter them by its land cover, so we could focus only on determined areas.

Create the option to close and open the metadata widget

it would be great to display a table with the metadata for each fire event from FIRMS. This would only be for the FIRMS events and not custom events made by clicking on the map. Perhaps it would be useful to add some filters for the alerts based on the metadata, such as filter for confidence.

When there are too many alerts, the performance of the app is highly affected, we should limit the number of fire alerts to be shown at once.

The legend is not appearing in map. So cannot switch between Planet scenes.

As referenced here 12rambau/sepal_ui#212

Once the last version of sepal_ui is released, include planet credentials log-in.

as suggested by @12rambau in openforis/sepal-doc#55 (review)

The fire alerts are based on modis imagery and I believe the resolution is 375m. Could you make the red box showing the alert match the resolution?

One of the main disadvantages of this module is the difficulty to handle large alert data, especially when trying to clip geometries to countries with complex geometries and/or several polygons.
Currently, I've noticed a new project of dask geopandas that could help to parallelize the process... I will keep this here until we can implement it: https://github.com/geopandas/dask-geopandas

Depending on the number of alerts, the browser could be blocked and the loading wouldn't be efficient due to the alerts are loaded on the client side, as posted in SEPAL openforis/sepal#241, would be nice to find a VectorTiling process and so allow as many alerts as are in the AOI.

In the meanwhile, a workaround would be to limit the number of alerts displayed at the same time with the option to paginate them.

It would be really useful make the option to download the images that represents the pre and post-fire detection

As the AOI is filled, there is a light blue filter on anything displayed in the map.

With the AOI selected:

without:

And also deactivate it when the planet API is not valid

By exploring the app, the usability could be improved if the settings can be collapsed to the left or separate them into a new tab

Map display not working correctly (for example cannot zoom). Some buttons and dropdowns not working (e.g. cannot enter FIRMS API credentials, cannot modify Planet date for imagery selection).

When confidence option is set as All, the alerts dropdown list is not displaying the alerts

there are some planet api-keys that doesn't support the daily and historical imagery., would be nice to filter them before display them in the map, or create an alert-warning message that let the user know that the max zoom available for those images is limited

Instead of open the layers in the layers control

The navigation bar is taking lots of space of the map

The historical data is gathered here https://firms.modaps.eosdis.nasa.gov/download/, however, it works with requests... would be nice to have an option to upload own alerts in the app, so the user could inspect them with the planet imagery.

I receive this error:
'DataFrame' object has no attribute 'longitude'
when trying to download any alerts (recent) for Mexico

Would be nice to have the option to save the current alert (after asses its occurrence) and provide a download file (.csv, shapefile, jsson).

There is a new way to get alerts from the FIRMS platform, it is by the API key https://firms.modaps.eosdis.nasa.gov/api/.

I share with you the link to my related commit : sepal-contrib/planet-order@3a1e1a0