[{"id":31202,"web_url":"https://patchwork.libcamera.org/comment/31202/","msgid":"<172621836031.3474483.11988287206831473721@ping.linuxembedded.co.uk>","date":"2024-09-13T09:06:00","subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Stefan Klug (2024-09-13 08:57:20)\n> Add tests for the Interpolator class.\n> \n> Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>\n> ---\n>  test/ipa/libipa/interpolator.cpp | 55 ++++++++++++++++++++++++++++++++\n>  test/ipa/libipa/meson.build      | 15 +++++++++\n>  test/ipa/meson.build             |  1 +\n>  3 files changed, 71 insertions(+)\n>  create mode 100644 test/ipa/libipa/interpolator.cpp\n>  create mode 100644 test/ipa/libipa/meson.build\n> \n> diff --git a/test/ipa/libipa/interpolator.cpp b/test/ipa/libipa/interpolator.cpp\n> new file mode 100644\n> index 000000000000..adb215386f0e\n> --- /dev/null\n> +++ b/test/ipa/libipa/interpolator.cpp\n> @@ -0,0 +1,55 @@\n> +/* SPDX-License-Identifier: GPL-2.0-or-later */\n> +/*\n> + * Copyright (C) 2024, Paul Elder <paul.elder@ideasonboard.com>\n> + *\n> + * Miscellaneous utility tests\n> + */\n> +\n> +#include <cmath>\n> +#include <iostream>\n> +#include <map>\n> +#include <stdint.h>\n> +#include <stdio.h>\n> +\n> +#include \"../src/ipa/libipa/interpolator.h\"\n> +\n> +#include \"test.h\"\n> +\n> +using namespace std;\n> +using namespace libcamera;\n> +using namespace ipa;\n> +\n> +#define ASSERT_EQ(a, b) \\\n> +       if((a) != (b)) { printf(#a  \" != \" #b \"\\n\"); \\\n> +               return TestFail; \\\n\nWe wouldn't normally hide return statements in a macro - but given the\nsmall scope of the macro here, and how it helps improve readability, and\nwe're in tests- I think it's fine, and helpful\n\n\n\n> +       }\n> +\n> +class InterpolatorTest : public Test\n> +{\n> +protected:\n> +\n> +       int run()\n> +       {\n> +               Interpolator<int> interpolator;\n> +               interpolator.setData({{10,100},{20,200},{30,300}});\n> +\n> +               ASSERT_EQ(interpolator.getInterpolated(0), 100);\n> +               ASSERT_EQ(interpolator.getInterpolated(10), 100);\n> +               ASSERT_EQ(interpolator.getInterpolated(20), 200);\n> +               ASSERT_EQ(interpolator.getInterpolated(25), 250);\n> +               ASSERT_EQ(interpolator.getInterpolated(30), 300);\n> +               ASSERT_EQ(interpolator.getInterpolated(40), 300);\n> +\n> +               interpolator.setQuantization(10);\n> +               unsigned int q = 0;\n> +               ASSERT_EQ(interpolator.getInterpolated(25, &q), 300);\n> +               ASSERT_EQ(q, 30);\n> +               ASSERT_EQ(interpolator.getInterpolated(24, &q), 200);\n> +               ASSERT_EQ(q, 20);\n> +               \n\nnit: Delete a blank line\n\nReviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n\n> +\n> +               return TestPass;\n> +       }\n> +};\n> +\n> +TEST_REGISTER(InterpolatorTest)\n> diff --git a/test/ipa/libipa/meson.build b/test/ipa/libipa/meson.build\n> new file mode 100644\n> index 000000000000..4d2427dbd4e7\n> --- /dev/null\n> +++ b/test/ipa/libipa/meson.build\n> @@ -0,0 +1,15 @@\n> +# SPDX-License-Identifier: CC0-1.0\n> +\n> +libipa_test = [\n> +    {'name': 'interpolator', 'sources': ['interpolator.cpp']},\n> +]\n> +\n> +foreach test : libipa_test\n> +    exe = executable(test['name'], test['sources'],\n> +                     dependencies : [libcamera_private, libipa_dep],\n> +                     link_with : [test_libraries],\n> +                     include_directories : [test_includes_internal,\n> +                                            '../../../src/ipa/libipa/'])\n> +\n> +    test(test['name'], exe, suite : 'ipa')\n> +endforeach\n> diff --git a/test/ipa/meson.build b/test/ipa/meson.build\n> index e9871aba44ee..63820de54899 100644\n> --- a/test/ipa/meson.build\n> +++ b/test/ipa/meson.build\n> @@ -1,5 +1,6 @@\n>  # SPDX-License-Identifier: CC0-1.0\n>  \n> +subdir('libipa')\n>  subdir('rkisp1')\n>  \n>  ipa_test = [\n> -- \n> 2.43.0\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id E7795C3257\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 13 Sep 2024 09:06:07 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9AF11634FF;\n\tFri, 13 Sep 2024 11:06:06 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 01A44634F4\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 13 Sep 2024 11:06:04 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 4199AE0D;\n\tFri, 13 Sep 2024 11:04:45 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"M7jcQfqH\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1726218285;\n\tbh=DuTQNWzjmAwxnoP5+fwnVTQZDNPWLfkK54tN+OsDO9E=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=M7jcQfqHU88SbtMuQqDJMHCjgzwHhSLRNxXF2qHy6VXTcXxMlwddYXquHcjWesP7m\n\t295dVj7WCFhemKdD8BB1Z5fLdzAAoAlVc6oyo4q5G9wi82+Riwt87/z9APqXNtli8+\n\tlu2Bk/7IABwBF7sUIn3AHGdMpPIo70+Xzy4DmwiU=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20240913075750.35115-3-stefan.klug@ideasonboard.com>","References":"<20240913075750.35115-1-stefan.klug@ideasonboard.com>\n\t<20240913075750.35115-3-stefan.klug@ideasonboard.com>","Subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"Stefan Klug <stefan.klug@ideasonboard.com>","To":"Stefan Klug <stefan.klug@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org","Date":"Fri, 13 Sep 2024 10:06:00 +0100","Message-ID":"<172621836031.3474483.11988287206831473721@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":31203,"web_url":"https://patchwork.libcamera.org/comment/31203/","msgid":"<172621840464.3474483.7296905648022308795@ping.linuxembedded.co.uk>","date":"2024-09-13T09:06:44","subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Quoting Kieran Bingham (2024-09-13 10:06:00)\n> Quoting Stefan Klug (2024-09-13 08:57:20)\n> > Add tests for the Interpolator class.\n> > \n> > Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>\n> > ---\n> >  test/ipa/libipa/interpolator.cpp | 55 ++++++++++++++++++++++++++++++++\n> >  test/ipa/libipa/meson.build      | 15 +++++++++\n> >  test/ipa/meson.build             |  1 +\n> >  3 files changed, 71 insertions(+)\n> >  create mode 100644 test/ipa/libipa/interpolator.cpp\n> >  create mode 100644 test/ipa/libipa/meson.build\n> > \n> > diff --git a/test/ipa/libipa/interpolator.cpp b/test/ipa/libipa/interpolator.cpp\n> > new file mode 100644\n> > index 000000000000..adb215386f0e\n> > --- /dev/null\n> > +++ b/test/ipa/libipa/interpolator.cpp\n> > @@ -0,0 +1,55 @@\n> > +/* SPDX-License-Identifier: GPL-2.0-or-later */\n> > +/*\n> > + * Copyright (C) 2024, Paul Elder <paul.elder@ideasonboard.com>\n\nI think you can claim this though if you prefer ;-)\n\n> > + *\n> > + * Miscellaneous utility tests\n> > + */\n> > +\n> > +#include <cmath>\n> > +#include <iostream>\n> > +#include <map>\n> > +#include <stdint.h>\n> > +#include <stdio.h>\n> > +\n> > +#include \"../src/ipa/libipa/interpolator.h\"\n> > +\n> > +#include \"test.h\"\n> > +\n> > +using namespace std;\n> > +using namespace libcamera;\n> > +using namespace ipa;\n> > +\n> > +#define ASSERT_EQ(a, b) \\\n> > +       if((a) != (b)) { printf(#a  \" != \" #b \"\\n\"); \\\n> > +               return TestFail; \\\n> \n> We wouldn't normally hide return statements in a macro - but given the\n> small scope of the macro here, and how it helps improve readability, and\n> we're in tests- I think it's fine, and helpful\n> \n> \n> \n> > +       }\n> > +\n> > +class InterpolatorTest : public Test\n> > +{\n> > +protected:\n> > +\n> > +       int run()\n> > +       {\n> > +               Interpolator<int> interpolator;\n> > +               interpolator.setData({{10,100},{20,200},{30,300}});\n> > +\n> > +               ASSERT_EQ(interpolator.getInterpolated(0), 100);\n> > +               ASSERT_EQ(interpolator.getInterpolated(10), 100);\n> > +               ASSERT_EQ(interpolator.getInterpolated(20), 200);\n> > +               ASSERT_EQ(interpolator.getInterpolated(25), 250);\n> > +               ASSERT_EQ(interpolator.getInterpolated(30), 300);\n> > +               ASSERT_EQ(interpolator.getInterpolated(40), 300);\n> > +\n> > +               interpolator.setQuantization(10);\n> > +               unsigned int q = 0;\n> > +               ASSERT_EQ(interpolator.getInterpolated(25, &q), 300);\n> > +               ASSERT_EQ(q, 30);\n> > +               ASSERT_EQ(interpolator.getInterpolated(24, &q), 200);\n> > +               ASSERT_EQ(q, 20);\n> > +               \n> \n> nit: Delete a blank line\n> \n> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n> \n> > +\n> > +               return TestPass;\n> > +       }\n> > +};\n> > +\n> > +TEST_REGISTER(InterpolatorTest)\n> > diff --git a/test/ipa/libipa/meson.build b/test/ipa/libipa/meson.build\n> > new file mode 100644\n> > index 000000000000..4d2427dbd4e7\n> > --- /dev/null\n> > +++ b/test/ipa/libipa/meson.build\n> > @@ -0,0 +1,15 @@\n> > +# SPDX-License-Identifier: CC0-1.0\n> > +\n> > +libipa_test = [\n> > +    {'name': 'interpolator', 'sources': ['interpolator.cpp']},\n> > +]\n> > +\n> > +foreach test : libipa_test\n> > +    exe = executable(test['name'], test['sources'],\n> > +                     dependencies : [libcamera_private, libipa_dep],\n> > +                     link_with : [test_libraries],\n> > +                     include_directories : [test_includes_internal,\n> > +                                            '../../../src/ipa/libipa/'])\n> > +\n> > +    test(test['name'], exe, suite : 'ipa')\n> > +endforeach\n> > diff --git a/test/ipa/meson.build b/test/ipa/meson.build\n> > index e9871aba44ee..63820de54899 100644\n> > --- a/test/ipa/meson.build\n> > +++ b/test/ipa/meson.build\n> > @@ -1,5 +1,6 @@\n> >  # SPDX-License-Identifier: CC0-1.0\n> >  \n> > +subdir('libipa')\n> >  subdir('rkisp1')\n> >  \n> >  ipa_test = [\n> > -- \n> > 2.43.0\n> >","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 7DE99C3257\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 13 Sep 2024 09:06:49 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 1D0C0634FF;\n\tFri, 13 Sep 2024 11:06:49 +0200 (CEST)","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 6AE6E634F4\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 13 Sep 2024 11:06:47 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 88B6B1011;\n\tFri, 13 Sep 2024 11:05:28 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"D3Fgj9x6\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1726218328;\n\tbh=68Saa2xzblxzq/qhCaZvlKN2wbuSAA2YTr+xpB6yuCU=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=D3Fgj9x6wY8+ZLoP5HLgVhCaf9AJ7y1NPDcZW7iNF8/19U3Cqa8bf8On7QhJO5ho1\n\tv4o4wDRl/mjm/yq2RUoofSTf3ldoy6Rj5vVMuMTWQsCWB7Mfhd+VKZeKRg8DFUx+13\n\tBLcRvH824D138vwwyxYPFvKz53QS2GnF0Wf3MCpc=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<172621836031.3474483.11988287206831473721@ping.linuxembedded.co.uk>","References":"<20240913075750.35115-1-stefan.klug@ideasonboard.com>\n\t<20240913075750.35115-3-stefan.klug@ideasonboard.com>\n\t<172621836031.3474483.11988287206831473721@ping.linuxembedded.co.uk>","Subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","Cc":"Stefan Klug <stefan.klug@ideasonboard.com>","To":"Stefan Klug <stefan.klug@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org","Date":"Fri, 13 Sep 2024 10:06:44 +0100","Message-ID":"<172621840464.3474483.7296905648022308795@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":31211,"web_url":"https://patchwork.libcamera.org/comment/31211/","msgid":"<ZuQQ648hqYb5vYPQ@pyrite.rasen.tech>","date":"2024-09-13T10:16:11","subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/people/17/","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"content":"On Fri, Sep 13, 2024 at 09:57:20AM +0200, Stefan Klug wrote:\n> Add tests for the Interpolator class.\n> \n> Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com>\n> ---\n>  test/ipa/libipa/interpolator.cpp | 55 ++++++++++++++++++++++++++++++++\n>  test/ipa/libipa/meson.build      | 15 +++++++++\n>  test/ipa/meson.build             |  1 +\n>  3 files changed, 71 insertions(+)\n>  create mode 100644 test/ipa/libipa/interpolator.cpp\n>  create mode 100644 test/ipa/libipa/meson.build\n> \n> diff --git a/test/ipa/libipa/interpolator.cpp b/test/ipa/libipa/interpolator.cpp\n> new file mode 100644\n> index 000000000000..adb215386f0e\n> --- /dev/null\n> +++ b/test/ipa/libipa/interpolator.cpp\n> @@ -0,0 +1,55 @@\n> +/* SPDX-License-Identifier: GPL-2.0-or-later */\n> +/*\n> + * Copyright (C) 2024, Paul Elder <paul.elder@ideasonboard.com>\n\n:D\n\n> + *\n> + * Miscellaneous utility tests\n> + */\n> +\n> +#include <cmath>\n> +#include <iostream>\n> +#include <map>\n> +#include <stdint.h>\n> +#include <stdio.h>\n> +\n> +#include \"../src/ipa/libipa/interpolator.h\"\n> +\n> +#include \"test.h\"\n> +\n> +using namespace std;\n> +using namespace libcamera;\n> +using namespace ipa;\n> +\n> +#define ASSERT_EQ(a, b) \\\n> +\tif((a) != (b)) { printf(#a  \" != \" #b \"\\n\"); \\\n\nnit: newline?\n\n> +\t\treturn TestFail; \\\n> +\t}\n> +\n> +class InterpolatorTest : public Test\n> +{\n> +protected:\n> +\n> +\tint run()\n> +\t{\n> +\t\tInterpolator<int> interpolator;\n> +\t\tinterpolator.setData({{10,100},{20,200},{30,300}});\n\nnit: spacing\n\nOther than that, looks good to me\n\n\nReviewed-by: Paul Elder <paul.elder@ideasonboard.com>\n\n> +\n> +\t\tASSERT_EQ(interpolator.getInterpolated(0), 100);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(10), 100);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(20), 200);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(25), 250);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(30), 300);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(40), 300);\n> +\n> +\t\tinterpolator.setQuantization(10);\n> +\t\tunsigned int q = 0;\n> +\t\tASSERT_EQ(interpolator.getInterpolated(25, &q), 300);\n> +\t\tASSERT_EQ(q, 30);\n> +\t\tASSERT_EQ(interpolator.getInterpolated(24, &q), 200);\n> +\t\tASSERT_EQ(q, 20);\n> +\t\t\n> +\n> +\t\treturn TestPass;\n> +\t}\n> +};\n> +\n> +TEST_REGISTER(InterpolatorTest)\n> diff --git a/test/ipa/libipa/meson.build b/test/ipa/libipa/meson.build\n> new file mode 100644\n> index 000000000000..4d2427dbd4e7\n> --- /dev/null\n> +++ b/test/ipa/libipa/meson.build\n> @@ -0,0 +1,15 @@\n> +# SPDX-License-Identifier: CC0-1.0\n> +\n> +libipa_test = [\n> +    {'name': 'interpolator', 'sources': ['interpolator.cpp']},\n> +]\n> +\n> +foreach test : libipa_test\n> +    exe = executable(test['name'], test['sources'],\n> +                     dependencies : [libcamera_private, libipa_dep],\n> +                     link_with : [test_libraries],\n> +                     include_directories : [test_includes_internal,\n> +                                            '../../../src/ipa/libipa/'])\n> +\n> +    test(test['name'], exe, suite : 'ipa')\n> +endforeach\n> diff --git a/test/ipa/meson.build b/test/ipa/meson.build\n> index e9871aba44ee..63820de54899 100644\n> --- a/test/ipa/meson.build\n> +++ b/test/ipa/meson.build\n> @@ -1,5 +1,6 @@\n>  # SPDX-License-Identifier: CC0-1.0\n>  \n> +subdir('libipa')\n>  subdir('rkisp1')\n>  \n>  ipa_test = [\n> -- \n> 2.43.0\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 48DABC3257\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 13 Sep 2024 10:16:18 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 04EBF634F8;\n\tFri, 13 Sep 2024 12:16:18 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E94FD634F5\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 13 Sep 2024 12:16:15 +0200 (CEST)","from pyrite.rasen.tech (213-229-8-243.static.upcbusiness.at\n\t[213.229.8.243])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 19A4673;\n\tFri, 13 Sep 2024 12:14:57 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"CFzPLkc9\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1726222497;\n\tbh=M81jch7LRaJyGkTXELqaL+WhAzCBIfKie6FDWc/DzVU=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=CFzPLkc9XJRN1fK6wXHwA9tyFKG0uxcg3GGTuN1PRaqC2HbpN9IYvdAIHXYnDt2tv\n\txGr7LqDEXABcwPvXIy5/GahkCx6QcvOILkDgLMakgoWzb06WCMzbd7lYfLCZFW5L1e\n\tjbQsMVJ4KbUTnZE6Lio3/YyARTOetni8AABPE9ME=","Date":"Fri, 13 Sep 2024 12:16:11 +0200","From":"Paul Elder <paul.elder@ideasonboard.com>","To":"Stefan Klug <stefan.klug@ideasonboard.com>","Cc":"libcamera-devel@lists.libcamera.org","Subject":"Re: [PATCH v2 2/9] test: ipa: libipa: Add tets for Interpolator","Message-ID":"<ZuQQ648hqYb5vYPQ@pyrite.rasen.tech>","References":"<20240913075750.35115-1-stefan.klug@ideasonboard.com>\n\t<20240913075750.35115-3-stefan.klug@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20240913075750.35115-3-stefan.klug@ideasonboard.com>","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]