Closed santilland closed 1 year ago
@patrick-griffiths @kegro Wind turbine POIs are Integrated in https://gtif-testing.eox.at/gtif_windmill/
as auxiliary "overlay" layer (red geometry only) for Wind Energy Indicator (REP1) without any interaction
and as a special indicator REP6 with clustering enabled to improve "findability" up to zoom 13 and hover over geometry to show properties of each detection
[ ] pending data description markdown to show
@lubojr nice to see this!
I would suggest to deactivate the "power open infrastructure map" base layer by default (but keep it optional but deactivated)
@patrick-griffiths Thanks for a review! please tag @lubojr - me. Who you tagged is an another person not connected to this project.
1) As a common user looking at the GTIF platform I would say that showing a single S2 scene for a single detection geojson does not bring any value. I would say that rather showing all of them on the map at the same time brings more overview to where many are against where little are. If we really want to show s2 image together with a detection (for a single wind turbine), we will need to use the same approach and UI as with the reservoirs. Please give me a second confirmation if this is expected and I shall redo the approach. 2) selection was just a workaround for now to enable the hover over interaction, we shall move that functionality to another config to still enable hover, but disable the "selection" - thanks for noticing it though 3) will leave hidden by default, not a problem to change
@lubojr
Okay, thanks for confirmation! Will do.
Hello @patrick-griffiths we just checked back with our team for the s2-cloudless and there should not be much difference between the s2-cloudless mosaic and the individual s2 scenes.
Here is an example taking an s2 scene from the eo-browser:
compared to the s2-cloudless mosaic
compared to the austrian orthofoto
here with the detections
If the purpose it to show what can be done with s2 imagery, i would understand going the route of showing individual scenes. If the purpose is to allow the user to identify if the detections are correct i would go with leaving it as is, but switching the orthofoto layer as default
One thing to keep in mind is that unless the orthophoto is up-to-date, it might not show the true situation. We have seen some places where wind parks have been redesigned and small turbines replaced with larger ones. For example below the detections look wrong in the high res imagery from Bing, Google, ESRI, be they look obviously correct in Sentinel-2:
Bing
ESRI
Google
Sentinel-2
The Sentinel-2 image is from January 2023. If possible I would by default show the specific S2 image for each detection and have the option to view the ortho or something else high-res for further validation.
Also if you use the s2 cloudless mosaic, you miss out on one of the most striking features of turbines in S2 imagery - the winter time shadow.
@kegro Thank you for a response. Completely understand. I did not think of this aspect. We will redo the concept into "reservoir-like" approach and match it against S2 scenes from SH.
@lubojr @santilland I fully agree with @kegro here... We need the original S2 images, as indicated in the detections geoJson.. S2cloudless should be added as an optional baselayer, but indeed detections were done with late season (long shadow) imagery which is not used for s2cloudless (besides the not so pretty blurry appearance at pixel level)
- @kegro can you confirm: we only need to consider S2 here
and yes sorry for the late reply, we only consider S2 in the model so far.
@patrick-griffiths @kegro New version of REP6 indicator - wind turbines is integrated on the https://gtif.esa.int with a new functionality that 'singleclick' on a detection geometry configures the main map time equal to the detection_time
from provided data. See PR https://github.com/eurodatacube/eodash/pull/2229 for more info.
https://github.com/eurodatacube/eodash/assets/19269667/6b95d127-8464-480b-8af8-4df9737c41d7
Can we close the issue?
this is great @lubojr - nicely solved with providing the corresponding imagery automatically! This is good to go for the demo branch! @kegro feel free to take a look above.
@lubojr could you make sure to integrate the data descriptors provided by DHI last Friday? I just forwarded the email with this content again...
@patrick-griffiths
Thank you for a review and resend. I just added the markdown to the staging branch.
@lubojr one more small thing: please deactivate the S2-mosaic base layer
currently Sentinel-a L2A is displayed by default and only on click the corresponding S2 imagery is shown. I woud prefer to have a neutral base layer like "geoland Gelaende"
@patrick-griffiths Current default baselayer is Terrain light
I should remove the eox cloudless layer from the list of available baselayers for the indicator REP6 and use geoland Gelaende as a default baselayer?
Yes this is fantastic! Nicely done ;)
https://github.com/eurodatacube/eodash/issues/2183#issuecomment-1568319313 changed like this in gtif_staging
ld remove the eox cloudless layer from the list of available baselayers for the indicator REP6 and use geoland Gelaende as a default
Yes please!
We received geojson with bbox detections geometry, confidence and s2 aquisition date. Approach would be to convert data to "pois" allowing for clustering ability, then load geometries to geodb so that they can be fetched once poi is selected.
sample JSON
expected table structure: aoi, aoi_id, indicator, geometry, time, input_data, eo_sensor, measurement_value
where: aoi = coordinates of center point of the detection - lat,lon string aoi_id = some unique ID of the detection (can be stringified number autoincrement) indicator = 'REP1-detections' geometry = the detection geometry time = to be parsed from the s2 aquisition date, save as ISO8601 input_data = "S2L2A" eo_sensor = "S2L2A" sub_aoi = the detection geometry as a WKT measurement_value = the confidence
Example same structure from eodash tables but not all columns need to be defined for eodash client.