adding manage project linkage with SEALS#3
Open
mamun4105 wants to merge 7 commits intojandrewjohnson:mainfrom
Open
adding manage project linkage with SEALS#3mamun4105 wants to merge 7 commits intojandrewjohnson:mainfrom
mamun4105 wants to merge 7 commits intojandrewjohnson:mainfrom
Conversation
These are one time use functions to prepare global level input dataset for various models. A typical user does not need to use these functions as these inputs are in the base data folder
… shock These functions estimates pollination sufficiency from a lulc and then compare the scenario lulc with baseline lulc to calculate shocks depending on the crop adjusted pollination dependence. These functions are not modified much other than making it homogenous such that it can take any scenario parameter provided by seals. Previously these functions were static only for 2017 baseline and 2022 scenario
This version of InVEST SDR is modified such that it has light computation compared to InVEST SDR model. I did not run the whole SDR model because we only use usle output from SDR. Running a low computation version makes sense to save time.
…el as column name when filtering AOI This version of the function make disputed lands to project AOI. I also added a local function that is similar to hb.extract_features_in_shapefile_by_attribute but it can dissolve features if the length of features is greater than 1. Later we may consider adding this fucntion directly yo hb.
…ice and economic implications are added in the pipeline
This test run is separated from standard seals test run. It take projected CRS as input. Projected CRS is critical for running invest models
|
@jandrewjohnson I wanted to check if we should proceed with submitting this pull request to the main seals_dev repository, given that we're planning to create a standalone deliverable for the World Bank. Alternatively, should we focus on refining the SEALS_MANAGE_DEV repository (https://github.com/jandrewjohnson/seals_manage_dev) instead? Looking forward to your guidance! |
Owner
|
Hey both, sorry I didn't make it to edit the PR in time for today's
meeting. Instead, let's just go with it exactly as you have it Saleh and
then we'll address any upgrades after our meeting today. Sound okay?
…On Tue, Nov 19, 2024 at 2:11 PM Lingling Liu ***@***.***> wrote:
@jandrewjohnson <https://github.com/jandrewjohnson> I wanted to check if
we should proceed with submitting this pull request to the main seals_dev
repository, given that we're planning to create a standalone deliverable
for the World Bank. Alternatively, should we focus on refining the
SEALS_MANAGE_DEV repository (
https://github.com/jandrewjohnson/seals_manage_dev) instead?
Looking forward to your guidance!
—
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFVEPDSAH64IZDDTJB3XQET2BOLQRAVCNFSM6AAAAABR4ET57CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBWGY2TIOJXGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Justin Andrew Johnson
Assistant professor in Applied Economics, University of Minnesota
1994 Buford Ave, St Paul, MN 55108
Mobile: 1-612-961-2382, he/him
Web: justinandrewjohnson.com
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Justin,
I pulled the latest version you have and then pushed the newest edit I have. There is no conflict in the merge. All changes are handled by separate functions such that your existing functions do not change.
Key changes:
Thank you!
Saleh