IMMM-SFA / gamut

An R package to identify multi-sector teleconnection complexity
https://immm-sfa.github.io/gamut/
Other
0 stars 3 forks source link

Jackson MS fix #90

Closed swd-turner closed 3 years ago

swd-turner commented 4 years ago

Jackson MS needs a fix on the runoff data to update for the key watershed (now 3743, not 3683). This is addressed currently with a temporary fix in count_watershed_data.R

KristianNelson commented 4 years ago

Does this mean we need to run the PIC function again in order to get the new runoff calculation? Since it is currently using the older watershed shape.

swd-turner commented 4 years ago

Yea, if you run Jackson MS you'll find it fails because it can't grab a runoff time series associated with the new key watershed. My temporary fix has been to revert to the old watershed for the runoff only (which is a reasonable assumption). This issue is not urgent but something we should address along with issue #85 I reckon