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

RADD Alerts #643

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

RADD Alerts #643

wants to merge 3 commits into from

Conversation

nrtwur
Copy link

@nrtwur nrtwur commented Jun 26, 2023

Pull request for the first version of RADD Alerts.

What shall I do about the following errors?

Issue('WUR/radd_alerts', 'WUR/WUR_radd_alerts.json', 'examples', 'Missing preview script: WUR/WUR_radd_alerts_preview.js', IssueLevel.error)
Issue('WUR/radd_alerts', 'WUR/WUR_radd_alerts.json', 'keywords_tree', 'Only one instance of "wur"', IssueLevel.error)

@google-cla
Copy link

google-cla bot commented Jun 26, 2023

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.

@jdbcode
Copy link
Member

jdbcode commented Jul 10, 2023

Sorry for the delayed reply. Have you had any correspondence via email with the EE data team regarding these issues?
Just wanting to make sure we're responding to the most recent thread.

@jdbcode
Copy link
Member

jdbcode commented Jul 10, 2023

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

@nrtwur
Copy link
Author

nrtwur commented Jul 19, 2023

Thank you such much for getting back to us. We fixed the CLA issue and this is the most recent thread.

'forest',
'radar',
'sentinel',
'wur',
Copy link
Collaborator

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -0,0 +1,14 @@
var radd = ee.ImageCollection('projects/radar-wur/raddalert/v1');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

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';
Copy link
Collaborator

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?

Copy link
Author

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?

@@ -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)
Copy link
Collaborator

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

eg, https://github.com/google/earthengine-catalog/blob/main/examples/NASA/NASA_NASADEM_HGT_001.js

Copy link
Author

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

'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).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no comma before "separately"

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

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).
Copy link
Collaborator

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

Copy link
Author

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

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)
Copy link
Collaborator

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Citation was removed here

],
},
'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.
Copy link
Collaborator

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)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

|||,
}
{
name: 'forest_baseline',
Copy link
Collaborator

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

Copy link
Author

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?

@nrtwur
Copy link
Author

nrtwur commented Oct 17, 2023

Dear Reader,

I was wondering: What is the status after the last changes made from our site?

Thank you very much in advance.

@nrtwur
Copy link
Author

nrtwur commented Nov 2, 2023

@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.

@nrtwur nrtwur requested a review from simonff November 13, 2023 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants