vicelab / cen-sierra-pywr

Code base for modeling the central Sierra Nevada hydropower systems
3 stars 5 forks source link

JSON file in USJ needs revision #128

Open mlmaskey opened 3 years ago

mlmaskey commented 3 years ago

There exist the following discrepancies among the nodes or links defined in the pywr_model.json file for Upper San Joaquin River and openagua:

After fixing all the links, the model run becomes infeasible. So, it ought to stick to the default case. @rheinheimer , it seems we need to fix the network or JSON file to get the appropriate result.

drheinheimer commented 3 years ago

@mlmaskey, thanks for this. A few points to note:

  1. The naming is indeed a mess. This is a result of pulling from the WEAP models (in which there is no “Big Creek 5 Diversion Reservoir”, for example, just a diversion from the river), and having some auto-naming scheme in OpenAgua (rather than having to name every single diversion/junction/split link manually).
  2. Absolutely none of this affects model performance/results. All needed facilities are included, even if they don't currently align 1:1 with what is in the OpenAgua schematic.
  3. While we could manually change the Pywr JSON file to map 1:1 with what is in OpenAgua, I'm not sure it's a good use of time, since a) see 2. above, and b) there is an additional processing of the original model JSON file to simplify the network, effectively removing most of these nodes that we'd be adding in.

HOWEVER, it might be worth renaming those key facilities that are currently mis-named in the JSON file, such as the Big Creek 5 Diversion Reservoir (possibly all diversion reservoirs are so misnamed)