Skip to content

Conversation

@gtempus
Copy link

@gtempus gtempus commented Sep 6, 2024

Overview

I'll add more information next week!

Use the GFW Data API data mart.
It's currently at:
http://gfw-data-api-elb-shared-dev-lb-10091095.us-east-1.elb.amazonaws.com:30616/#tag/Forest-Change-Analysis

Update the server-side dataMartRequest URL configuration to try it out!

Notes

I split the code by adding two helper functions depending on isDownload. I wanted to be able to show the extreme difference between the two approaches. The GFW Data API data mart makes for a much more straightforward interface.

@jterry64
Copy link
Member

jterry64 commented Sep 6, 2024

@gtempus just to make sure I understand, you're trying to highlight the difference between get_net_tree_cover_change_results and get_net_tree_cover_change_download_url, where the former uses the data mart, and the latter uses the existing method?

@gtempus
Copy link
Author

gtempus commented Sep 9, 2024

Yes, @jterry64. I was commenting that since we don't have the download endpoint implemented, it's easy to see how the data mart simplifies the integration between Data API and Flagship.

@gtempus gtempus force-pushed the feat/use-datamart-service branch from 3bc320d to 9f0118a Compare October 4, 2024 17:35
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4848 October 4, 2024 17:35 Inactive
@gtempus gtempus force-pushed the feat/use-datamart-service branch from 9f0118a to 72352ac Compare October 4, 2024 22:35
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4848 October 4, 2024 22:35 Inactive
@gtempus gtempus force-pushed the feat/use-datamart-service branch from 72352ac to eb74b25 Compare October 7, 2024 22:26
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4848 October 7, 2024 22:26 Inactive
@wri7tno wri7tno closed this Jul 7, 2025
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.

5 participants