From 6648978ad655f43cc2fb97b5684e16c4b3516795 Mon Sep 17 00:00:00 2001 From: Fati-Mon <150450583+Fati-Mon@users.noreply.github.com> Date: Tue, 22 Oct 2024 14:54:34 +0200 Subject: [PATCH] Update natural-flow.ipynb Adjusted the text a little, otherwise the different averages from different datasets may cause confusion by users. --- docs/tutorial/natural-flow.ipynb | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/docs/tutorial/natural-flow.ipynb b/docs/tutorial/natural-flow.ipynb index 3807769bf..c88ded66c 100644 --- a/docs/tutorial/natural-flow.ipynb +++ b/docs/tutorial/natural-flow.ipynb @@ -48,7 +48,7 @@ "source": [ "# Crystal River Basin\n", "We will examine a straightforward example of the Crystal river basin, which includes a main river and a single tributary flowing into the sea (see @fig-crystal-basin).\n", - "An average discharge of $44.45 \\text{ m}^3/\\text{s}$ is measured at the confluence.\n", + "Between 2014 and 2023 the average discharge of $44.45 \\text{ m}^3/\\text{s}$ is measured at the confluence.\n", "In this module, the basin is free of any activities, allowing the model to simulate the natural flow.\n", "The next step is to include a demand (irrigation) that taps from a canal out of the main river.\n", "\n", @@ -89,7 +89,9 @@ "metadata": {}, "source": [ "### Setup paths and model configuration\n", - "Reference the paths of the Ribasim installation and model directory and define the time period (2022-01-01 until 2023-01-01) for the model simulation.\n", + "Reference the paths of the Ribasim installation and model directory and define the time period. \n" + "The used simulation period is defined by the `starttime` and `endtime` of the model, not by the input timeseries. \n" + "For now we will look into the years from 2022-01-01 until 2023-01-01 for the model simulation.\n", "The coordinate reference system (CRS) is also required, and set to [EPSG:4326](https://epsg.io/4326), which means all coordinates are interpreted as latitude and longitude values.\n", "The CRS is important for correctly placing Ribasim models on the map, but since this is a fictional model, it is not important." ] @@ -117,8 +119,8 @@ "source": [ "### FlowBoundary nodes\n", "The Crystal basin consists of two inflow points, the tributary and the main Crystal river, we will call them Minor and Main respectively.\n", - "This is a monthly inflow timeseries from 2014 to 2023.\n", - "The used simulation period is defined by the `starttime` and `endtime` of the model, not by the input timeseries." + "The .csv file provided in the .zip folder contains the monthly inflow timeseries from 2014 to 2023.\n", + "You could read the .csv directly as a dataframe or add the flows between 2022 and 2023 manually as follows: " ] }, { @@ -135,8 +137,8 @@ "data[\"total\"] = data[\"minor\"] + data[\"main\"]\n", "display(data)\n", "\n", - "# Average and max inflow of the total inflow data timeseries\n", - "# From 2014 - 2023\n", + "# Average and max inflow of the total inflow data implemented\n", + "# From 2022 - 2023\n", "print(\"Average inflow [m3/s]:\", data[\"total\"].mean())\n", "print(\"Maximum inflow [m3/s]:\", data[\"total\"].max())\n", "\n",