Cover Letter Detail
Show a cover letter.
GET /api/covers/21465/?format=api
{ "id": 21465, "url": "https://patchwork.libcamera.org/api/covers/21465/?format=api", "web_url": "https://patchwork.libcamera.org/cover/21465/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20241001191354.55056-1-jacopo.mondi@ideasonboard.com>", "date": "2024-10-01T19:13:44", "name": "[v10,0/5] Add Face Detection Controls", "submitter": { "id": 143, "url": "https://patchwork.libcamera.org/api/people/143/?format=api", "name": "Jacopo Mondi", "email": "jacopo.mondi@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/21465/mbox/", "series": [ { "id": 4647, "url": "https://patchwork.libcamera.org/api/series/4647/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4647", "date": "2024-10-01T19:13:44", "name": "Add Face Detection Controls", "version": 10, "mbox": "https://patchwork.libcamera.org/series/4647/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/21465/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 57D39BD80A\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 1 Oct 2024 19:14:05 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 15F77618D7;\n\tTue, 1 Oct 2024 21:14:04 +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 B0BF260553\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 1 Oct 2024 21:14:01 +0200 (CEST)", "from ideasonboard.com (unknown [5.77.89.72])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 1CE2AA1A;\n\tTue, 1 Oct 2024 21:12: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=\"wWOiWVn2\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1727809949;\n\tbh=QsajVl1tjVOvkKsmylTPyJgh1+mGwd3T3PMxPBVWuWM=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=wWOiWVn2qYqs1Z+lP6t6sOKIxOm6VJCb/RLtSFU2qDgaBKCJPo21o000bN/NaE+9C\n\tdU/gWnm7P+m/NjyTTO2VRQmZpca97NWo7r/2+N/bQJJvoMfMoPo9t/ivQXOauZZq9/\n\tdFDf4DRKFFaobTaisOY3sPtwnDJVpTvickG0be8c=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>,\n\tHarvey Yang <chenghaoyang@google.com>", "Subject": "[PATCH v10 0/5] Add Face Detection Controls", "Date": "Tue, 1 Oct 2024 21:13:44 +0200", "Message-ID": "<20241001191354.55056-1-jacopo.mondi@ideasonboard.com>", "X-Mailer": "git-send-email 2.46.1", "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": "I took over \"[PATCH v8 0/3] Add Face Detection Controls\" on top of the patch\nto the Rectangle class on which we have gone back and forth for the last week\n\nI'll merge this version\n\nv10:\n- Simplify patches 1 and 2 to avoid further discussions\n- Make subtraction between signed integers safer\n\nconstexpr Rectangle(const Point &point1, const Point &point2)\n\t: Rectangle(std::min(point1.x, point2.x), std::min(point1.y, point2.y),\n\t\t static_cast<unsigned int>(std::max(point1.x, point2.x)) -\n\t\t static_cast<unsigned int>(std::min(point1.x, point2.x)),\n\t\t static_cast<unsigned int>(std::max(point1.y, point2.y)) -\n\t\t static_cast<unsigned int>(std::min(point1.y, point2.y)))\n- Collect tags\n\nPipeline:\nhttps://gitlab.freedesktop.org/camera/libcamera/-/pipelines/1281864\n\nv9:\n- add \"libcamera: geometry: Clarify Rectangle's top-left corner\"\n- Rebase 2/5 on top of this using min(p1.y, p2.y)\n- Break out 3/5 from 4/5\n- Reword parts of 4/5 to make it more similar to the existing controls\n descriptions\n- Make FaceDetectMode a int32_t as required by\n e6da224926b0 (\"libcamera: controls: Handle enum values without a cast\")\n- Use int32_t as a type for FaceDetectMode in 5/5\n- Populate ANDROID_STATISTICS_FACE_SCORES with actual face scores\n- Populate the ANDROID_STATISTICS_FACE_DETECT_MODE metadata from settings\n only if present\n\nPipeline:\nhttps://gitlab.freedesktop.org/camera/libcamera/-/pipelines/1280968\n\n\nv8: Reverted back to v5 that new control ids in the draft.\n\nv7: Fixed comments.\n\nv6:\n- Added control_ids_android as the new control id list.\n - Moved the new control ids there, instead of in draft.\n\nv5:\n- Rewrote Rectangle's c'tor that allows any two diagonal points.\n - Added unit tests.\n\nv4:\n- Added support of FaceDetectModeFull and FaceDetectFaceIds.\n- Fixed descriptions of control sizes.\n\nv3: Applied fixes according to Jacopo's comments.\n- Moved the new face detection controls from core to draft.\n- Amended new controls' descriptions.\n\nv2:\n- Squashed the fourth CL into the last patch, as it's a fix.\n- Fixed CLs based on checkstyle's suggestions.\n\nHarvey Yang (1):\n libcamera: android: Add face detection control support\n\nJacopo Mondi (1):\n libcamera: geometry: Clarify Rectangle's top-left corner\n\nYudhistira Erlandinata (3):\n libcamera: geometry: Add two-point Rectangle constructor\n libcamera: controls: Add ControlTypePoint\n libcamera: control_ids_draft: Add face detection controls\n\n include/libcamera/controls.h | 6 ++\n include/libcamera/geometry.h | 9 +++\n src/android/camera_capabilities.cpp | 44 +++++++++++++--\n src/android/camera_device.cpp | 66 +++++++++++++++++++++-\n src/libcamera/control_ids_draft.yaml | 82 ++++++++++++++++++++++++++++\n src/libcamera/controls.cpp | 6 ++\n src/libcamera/geometry.cpp | 20 +++++++\n test/geometry.cpp | 14 +++++\n 8 files changed, 241 insertions(+), 6 deletions(-)\n\n--\n2.46.1" }