Skip to content

Commit 0d20bcd

Browse files
committed
Removed some TODOs from section 2.4
1 parent fa2fa78 commit 0d20bcd

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

Diff for: notebooks/04_Processing_Long_Climate_Data_Records.ipynb

+1-6
Original file line numberDiff line numberDiff line change
@@ -1243,7 +1243,6 @@
12431243
"source": [
12441244
"%%timeit\n",
12451245
"\n",
1246-
"# TODO Note that we shouldn't try to assign any variables inside a timeit block\n",
12471246
"potential_et(first_day).compute()"
12481247
]
12491248
},
@@ -1264,8 +1263,6 @@
12641263
"metadata": {},
12651264
"outputs": [],
12661265
"source": [
1267-
"# TODO About 700 ms for a single day\n",
1268-
"\n",
12691266
"20e-3 * ds.time.size"
12701267
]
12711268
},
@@ -1463,7 +1460,7 @@
14631460
"id": "aeea53ab-fe75-4651-baf5-bfcc228b34fa",
14641461
"metadata": {},
14651462
"source": [
1466-
"**We must not forget to divide the result by 30 (days) to get the mean daily precipitation.**"
1463+
"**We must not forget to divide the result by 30 (days) to get the mean daily precipitation.** This is a simple approximation as some months don't have exactly 30 days."
14671464
]
14681465
},
14691466
{
@@ -1473,8 +1470,6 @@
14731470
"metadata": {},
14741471
"outputs": [],
14751472
"source": [
1476-
"# TODO Note that we're using a rough approximation of the number of days in a month\n",
1477-
"\n",
14781473
"chirps_tiaret_daily = chirps_tiaret_resampled.mean(['x', 'y']) / 30\n",
14791474
"chirps_tiaret_daily"
14801475
]

0 commit comments

Comments
 (0)