Skip to content

Commit fd49561

Browse files
RolfHutsverhoevenPeter9192
authored
Finish case3 (#29)
* Changed strat-time in use case 3 to be in line with use case 2 * text changes to notebook 3. Needs re-running but is taking (too?) long to start container on research cloud * oops. This commit is without the output folder! * fixed notebook 3 * finished text on case 3 and case 4. Needs final run to generate results and have cell-counts be nice in order. Once that is done, branch can be merged back into main. * Case4 output (#27) * Make case2 fair (#25) * tried to make notebook 2 FAIR, failed on technicallities. Try again after weekend * Found the porblem: using configparser creates .ini files that are byte for byte identical, but create an error when calling initialize(). Raised an issue for this, not something I can fix. * Fixed problem: used work-around * final edits to text of notebook2 before pull request * fixed and made FAIR case 2. Solved issue with configParser (thanks Stefan!) * Added output files from case 4 * Reran case 4 Co-authored-by: Rolf Hut <[email protected]> * moved files from case 5. Had to change filenames for 3 and 4 into 4 and 5 because Overleaf doesn't allow lists of sections to start at zero. Will change file names of other notebooks in other branch. * finished case 3 (preivious case 2) by moving figure, changing a few lines in the notebook. Also fixed a final problem in notebook 4 (file names of figures) * Rename Case0 to Case1 * Rename case1 to case2 * rename case5 to Case5 (capital) * Remove duplicate Case4 Co-authored-by: Stefan Verhoeven <[email protected]> Co-authored-by: Peter Kalverla <[email protected]>
1 parent 5bc031a commit fd49561

9 files changed

+7
-7
lines changed
File renamed without changes.
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"cell_type": "markdown",
1212
"metadata": {},
1313
"source": [
14-
"# Case study 2: replace subbasin in PCRGlobWB2.0 with a MARRMoT model\n",
14+
"# Case study 3: replace subbasin in PCRGlobWB2.0 with a MARRMoT model\n",
1515
"This notebooks demonstrates how to use eWaterCycle to combine the output of two very different models in a single experiment. We run PCRGlobWB2.0 for the Rhine basin, but we replace the Moselle sub-basin with a MARRMoT model.\n",
1616
"\n",
1717
"PCRGlobWB2.0 (Sutanudjaja et al 2017, [10.5194/gmd-11-2429-2018](https://doi.org/10.5194/gmd-11-2429-2018)) is a distributed model written in python and MARRMoT (Knoben et al 2019, [10.5194/gmd-12-2463-2019](https://doi.org/10.5194/gmd-12-2463-2019)) is a suite of conceptual models written in Matlab. To make the difference as large as possible, we choose the simplest model available within MARRMoT: the m01, a single leaky bucket. This example is neither meant to show that replacing the entire Moselle sub-basin by a single leaky bucket is better than using PCRGlobWB2.0, nor that PCRGlobWB2.0 is better than MARRMoT. This example is to show the capabillities of the eWaterCycle platform by coupling two models with different philosophies (lumped versus distributed) and different programming languages (python versus Matlab) into a single computational hydrological experiment.\n",
@@ -815,7 +815,7 @@
815815
"# Overlay ocean and coastines\n",
816816
"ax.add_feature(cfeature.OCEAN, zorder=2)\n",
817817
"ax.coastlines(zorder=3)\n",
818-
"fig.savefig(f\"pcrglobwb_RolfTestRhine_discharge_map\", bbox_inches=\"tight\", dpi=300)"
818+
"fig.savefig(f\"figures/Case3_Rhine_discharge_map\", bbox_inches=\"tight\", dpi=300)"
819819
]
820820
},
821821
{
@@ -869,14 +869,14 @@
869869
"ewatercycle.analysis.hydrograph(\n",
870870
" discharge=df,\n",
871871
" reference=\"Observations from GRDC\",\n",
872-
" filename=\"case2CouplingModels\",\n",
872+
" filename=\"figures/case3_CouplingModels\",\n",
873873
")"
874874
]
875875
}
876876
],
877877
"metadata": {
878878
"kernelspec": {
879-
"display_name": "Python 3",
879+
"display_name": "Python 3 (ipykernel)",
880880
"language": "python",
881881
"name": "python3"
882882
},
@@ -890,7 +890,7 @@
890890
"name": "python",
891891
"nbconvert_exporter": "python",
892892
"pygments_lexer": "ipython3",
893-
"version": "3.7.7"
893+
"version": "3.9.7"
894894
}
895895
},
896896
"nbformat": 4,

Case4ForcePCRGlobWBWithadditionalEvapInfo.ipynb renamed to Case4_ForcePCRGlobWBWithadditionalEvapInfo.ipynb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -668,7 +668,7 @@
668668
"ax.coastlines(zorder=3)\n",
669669
"\n",
670670
"fig.savefig(\n",
671-
" f\"figures/Case3_ForcePCRGlobWithAdditionalDataMap\", bbox_inches=\"tight\", dpi=300\n",
671+
" f\"figures/Case4_ForcePCRGlobWithAdditionalDataMap\", bbox_inches=\"tight\", dpi=300\n",
672672
")"
673673
]
674674
},
@@ -710,7 +710,7 @@
710710
}
711711
],
712712
"source": [
713-
"ewatercycle.analysis.hydrograph(discharge_data, reference=\"GRDC\", filename=\"figures/Case3_ForcePCRGlobWithAdditionalDataHydrograph\")"
713+
"ewatercycle.analysis.hydrograph(discharge_data, reference=\"GRDC\", filename=\"figures/Case4_ForcePCRGlobWithAdditionalDataHydrograph\")"
714714
]
715715
},
716716
{
File renamed without changes.
117 KB
Loading
75.8 KB
Loading
135 KB
Loading

figures/case3_CouplingModels.png

110 KB
Loading

0 commit comments

Comments
 (0)