[{"id":4794,"web_url":"https://patchwork.libcamera.org/comment/4794/","msgid":"","date":"2020-05-12T08:25:59","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Laurent,\n\nOn 12/05/2020 01:03, Laurent Pinchart wrote:\n> W605 invalid escape sequence '\\.'\n> \n\nWould these be better interpreted as a 'raw' string?:\n\n> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> + col = re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n\nThat is how I have been handling regexes in python:\n\n https://docs.python.org/3/howto/regex.html#the-backslash-plague\n\n--\nKieran\n\n\n> Signed-off-by: Laurent Pinchart \n> ---\n> utils/raspberrypi/ctt/ctt.py | 4 ++--\n> utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n> 2 files changed, 4 insertions(+), 4 deletions(-)\n> \n> diff --git a/utils/raspberrypi/ctt/ctt.py b/utils/raspberrypi/ctt/ctt.py\n> index ff264176f040..4d8d6addf415 100755\n> --- a/utils/raspberrypi/ctt/ctt.py\n> +++ b/utils/raspberrypi/ctt/ctt.py\n> @@ -41,8 +41,8 @@ def get_col_lux(string):\n> \"\"\"\n> Extract colour and lux values from filename\n> \"\"\"\n> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> - lux = re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> + col = re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> + lux = re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> try:\n> col = col.group(1)\n> except AttributeError:\n> diff --git a/utils/raspberrypi/ctt/ctt_awb.py b/utils/raspberrypi/ctt/ctt_awb.py\n> index 58ef8432fb86..e97d833d0e49 100644\n> --- a/utils/raspberrypi/ctt/ctt_awb.py\n> +++ b/utils/raspberrypi/ctt/ctt_awb.py\n> @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n> plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n> for i, ct in enumerate(rbs_hat[2]):\n> plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n> - plt.xlabel('$\\hat{r}$')\n> - plt.ylabel('$\\hat{b}$')\n> + plt.xlabel('$\\\\hat{r}$')\n> + plt.ylabel('$\\\\hat{b}$')\n> \"\"\"\n> optional set axes equal to shortest distance so line really does\n> looks perpendicular and everybody is happy\n>","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 044CA603DF\n\tfor ;\n\tTue, 12 May 2020 10:26:03 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 6D3F233E;\n\tTue, 12 May 2020 10:26:02 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"P/PJNtnq\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589271962;\n\tbh=Gl8BjTGJOh3uvi5DFenIRlwlT9qwaPfphLQv+Olj5mw=;\n\th=Reply-To:Subject:To:References:From:Date:In-Reply-To:From;\n\tb=P/PJNtnqFZ/czzXAXgrMgoORZsnocqm3sJp02eKZKkNpAgtD/eT4cYOE9KmMqcvpl\n\tQiYtCIpeHpgLfoJYX8jN0pO6GzpaSo2fDS46h9zBo+BF4X7/uZNvgoaAyd5zXA62jP\n\t8yQxWAL9aS0I/PQZUQuDVM6QgJ30RCKj479OGk7A=","Reply-To":"kieran.bingham@ideasonboard.com","To":"Laurent Pinchart ,\n\tlibcamera-devel@lists.libcamera.org","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"","Date":"Tue, 12 May 2020 09:25:59 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.7.0","MIME-Version":"1.0","In-Reply-To":"<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"7bit","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Tue, 12 May 2020 08:26:03 -0000"}},{"id":4801,"web_url":"https://patchwork.libcamera.org/comment/4801/","msgid":"<20200512163855.GF5861@pendragon.ideasonboard.com>","date":"2020-05-12T16:38:55","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Tue, May 12, 2020 at 09:25:59AM +0100, Kieran Bingham wrote:\n> On 12/05/2020 01:03, Laurent Pinchart wrote:\n> > W605 invalid escape sequence '\\.'\n> > \n> \n> Would these be better interpreted as a 'raw' string?:\n> \n> > - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> > + col = re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> \n> That is how I have been handling regexes in python:\n> \n> https://docs.python.org/3/howto/regex.html#the-backslash-plague\n\nIt's a good point. I'll submit a new version.\n\nWould you do the same for the strings in ctt_awb.py, or just for the\nregexp ?\n\n> > Signed-off-by: Laurent Pinchart \n> > ---\n> > utils/raspberrypi/ctt/ctt.py | 4 ++--\n> > utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n> > 2 files changed, 4 insertions(+), 4 deletions(-)\n> > \n> > diff --git a/utils/raspberrypi/ctt/ctt.py b/utils/raspberrypi/ctt/ctt.py\n> > index ff264176f040..4d8d6addf415 100755\n> > --- a/utils/raspberrypi/ctt/ctt.py\n> > +++ b/utils/raspberrypi/ctt/ctt.py\n> > @@ -41,8 +41,8 @@ def get_col_lux(string):\n> > \"\"\"\n> > Extract colour and lux values from filename\n> > \"\"\"\n> > - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> > - lux = re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> > + col = re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> > + lux = re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> > try:\n> > col = col.group(1)\n> > except AttributeError:\n> > diff --git a/utils/raspberrypi/ctt/ctt_awb.py b/utils/raspberrypi/ctt/ctt_awb.py\n> > index 58ef8432fb86..e97d833d0e49 100644\n> > --- a/utils/raspberrypi/ctt/ctt_awb.py\n> > +++ b/utils/raspberrypi/ctt/ctt_awb.py\n> > @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n> > plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n> > for i, ct in enumerate(rbs_hat[2]):\n> > plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n> > - plt.xlabel('$\\hat{r}$')\n> > - plt.ylabel('$\\hat{b}$')\n> > + plt.xlabel('$\\\\hat{r}$')\n> > + plt.ylabel('$\\\\hat{b}$')\n> > \"\"\"\n> > optional set axes equal to shortest distance so line really does\n> > looks perpendicular and everybody is happy","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id EE300603E0\n\tfor ;\n\tTue, 12 May 2020 18:39:02 +0200 (CEST)","from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi\n\t[81.175.216.236])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 65DB151F;\n\tTue, 12 May 2020 18:39:02 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"tSx35gF8\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589301542;\n\tbh=CVLbXwyONEkyusMUJSR+EyyioNCGMCq5lM/2kPAuBc4=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=tSx35gF87vvemCOY9vRpEbJb837EKGuCMrPcTLSwlU62gTGcEoA/WTTK0lVvMd+bA\n\tbcUvtFcZO1hKsZQ5l5hhKrWs1N4t3+oZGTH1Jt7XTaF6dm7kbu3cx9Z5XkcipT0lY5\n\ttes+n8xWpt5dF7N4V9aDsRJsl2Ibi5urS3qSO55Y=","Date":"Tue, 12 May 2020 19:38:55 +0300","From":"Laurent Pinchart ","To":"Kieran Bingham ","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20200512163855.GF5861@pendragon.ideasonboard.com>","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Tue, 12 May 2020 16:39:03 -0000"}},{"id":4817,"web_url":"https://patchwork.libcamera.org/comment/4817/","msgid":"","date":"2020-05-14T09:08:43","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Laurent\n\nOn 12/05/2020 17:38, Laurent Pinchart wrote:\n> Hi Kieran,\n> \n> On Tue, May 12, 2020 at 09:25:59AM +0100, Kieran Bingham wrote:\n>> On 12/05/2020 01:03, Laurent Pinchart wrote:\n>>> W605 invalid escape sequence '\\.'\n>>>\n>>\n>> Would these be better interpreted as a 'raw' string?:\n>>\n>>> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n>>> + col = re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n>>\n>> That is how I have been handling regexes in python:\n>>\n>> https://docs.python.org/3/howto/regex.html#the-backslash-plague\n> \n> It's a good point. I'll submit a new version.\n> \n> Would you do the same for the strings in ctt_awb.py, or just for the\n> regexp ?\n\nthe 'r' prefix is 'raw' not 'regexp', so I believe it applies to the\nct_awb.py edits too.\n\nMight be worth testing the change somehow though to make sure it still\nworks with the matplot component.\n\nI don't see a reason for it not to work ... but still.\n\n--\nRegards\n\nKieran\n\n\n>>> Signed-off-by: Laurent Pinchart \n>>> ---\n>>> utils/raspberrypi/ctt/ctt.py | 4 ++--\n>>> utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n>>> 2 files changed, 4 insertions(+), 4 deletions(-)\n>>>\n>>> diff --git a/utils/raspberrypi/ctt/ctt.py b/utils/raspberrypi/ctt/ctt.py\n>>> index ff264176f040..4d8d6addf415 100755\n>>> --- a/utils/raspberrypi/ctt/ctt.py\n>>> +++ b/utils/raspberrypi/ctt/ctt.py\n>>> @@ -41,8 +41,8 @@ def get_col_lux(string):\n>>> \"\"\"\n>>> Extract colour and lux values from filename\n>>> \"\"\"\n>>> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n>>> - lux = re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n>>> + col = re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n>>> + lux = re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n>>> try:\n>>> col = col.group(1)\n>>> except AttributeError:\n>>> diff --git a/utils/raspberrypi/ctt/ctt_awb.py b/utils/raspberrypi/ctt/ctt_awb.py\n>>> index 58ef8432fb86..e97d833d0e49 100644\n>>> --- a/utils/raspberrypi/ctt/ctt_awb.py\n>>> +++ b/utils/raspberrypi/ctt/ctt_awb.py\n>>> @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n>>> plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n>>> for i, ct in enumerate(rbs_hat[2]):\n>>> plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n>>> - plt.xlabel('$\\hat{r}$')\n>>> - plt.ylabel('$\\hat{b}$')\n>>> + plt.xlabel('$\\\\hat{r}$')\n>>> + plt.ylabel('$\\\\hat{b}$')\n>>> \"\"\"\n>>> optional set axes equal to shortest distance so line really does\n>>> looks perpendicular and everybody is happy\n>","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 81FE160DE6\n\tfor ;\n\tThu, 14 May 2020 11:08:48 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 773A99A8;\n\tThu, 14 May 2020 11:08:47 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"gsjBRITM\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589447328;\n\tbh=TFRT5iWVzKMRMY5gztAF37w7uVaAFF4dn+oWRaEeidQ=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=gsjBRITMmfyI10sw3dQNbKwQdg9qXnIbV9OsIwiQn26BTFbE8ndA1z3ibfqBQAB6N\n\t58Y0NF0f6nxuLLlUYw9vHjdrBlY01sPRhYfVwkf3K8Os4zESy/U54znGwzs2++AEdv\n\t7sL8EdhyBX+kTe3F84lX18YYAfZgXYrdhlI1x+Jo=","Reply-To":"kieran.bingham@ideasonboard.com","To":"Laurent Pinchart ","Cc":"libcamera-devel@lists.libcamera.org","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>\n\t\n\t<20200512163855.GF5861@pendragon.ideasonboard.com>","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"","Date":"Thu, 14 May 2020 10:08:43 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.7.0","MIME-Version":"1.0","In-Reply-To":"<20200512163855.GF5861@pendragon.ideasonboard.com>","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"7bit","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 14 May 2020 09:08:48 -0000"}},{"id":4819,"web_url":"https://patchwork.libcamera.org/comment/4819/","msgid":"","date":"2020-05-14T09:47:35","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":42,"url":"https://patchwork.libcamera.org/api/people/42/","name":"David Plowman","email":"david.plowman@raspberrypi.com"},"content":"Hi\n\nThanks everyone for fixing up all this stuff. (If I'd known there was an\nofficial \"pycodestyle\" I'd have done it myself ages ago!).\n\nI actually applied all 24 of these patches including this one, yesterday,\nand re-ran the tool to check that it still works. And it did, so I gave it\nthe nod!\n\nRegards\n\nDavid\n\nOn Thu, 14 May 2020 at 10:08, Kieran Bingham <\nkieran.bingham@ideasonboard.com> wrote:\n\n> Hi Laurent\n>\n> On 12/05/2020 17:38, Laurent Pinchart wrote:\n> > Hi Kieran,\n> >\n> > On Tue, May 12, 2020 at 09:25:59AM +0100, Kieran Bingham wrote:\n> >> On 12/05/2020 01:03, Laurent Pinchart wrote:\n> >>> W605 invalid escape sequence '\\.'\n> >>>\n> >>\n> >> Would these be better interpreted as a 'raw' string?:\n> >>\n> >>> - col =\n> re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> + col =\n> re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>\n> >> That is how I have been handling regexes in python:\n> >>\n> >> https://docs.python.org/3/howto/regex.html#the-backslash-plague\n> >\n> > It's a good point. I'll submit a new version.\n> >\n> > Would you do the same for the strings in ctt_awb.py, or just for the\n> > regexp ?\n>\n> the 'r' prefix is 'raw' not 'regexp', so I believe it applies to the\n> ct_awb.py edits too.\n>\n> Might be worth testing the change somehow though to make sure it still\n> works with the matplot component.\n>\n> I don't see a reason for it not to work ... but still.\n>\n> --\n> Regards\n>\n> Kieran\n>\n>\n> >>> Signed-off-by: Laurent Pinchart \n> >>> ---\n> >>> utils/raspberrypi/ctt/ctt.py | 4 ++--\n> >>> utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n> >>> 2 files changed, 4 insertions(+), 4 deletions(-)\n> >>>\n> >>> diff --git a/utils/raspberrypi/ctt/ctt.py\n> b/utils/raspberrypi/ctt/ctt.py\n> >>> index ff264176f040..4d8d6addf415 100755\n> >>> --- a/utils/raspberrypi/ctt/ctt.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt.py\n> >>> @@ -41,8 +41,8 @@ def get_col_lux(string):\n> >>> \"\"\"\n> >>> Extract colour and lux values from filename\n> >>> \"\"\"\n> >>> - col =\n> re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> - lux =\n> re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> + col =\n> re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> + lux =\n> re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> try:\n> >>> col = col.group(1)\n> >>> except AttributeError:\n> >>> diff --git a/utils/raspberrypi/ctt/ctt_awb.py\n> b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> index 58ef8432fb86..e97d833d0e49 100644\n> >>> --- a/utils/raspberrypi/ctt/ctt_awb.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n> >>> plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n> >>> for i, ct in enumerate(rbs_hat[2]):\n> >>> plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n> >>> - plt.xlabel('$\\hat{r}$')\n> >>> - plt.ylabel('$\\hat{b}$')\n> >>> + plt.xlabel('$\\\\hat{r}$')\n> >>> + plt.ylabel('$\\\\hat{b}$')\n> >>> \"\"\"\n> >>> optional set axes equal to shortest distance so line really\n> does\n> >>> looks perpendicular and everybody is happy\n> >\n>\n> --\n> Regards\n> --\n> Kieran\n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel\n>","headers":{"Return-Path":"","Received":["from mail-ot1-x343.google.com (mail-ot1-x343.google.com\n\t[IPv6:2607:f8b0:4864:20::343])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 6CBAF60DE6\n\tfor ;\n\tThu, 14 May 2020 11:47:47 +0200 (CEST)","by mail-ot1-x343.google.com with SMTP id c3so1818818otr.12\n\tfor ;\n\tThu, 14 May 2020 02:47:47 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=raspberrypi.com\n\theader.i=@raspberrypi.com\n\theader.b=\"HPAqHhJJ\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=KTXLlbChj/yUl6ZRF7csaySyMNBLdJVEbScIA8TecEc=;\n\tb=HPAqHhJJB3rvZW7S8IXNU1+bfpf2bYgJENldmgvEZKDLP8hs3gSRycUOHYVA+D+CWM\n\tI4poJD9IglRnHrR7kEbZCiLtDq7R4ZS7EtHdtVskQsbzE16EuamLDfdUly881RMHhm3F\n\tH4dxvMBq9A7sG2dTgX+i47bQEze+uR+Tii0A91JF2eFosJh+stV8l4/swUh1brKfFF3/\n\tKK38z5jjwkeyhfG2p9+576QaQuqd53phPJE4emECC5ayV8gdetnu2jy8lqqm3FOaIHUF\n\tQLMVMTEPIvy1KJXFsWTlcKt+ivIHZkWevYKBRyAMTbHwWqKrVMBIpyR3rUgfhIra4aFs\n\tEo3A==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=KTXLlbChj/yUl6ZRF7csaySyMNBLdJVEbScIA8TecEc=;\n\tb=SXSJiczYF8mZ3OKOHUfWpZ6ePNas+5I/W4cH82I3W6z36yfIyU1AnYf3RCuvHqHxiO\n\tH6bl+wAYWQuPoPkwY6AMfIqDpUgQrKp2rc4aaleT80qb6WrA/eJhIjIMYvDy7bi5Xigc\n\tlQI18RRlE414hHt/qpXWj+O5YeT+FImQAoMAGjK+eYHSfajJNY5uBczVeI8M8YIreyAx\n\t9tisfssc0CgOWxTNiFJ8O3Xvo64VfxIWVfsx8462+ubwT8x0GxdDUvl1EiuiObuJZOUL\n\tc1XBoJHGXzrcA1vCSXHZWnunpBgJPkQMe0h8bks+bSCBRa4FxFlqEzG5QjnZtPcBGD5E\n\thzlQ==","X-Gm-Message-State":"AOAM532KLffmpdZcd5biRKSO7tcrYBdcMntMMTfXwEWXM5DPOi6aWHjm\n\tkrWp3jNAH7wcj7O0xoTNyGxspYBuJ5Vzdi8olAzc+RsA","X-Google-Smtp-Source":"ABdhPJzaHF9ad2k1D5wF/eKoXcu0jHVim3sgawAj8eWt99LBSN4LM7iwx4DFf8aoSaN9+1L+vddPnTsVZShQIoSMAWc=","X-Received":"by 2002:a05:6830:158b:: with SMTP id\n\ti11mr2983873otr.160.1589449665734; \n\tThu, 14 May 2020 02:47:45 -0700 (PDT)","MIME-Version":"1.0","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>\n\t\n\t<20200512163855.GF5861@pendragon.ideasonboard.com>\n\t","In-Reply-To":"","From":"David Plowman ","Date":"Thu, 14 May 2020 10:47:35 +0100","Message-ID":"","To":"Kieran Bingham ","Cc":"Laurent Pinchart ,\n\tlibcamera-devel@lists.libcamera.org","Content-Type":"multipart/alternative; boundary=\"000000000000b719d505a5989603\"","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 14 May 2020 09:47:47 -0000"}},{"id":4820,"web_url":"https://patchwork.libcamera.org/comment/4820/","msgid":"<50e35e77-9441-3607-d4ea-6aacffb1c771@ideasonboard.com>","date":"2020-05-14T09:50:33","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi David,\n\nOn 14/05/2020 10:47, David Plowman wrote:\n> Hi\n> \n> Thanks everyone for fixing up all this stuff. (If I'd known there was an\n> official \"pycodestyle\" I'd have done it myself ages ago!).\n\nIt used to be called 'pep8' ... but got renamed to a more\nuseful/representative pycodestyle ;-)\n\n\n> I actually applied all 24 of these patches including this one,\n> yesterday, and re-ran the tool to check that it still works. And it did,\n> so I gave it the nod!\n\nGreat ;-)\n\n--\nKieran\n\n\n> \n> Regards\n> \n> David\n> \n> On Thu, 14 May 2020 at 10:08, Kieran Bingham\n> > wrote:\n> \n> Hi Laurent\n> \n> On 12/05/2020 17:38, Laurent Pinchart wrote:\n> > Hi Kieran,\n> >\n> > On Tue, May 12, 2020 at 09:25:59AM +0100, Kieran Bingham wrote:\n> >> On 12/05/2020 01:03, Laurent Pinchart wrote:\n> >>> W605 invalid escape sequence '\\.'\n> >>>\n> >>\n> >> Would these be better interpreted as a 'raw' string?:\n> >>\n> >>> - col =\n> re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> + col =\n> re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>\n> >> That is how I have been handling regexes in python:\n> >>\n> >>   https://docs.python.org/3/howto/regex.html#the-backslash-plague\n> >\n> > It's a good point. I'll submit a new version.\n> >\n> > Would you do the same for the strings in ctt_awb.py, or just for the\n> > regexp ?\n> \n> the 'r' prefix is 'raw' not 'regexp', so I believe it applies to the\n> ct_awb.py edits too.\n> \n> Might be worth testing the change somehow though to make sure it still\n> works with the matplot component.\n> \n> I don't see a reason for it not to work ... but still.\n> \n> --\n> Regards\n> \n> Kieran\n> \n> \n> >>> Signed-off-by: Laurent Pinchart\n> >\n> >>> ---\n> >>>  utils/raspberrypi/ctt/ctt.py     | 4 ++--\n> >>>  utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n> >>>  2 files changed, 4 insertions(+), 4 deletions(-)\n> >>>\n> >>> diff --git a/utils/raspberrypi/ctt/ctt.py\n> b/utils/raspberrypi/ctt/ctt.py\n> >>> index ff264176f040..4d8d6addf415 100755\n> >>> --- a/utils/raspberrypi/ctt/ctt.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt.py\n> >>> @@ -41,8 +41,8 @@ def get_col_lux(string):\n> >>>      \"\"\"\n> >>>      Extract colour and lux values from filename\n> >>>      \"\"\"\n> >>> -    col =\n> re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> -    lux =\n> re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> +    col =\n> re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>> +    lux =\n> re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$',\n> string)\n> >>>      try:\n> >>>          col = col.group(1)\n> >>>      except AttributeError:\n> >>> diff --git a/utils/raspberrypi/ctt/ctt_awb.py\n> b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> index 58ef8432fb86..e97d833d0e49 100644\n> >>> --- a/utils/raspberrypi/ctt/ctt_awb.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n> >>>          plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n> >>>          for i, ct in enumerate(rbs_hat[2]):\n> >>>              plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n> >>> -        plt.xlabel('$\\hat{r}$')\n> >>> -        plt.ylabel('$\\hat{b}$')\n> >>> +        plt.xlabel('$\\\\hat{r}$')\n> >>> +        plt.ylabel('$\\\\hat{b}$')\n> >>>          \"\"\"\n> >>>          optional set axes equal to shortest distance so line\n> really does\n> >>>          looks perpendicular and everybody is happy\n> >\n> \n> -- \n> Regards\n> --\n> Kieran\n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> \n> https://lists.libcamera.org/listinfo/libcamera-devel\n>","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 169E860DE6\n\tfor ;\n\tThu, 14 May 2020 11:50:38 +0200 (CEST)","from [192.168.0.20]\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 413E19A8;\n\tThu, 14 May 2020 11:50:37 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"pFsa7meB\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589449837;\n\tbh=E2myQv/KJBdtkerN++/3Cd+De+YZJD1oCU2ysKg9yRI=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=pFsa7meBxQzLrFG6Kh7p0FigBR2V49u9HSxJoy0CzTi/GyizkpW5cU6JzclfE2lGK\n\tWRZaTzKHwXG+zlxuoZUdMEAfQA8NEQCvwnMoRghkvU+m2EnIVUXQe2MoF5TE29MzJ/\n\tJeTUytuT2PuBLlitbhTtehsv2mo1OmGoG5Jcmduk=","Reply-To":"kieran.bingham@ideasonboard.com","To":"David Plowman ","Cc":"Laurent Pinchart ,\n\tlibcamera-devel@lists.libcamera.org","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>\n\t\n\t<20200512163855.GF5861@pendragon.ideasonboard.com>\n\t\n\t","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"<50e35e77-9441-3607-d4ea-6aacffb1c771@ideasonboard.com>","Date":"Thu, 14 May 2020 10:50:33 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.7.0","MIME-Version":"1.0","In-Reply-To":"","Content-Type":"text/plain; charset=utf-8","Content-Language":"en-GB","Content-Transfer-Encoding":"8bit","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Thu, 14 May 2020 09:50:38 -0000"}},{"id":4831,"web_url":"https://patchwork.libcamera.org/comment/4831/","msgid":"<20200517144143.GD5828@pendragon.ideasonboard.com>","date":"2020-05-17T14:41:43","subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Kieran,\n\nOn Thu, May 14, 2020 at 10:08:43AM +0100, Kieran Bingham wrote:\n> On 12/05/2020 17:38, Laurent Pinchart wrote:\n> > On Tue, May 12, 2020 at 09:25:59AM +0100, Kieran Bingham wrote:\n> >> On 12/05/2020 01:03, Laurent Pinchart wrote:\n> >>> W605 invalid escape sequence '\\.'\n> >>>\n> >>\n> >> Would these be better interpreted as a 'raw' string?:\n> >>\n> >>> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> >>> + col = re.search(r'([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> >>\n> >> That is how I have been handling regexes in python:\n> >>\n> >> https://docs.python.org/3/howto/regex.html#the-backslash-plague\n> > \n> > It's a good point. I'll submit a new version.\n> > \n> > Would you do the same for the strings in ctt_awb.py, or just for the\n> > regexp ?\n> \n> the 'r' prefix is 'raw' not 'regexp', so I believe it applies to the\n> ct_awb.py edits too.\n\nYes, I know it could. It's mostly useful for regexp as you know only\nliteral \\ are needed. For other strings that need both literal \\ and\nescaped characters, r can't be used. I'm thus wondering if a best\npractice would be to avoid it for non-regexp strings that don't make\nlarge usage of literal \\.\n\n> Might be worth testing the change somehow though to make sure it still\n> works with the matplot component.\n> \n> I don't see a reason for it not to work ... but still.\n>\n> >>> Signed-off-by: Laurent Pinchart \n> >>> ---\n> >>> utils/raspberrypi/ctt/ctt.py | 4 ++--\n> >>> utils/raspberrypi/ctt/ctt_awb.py | 4 ++--\n> >>> 2 files changed, 4 insertions(+), 4 deletions(-)\n> >>>\n> >>> diff --git a/utils/raspberrypi/ctt/ctt.py b/utils/raspberrypi/ctt/ctt.py\n> >>> index ff264176f040..4d8d6addf415 100755\n> >>> --- a/utils/raspberrypi/ctt/ctt.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt.py\n> >>> @@ -41,8 +41,8 @@ def get_col_lux(string):\n> >>> \"\"\"\n> >>> Extract colour and lux values from filename\n> >>> \"\"\"\n> >>> - col = re.search('([0-9]+)[kK](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> >>> - lux = re.search('([0-9]+)[lL](\\.(jpg|jpeg|brcm|dng)|_.*\\.(jpg|jpeg|brcm|dng))$', string)\n> >>> + col = re.search('([0-9]+)[kK](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> >>> + lux = re.search('([0-9]+)[lL](\\\\.(jpg|jpeg|brcm|dng)|_.*\\\\.(jpg|jpeg|brcm|dng))$', string)\n> >>> try:\n> >>> col = col.group(1)\n> >>> except AttributeError:\n> >>> diff --git a/utils/raspberrypi/ctt/ctt_awb.py b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> index 58ef8432fb86..e97d833d0e49 100644\n> >>> --- a/utils/raspberrypi/ctt/ctt_awb.py\n> >>> +++ b/utils/raspberrypi/ctt/ctt_awb.py\n> >>> @@ -256,8 +256,8 @@ def awb(Cam, cal_cr_list, cal_cb_list, plot):\n> >>> plt.scatter(rbs_hat[0], rbs_hat[1], color='red')\n> >>> for i, ct in enumerate(rbs_hat[2]):\n> >>> plt.annotate(str(ct), (rbs_hat[0][i], rbs_hat[1][i]))\n> >>> - plt.xlabel('$\\hat{r}$')\n> >>> - plt.ylabel('$\\hat{b}$')\n> >>> + plt.xlabel('$\\\\hat{r}$')\n> >>> + plt.ylabel('$\\\\hat{b}$')\n> >>> \"\"\"\n> >>> optional set axes equal to shortest distance so line really does\n> >>> looks perpendicular and everybody is happy","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 2FB0E60DF9\n\tfor ;\n\tSun, 17 May 2020 16:41:52 +0200 (CEST)","from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi\n\t[81.175.216.236])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 954AB258;\n\tSun, 17 May 2020 16:41:51 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"kdXB2cu/\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589726511;\n\tbh=kT3oqxIVPRC8w0/VBV0zbp/7M6ELNRoK2vGeIbOX2Sw=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=kdXB2cu/VmBZdCr2xwzYPtATJij+xcIMwOC4tGS1KJFKsgneY8lGz2K/HFmpfY0aT\n\tFox02eMiyLowQceEOhoBmGVBaY7dZIkDQxcZpLpuuIoA25XnteSyoEDI9Trk4w/tRC\n\ty7achDkXmCyXH+z6WBboPbhpeSqd4oVR/hNZZWyA=","Date":"Sun, 17 May 2020 17:41:43 +0300","From":"Laurent Pinchart ","To":"Kieran Bingham ","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20200517144143.GD5828@pendragon.ideasonboard.com>","References":"<20200512000322.11753-1-laurent.pinchart@ideasonboard.com>\n\t<20200512000322.11753-14-laurent.pinchart@ideasonboard.com>\n\t\n\t<20200512163855.GF5861@pendragon.ideasonboard.com>\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH 13/24] utils: raspberrypi: ctt: Fix\n\tpycodestyle W605","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Sun, 17 May 2020 14:41:52 -0000"}}]