-
Notifications
You must be signed in to change notification settings - Fork 45
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
Integration with third party apps: Further integration with FAO-WOCAT. #824
Comments
Deliverables for Integration tasks:
|
for FAO-WOCAT we want to allow the user to use the advanced settings for fao - wocat |
For FAO-WOcat the period is important to define in the beginning. Code : https://code.earthengine.google.com/0fe1c192e5137151df30a0d2ff7f6039 remove the start and end and replace them with just one box to add a period -> only for the FAO advanced options.
|
We need to make the remote server processing work based on the Code here: |
@Samweli I've given you admin access to the trends.earth API server, which is needed to deploy this feature (following the process outlined here). In addition to the GUI changes gee/sdg-15-3-1-sub-indicators/src/main.py will need to be modified to correctly handle the new FAO-WOCAT parameterization, as will gee/productivity/src/main.py for the case of users running just the productivity dataset but not the full SDG. New code for actually running the calculations (the code running the algorithm itself as opposed to just calling our API) should go somewhere in te_algorithms/gee (perhaps in te_algorithms/gee/productivity.py - but that script is getting a bit long and likely needs to be split up). As a side note, the code in |
Thanks
Thanks @azvoleff, I will create new issues for each of these action items and work on them. |
@azvoleff from the chat with Cesar he mentioned that the new FAO-WOCAT should be only available remotely, doesn't modifying these gee scripts make it also available locally? |
Further integration with FAO-WOCAT.
The text was updated successfully, but these errors were encountered: