-
Notifications
You must be signed in to change notification settings - Fork 66
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
RADD Alerts #643
base: main
Are you sure you want to change the base?
RADD Alerts #643
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Sorry for the delayed reply. Have you had any correspondence via email with the EE data team regarding these issues? |
One thing you'll need to do to contribute is sign the CLA. See the instructions here: https://github.com/google/earthengine-catalog/pull/643/checks?check_run_id=14728915671 |
Thank you such much for getting back to us. We fixed the CLA issue and this is the most recent thread. |
catalog/WUR/WUR_radd_alerts.jsonnet
Outdated
'forest', | ||
'radar', | ||
'sentinel', | ||
'wur', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add this keyword commented-out - we recommend to avoid adding keywords when there's just one instance
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
examples/WUR/WUR_radd_alerts.js
Outdated
@@ -0,0 +1,14 @@ | |||
var radd = ee.ImageCollection('projects/radar-wur/raddalert/v1'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also add a preview script like
https://github.com/google/earthengine-catalog/blob/main/examples/Slovakia/Slovakia_orthos_25cm_preview.js
to print a thumbnail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added a preview script for the thumbnail alongside the original example script.
@@ -0,0 +1,106 @@ | |||
local id = 'WUR/radd_alerts'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are starting to allow users add their assets to the catalog under their original ids (eg, projects/radar-wur/raddalert/v1'). Would you like to do that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the benefit of having the assets listed under their original ids?
examples/WUR/WUR_radd_alerts.js
Outdated
@@ -0,0 +1,14 @@ | |||
var radd = ee.ImageCollection('projects/radar-wur/raddalert/v1'); | |||
var geography = 'sa' // 'ca' (central america), 'sa' (south america), 'africa' (africa), 'asia' (asia & pacific) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a monochrome (eg, white) background image to hide the basemap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a white background in the example script
catalog/WUR/WUR_radd_alerts.jsonnet
Outdated
'gee:type': ee_const.gee_type.image_collection, | ||
description: ||| | ||
Radar satellite imagery from the European Space Agency’s Sentinel-1 mission is used to map new disturbances in primary humid tropical forest at 10 m spatial scale and in near real-time. Sentinel-1’s cloud-penetrating radar provides gap-free observations for the tropics consistently every 6 to 12 days. A new forest disturbance alert is triggered (low confidence alerts) based on a single observation from the latest Sentinel-1 image. Subsequent observations are used to increase confidence and confirm (high confidence alerts) or reject the alert. RADD (Radar for Detecting Deforestation) alerts are operational for the pan-tropics (50 countries) and updated weekly. | ||
The dataset provides a RADD alert layer (alert confidence and alert date) and a primary tropical forest baseline mask, separately for four geographies: Central America, South America, Africa, and Asia (Insular Southeast-Asia and parts of the Pacific). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no comma before "separately"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
catalog/WUR/WUR_radd_alerts.jsonnet
Outdated
Radar satellite imagery from the European Space Agency’s Sentinel-1 mission is used to map new disturbances in primary humid tropical forest at 10 m spatial scale and in near real-time. Sentinel-1’s cloud-penetrating radar provides gap-free observations for the tropics consistently every 6 to 12 days. A new forest disturbance alert is triggered (low confidence alerts) based on a single observation from the latest Sentinel-1 image. Subsequent observations are used to increase confidence and confirm (high confidence alerts) or reject the alert. RADD (Radar for Detecting Deforestation) alerts are operational for the pan-tropics (50 countries) and updated weekly. | ||
The dataset provides a RADD alert layer (alert confidence and alert date) and a primary tropical forest baseline mask, separately for four geographies: Central America, South America, Africa, and Asia (Insular Southeast-Asia and parts of the Pacific). | ||
Documentation: | ||
- RADD alert website (resources, versions and method updates). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this is just http://radd-alert.wur.nl/, you don't need to mention it - it's already in the provider section. You are welcome to link to a specific page here, but please use a markdown link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RADD alert website was removed here
catalog/WUR/WUR_radd_alerts.jsonnet
Outdated
The dataset provides a RADD alert layer (alert confidence and alert date) and a primary tropical forest baseline mask, separately for four geographies: Central America, South America, Africa, and Asia (Insular Southeast-Asia and parts of the Pacific). | ||
Documentation: | ||
- RADD alert website (resources, versions and method updates). | ||
- Reference paper – Reiche et al., 2021 “Forest disturbance alerts for the Congo Basin using Sentinel-1”, Environmental Research Letters, [https://doi.org/10.1088/1748-9326/abd0a8] (https://doi.org/10.1088/1748-9326/abd0a8) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also not needed - it's already in the citation section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Citation was removed here
catalog/WUR/WUR_radd_alerts.jsonnet
Outdated
], | ||
}, | ||
'sci:citation': ||| | ||
Reiche J, Mullissa A, Slagter B, Gou Y, Tsendbazar N, Odongo-Braun C, Vollrath A, Weisse M, Stolle F, Pickens A, Donchyts G, Clinton N, Gorelick N & Herold M, (2021), Forest disturbance alerts for the Congo Basin using Sentinel-1, Environmental Research Letters, https://doi.org/10.1088/1748-9326/abd0a8. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please write DOI as a link:
[https://doi.org/10.1088/1748-9326/abd0a8] (https://doi.org/10.1088/1748-9326/abd0a8)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
|||, | ||
} | ||
{ | ||
name: 'forest_baseline', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eg, on projects/radar-wur/raddalert/v1/africa_20230201 I only see Date and Alert. Which assets have forest_baseline?
Also, there should be a comma on line 65 before the forest_baseline section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We fixed the comma issue in line 65.
Regarding the forest baseline map:
The RADD alerts are structured as Multiband images in an imagecollection for four regions.
For each region the weekly updates are added to the imagecollection as multi-band images with the bands 'Alert' and 'Date'. The forest baseline map is a static layer for each of the four regions within the said Imagecollection. So there are four singleband images in the collection representing the forest baseline four regions. The band in the respective images are called 'Constant'. That's how the RADD alerts architecture is designed. Does that work for you and how could that be addressed?
Dear Reader, I was wondering: What is the status after the last changes made from our site? Thank you very much in advance. |
@simonff just checking again on the status of this pull request. Is there something I can do right now? Thank you very much in advance. |
Dear Reader, I hope this message finds you well. I apologize for the long period of inactivity, but we remain very interested in integrating the RADD alerts into an official asset. Could you please advise us on the next steps required to proceed? We’re eager to understand how we can support this process. @simonff, are you still overseeing this matter? Thank you in advance for your assistance, and we look forward to your guidance. Best regards, |
Pull request for the first version of RADD Alerts.
What shall I do about the following errors?