{"id":26904,"url":"https://patchwork.libcamera.org/api/covers/26904/?format=json","web_url":"https://patchwork.libcamera.org/cover/26904/","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":"<20260616120828.375547-1-stefan.klug@ideasonboard.com>","date":"2026-06-16T12:08:13","name":"[v1,0/2] Clarify rectangle behavior","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/26904/mbox/","series":[{"id":5998,"url":"https://patchwork.libcamera.org/api/series/5998/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=5998","date":"2026-06-16T12:08:13","name":"Clarify rectangle behavior","version":1,"mbox":"https://patchwork.libcamera.org/series/5998/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/26904/comments/","headers":{"Return-Path":"","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 4BECBC3261\n\tfor ;\n\tTue, 16 Jun 2026 12:08:48 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 89B0E625CE;\n\tTue, 16 Jun 2026 14:08:47 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E1E9B625B6\n\tfor ;\n\tTue, 16 Jun 2026 14:08:45 +0200 (CEST)","from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:f9ec:e24d:190b:e780])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 5FF77448;\n\tTue, 16 Jun 2026 14:08:12 +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=\"ic+pfUzK\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1781611692;\n\tbh=CWWxbIfUPYof2pfF4noz5U0omZ3K5g3EhA5UrWQT1B0=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=ic+pfUzKV85eU3EpDfreZagL6tEHMCtYgm8av17Cugv/jAmS04gHQH6DkG+rDLatv\n\t4ql2F9qU1f3z1us8ASewsls5k0v+ZlEY095nvbFi+hI/3Y/WhnBG7SsLNB9wbCZ6va\n\t1xXSfn1JoXF7K+SW0I2UjHoHINwcIWzKTzY15Ysk=","From":"Stefan Klug ","To":"libcamera-devel@lists.libcamera.org","Cc":"Stefan Klug ","Subject":"[PATCH v1 0/2] Clarify rectangle behavior","Date":"Tue, 16 Jun 2026 14:08:13 +0200","Message-ID":"<20260616120828.375547-1-stefan.klug@ideasonboard.com>","X-Mailer":"git-send-email 2.53.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":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"Hi all,\n\nThis series changes the behavior of PixelArraySize to represent the\nphysical sensor area instead of the readable pixel area. The reasoning\nis explained in a bit more detail in patch 1. Patch 2 is a small cleanup\nthat also shows how limited the impact on the overall project is.\n\nWith these patches applied I am able to draw proper bounding rectangles\nin camshark. An implemented is available in\nhttps://gitlab.freedesktop.org/camera/camshark/-/tree/sklug/next?ref_type=heads\n\nBest regards,\nStefan\n\nStefan Klug (2):\n libcamera: Clarify meaning of PixelArraySize and other rectangles\n libcamera: ipa: core: Rename IPACameraSensorInfo::activeAreaSize to\n pixelArraySize\n\n include/libcamera/ipa/core.mojom | 6 +-\n src/ipa/rkisp1/algorithms/lsc.cpp | 6 +-\n src/ipa/rpi/common/ipa_base.cpp | 4 +-\n src/libcamera/property_ids_core.yaml | 116 ++++++++++--------\n src/libcamera/sensor/camera_sensor_legacy.cpp | 28 +++--\n src/libcamera/sensor/camera_sensor_raw.cpp | 18 ++-\n 6 files changed, 96 insertions(+), 82 deletions(-)"}