-
Notifications
You must be signed in to change notification settings - Fork 71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really happy to see this. lgtm, but I won't approve, as I think CJ has a better understanding of the project parameters.
…Investing/PACTA_analysis into clean-web_tool_script_3
This also merges the AnalysisParameters file into the new Projectparameters_xxxx fil This was discussed with @cjyetman and will hopefully remove confusion. For each PACTA run what needs to be defined:
|
this works for me locally with everything as is
|
Thanks CJ. I've now removed those remaining AnalysisParameter files and added in some initial exec summary files for Liechtenstein. Otherwise the error is to be expected. |
cool... then I'm gonna approve these PRs... up to you to merge... |
Woohoo! Thanks. |
This is a significant PR with a few different functions. @jacobvjk this will allow us to have stress test functions across the scripts.
Aims:
Needs to be run in conjunction with updates to pacta-data that will be turned into a PR soon.
https://github.com/2DegreesInvesting/pacta-data/pull/19
https://github.com/2DegreesInvesting/create_interactive_report/pull/276
https://github.com/2DegreesInvesting/StressTestingModelDev/pull/121