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

Autorefresh import script for Flood Events #855

Merged
merged 12 commits into from
Feb 21, 2024
Merged

Conversation

ajaits
Copy link
Contributor

@ajaits ajaits commented May 3, 2023

Script to refresh flood events from earth engine.

Runs the events pipeline to generate flood events and count,area svobs for places for the current year.
Output is stored in GCS that is used for import.

SCHEDULES=scripts/floods:DynamicWorld_FloodEvents_AutoRefresh

@google-cla
Copy link

google-cla bot commented May 3, 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.

@ajaits ajaits changed the base branch from floods to ICTV_import May 3, 2023 12:10
@ajaits ajaits changed the base branch from ICTV_import to master May 3, 2023 12:10
Copy link
Contributor

@jehangiramjad jehangiramjad left a comment

Choose a reason for hiding this comment

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

Thanks! A quick first review

scripts/floods/README.md Outdated Show resolved Hide resolved
scripts/floods/manifest.json Outdated Show resolved Hide resolved
scripts/floods/requirements.txt Show resolved Hide resolved
scripts/earthengine/process_events.py Outdated Show resolved Hide resolved
scripts/earthengine/process_events.py Outdated Show resolved Hide resolved
scripts/earthengine/utils.py Outdated Show resolved Hide resolved
scripts/earthengine/utils.py Outdated Show resolved Hide resolved
scripts/earthengine/utils.py Show resolved Hide resolved
scripts/earthengine/utils.py Outdated Show resolved Hide resolved
scripts/earthengine/utils.py Show resolved Hide resolved
Copy link
Contributor

@jehangiramjad jehangiramjad left a comment

Choose a reason for hiding this comment

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

Approving to unblock for submission to allow import automation testing. Thanks for address most of the early comments!

@ajaits ajaits enabled auto-merge (squash) February 21, 2024 16:12
@ajaits ajaits merged commit 386c7b8 into datacommonsorg:master Feb 21, 2024
9 checks passed
shamimansari1988 pushed a commit to shamimansari1988/data that referenced this pull request Aug 12, 2024
* autorefresh script for flood events

* udpate pipeline config

* update floods configs

* fix unit tests

* add readme

* add requirements.txt for ee scripts

* fix review comments

* address review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants