Closed krishnakeshav closed 1 year ago
Discussed with @AaronPlex and agreed that code is near to first version. Hence, closing the PR now. It will merge the code to main branch and I will be working on incremental changes.
Thanks for this progress, Krishna. I just will put some comments here to remember what we can work on next before I forget them.
Some of the features we still need to work on include (a) assigning weights for network metrics, (b) including other ways of data such as GBIF (and which file format this data need to be), and (c) using weights for each crop when the user selects multiple crops. Additionally, check that cropharvest gives information about the HarvestedAreaFraction, not production.
I have created a new file CCRI_v2.R which reuses the functions from the original source. This wraps all the code into the functions that can be called from the shell.
SenstivityAnalysis()
is the entry point. This is still work in progress as I have not included functionalities from the last few lines of the original code. The catch for in this code is that user will have to callLoadConfig()
every-time it makes changes inparameters.yaml
Let me know your thoughts @AaronPlex @GarrettLab Once you have completed the review, I can move the code to main. This is one of the cleaner versions that I think should be brought to completion.