Fix leap year condition in monthly means example (#3464)

mickaellalande · dcherian · keewis · commit 1faf67c6e1e8 · 2019-11-21T22:10:20.000+01:00
* Typo correction in docs (#3387) * Update terminology.rst (#3455) Fixed broken link * Error in leap year? I've tried this script; however, it adds +1 to all months of the leap years. It sounds like an error, or I am wrong? So I wrote the condition "and month == 2" line 86 so that only the month of February gets +1. * Fix leap year (#3464) * Update doc/whats-new.rst Co-Authored-By: Deepak Cherian <dcherian@users.noreply.github.com>
diff --git a/doc/examples/monthly-means.ipynb b/doc/examples/monthly-means.ipynb
@@ -107,7 +107,7 @@
     "    \n",
     "    for i, (month, year) in enumerate(zip(time.month, time.year)):\n",
     "        month_length[i] = cal_days[month]\n",
-    "        if leap_year(year, calendar=calendar):\n",
+    "        if leap_year(year, calendar=calendar) and month == 2:\n",
     "            month_length[i] += 1\n",
     "    return month_length"
    ]

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@`
`107`	`107`	`" \n",`
`108`	`108`	`" for i, (month, year) in enumerate(zip(time.month, time.year)):\n",`
`109`	`109`	`" month_length[i] = cal_days[month]\n",`
`110`		`- " if leap_year(year, calendar=calendar):\n",`
	`110`	`+ " if leap_year(year, calendar=calendar) and month == 2:\n",`
`111`	`111`	`" month_length[i] += 1\n",`
`112`	`112`	`" return month_length"`
`113`	`113`	`]`