{"id":23001,"url":"https://patchwork.libcamera.org/api/covers/23001/?format=json","web_url":"https://patchwork.libcamera.org/cover/23001/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20250324170803.103296-1-stefan.klug@ideasonboard.com>","date":"2025-03-24T17:07:35","name":"[0/5] Fix histogram for some (corner) cases","submitter":{"id":184,"url":"https://patchwork.libcamera.org/api/people/184/?format=json","name":"Stefan Klug","email":"stefan.klug@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/23001/mbox/","series":[{"id":5077,"url":"https://patchwork.libcamera.org/api/series/5077/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=5077","date":"2025-03-24T17:07:35","name":"Fix histogram for some (corner) cases","version":1,"mbox":"https://patchwork.libcamera.org/series/5077/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/23001/comments/","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 506F3C3213\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 24 Mar 2025 17:08:16 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 45F3F68951;\n\tMon, 24 Mar 2025 18:08:15 +0100 (CET)","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 EC4856893F\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 24 Mar 2025 18:08:13 +0100 (CET)","from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:854:5bc6:b2a7:e242])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 344C5455;\n\tMon, 24 Mar 2025 18:06:27 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"gIZDFq7a\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1742835987;\n\tbh=ekyF1p9TGbcdXbbdAvgVW2OAPFBvhiEo/w3YhNdsQEw=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=gIZDFq7ajAQJFNPKbO8lH/CnFyNVD4HkZhiDYEDr14dVVdCPYqWKfXw0/lA5ceBJY\n\tYQSjvFU68n8fQMbvzjH2ACnL1Fi3zJggVE2xKtB2ARYqCbUGP8u0fM+M8IBMT2Flkw\n\t7mJuN1iO27KbMyQPYVDDby4WG0JQR8mJ4OZ548jg=","From":"Stefan Klug <stefan.klug@ideasonboard.com>","To":"libcamera-devel@lists.libcamera.org","Cc":"Stefan Klug <stefan.klug@ideasonboard.com>","Subject":"[PATCH 0/5] Fix histogram for some (corner) cases","Date":"Mon, 24 Mar 2025 18:07:35 +0100","Message-ID":"<20250324170803.103296-1-stefan.klug@ideasonboard.com>","X-Mailer":"git-send-email 2.43.0","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","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>"},"content":"Hi all,\n\nduring my work on WDR I stumbled over a few non-linearities when dealing\nwith small inter quantile mean ranges. These led to hard to handle\noscillations. This series adds a testcase for the histogram class and\nfixes the related issues.\n\nPatch 1 adds the basic tests that pass with the current implementation.\n\nPatch 2-3 fix issues in the quantile() implementation and add\ncorresponding tests.\n\nPatch 4-5 fix issues in the interQuantileMean() implementation and also\nadd tests.\n\nI added the tests after the fixes, so that I don't have to toggle should_fail\nin the meson.build file all the time (And CI doesn't test every commit anyways).\nTo test that in review, I believe the easiest is to temporarily revert the\ncorresponding fix and see the test fail.\n\nBest regards,\nStefan\n\nStefan Klug (5):\n  test: ipa: libipa: Add histogram tests\n  libipa: histogram: Fix quantile() calculation for fractional results\n  test: ipa: libipa: histogram: Add tests for quantile() returning a\n    fraction\n  libipa: histogram: Fix interQuantileMean() for small ranges\n  test: ipa: libipa: histogram: Add tests for small inter quantile mean\n    ranges\n\n src/ipa/libipa/histogram.cpp  | 23 ++++++------\n test/ipa/libipa/histogram.cpp | 66 +++++++++++++++++++++++++++++++++++\n test/ipa/libipa/meson.build   |  1 +\n 3 files changed, 80 insertions(+), 10 deletions(-)\n create mode 100644 test/ipa/libipa/histogram.cpp"}