Patch Detail
Show a patch.
GET /api/1.1/patches/20460/?format=api
{ "id": 20460, "url": "https://patchwork.libcamera.org/api/1.1/patches/20460/?format=api", "web_url": "https://patchwork.libcamera.org/patch/20460/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/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": "<20240628104828.2928109-2-stefan.klug@ideasonboard.com>", "date": "2024-06-28T10:46:54", "name": "[v2,01/25] libtuning: Backport improvements in MacBeth search reliability", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "1b2fa2ff0918c4cae5a0848664bbe5235e2eb810", "submitter": { "id": 184, "url": "https://patchwork.libcamera.org/api/1.1/people/184/?format=api", "name": "Stefan Klug", "email": "stefan.klug@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/20460/mbox/", "series": [ { "id": 4430, "url": "https://patchwork.libcamera.org/api/1.1/series/4430/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=4430", "date": "2024-06-28T10:46:53", "name": "Add ccm calibration to libtuning", "version": 2, "mbox": "https://patchwork.libcamera.org/series/4430/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/20460/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/20460/checks/", "tags": {}, "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 06767BD87C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 28 Jun 2024 10:48:44 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id B3FFE62CA0;\n\tFri, 28 Jun 2024 12:48:43 +0200 (CEST)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 78E56619E5\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 28 Jun 2024 12:48:42 +0200 (CEST)", "from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:82ab:924:d918:cd24])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id EBA76BEB;\n\tFri, 28 Jun 2024 12:48:17 +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=\"opxY5xMq\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1719571698;\n\tbh=LQcERWemL4LbddVAr9NVb+1aikejkI2QMqiKNYg7B9w=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=opxY5xMqdY5w6MWF9wGZW3XE1StFdjL2Tp/fHDezqGij3rMx38xbjYp2BmUlnzizv\n\tZ90kzzJhMMXKEcr2lQoTJCEfj6SbBYF2xla939+wbAhCKIid/cixANjRUES9CSVHBa\n\tiz4OnHM2Y1HHDun8o3mqsSni315VDZnWh2fa/bf4=", "From": "Stefan Klug <stefan.klug@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Cc": "Stefan Klug <stefan.klug@ideasonboard.com>,\n\tPaul Elder <paul.elder@ideasonboard.com>", "Subject": "[PATCH v2 01/25] libtuning: Backport improvements in MacBeth search\n\treliability", "Date": "Fri, 28 Jun 2024 12:46:54 +0200", "Message-ID": "<20240628104828.2928109-2-stefan.klug@ideasonboard.com>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20240628104828.2928109-1-stefan.klug@ideasonboard.com>", "References": "<20240628104828.2928109-1-stefan.klug@ideasonboard.com>", "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": "Port 66479605baca4a22e2b7a17c2a8cf9f9be9a7724 into libtuning.\nOriginal message:\nImprove the Macbeth Chart search reliability\n\nPreviously the code would brighten up images in case the Macbeth Chart\nis slightly dark, and also zoom in on sections of it to look for\ncharts occupying less of the field of view. But it would not do both\ntogether.\n\nThis change makes the search for smaller charts also repeat that\nsearch for the brightened up images that it made earlier, thereby\nincreasing the chances of success for non-optimal tuning images.\n\nSigned-off-by: Stefan Klug <stefan.klug@ideasonboard.com>\nReviewed-by: Paul Elder <paul.elder@ideasonboard.com>\n---\n utils/tuning/libtuning/macbeth.py | 38 ++++++++++++++++++-------------\n 1 file changed, 22 insertions(+), 16 deletions(-)", "diff": "diff --git a/utils/tuning/libtuning/macbeth.py b/utils/tuning/libtuning/macbeth.py\nindex e11824646a4f..81f3e87c9088 100644\n--- a/utils/tuning/libtuning/macbeth.py\n+++ b/utils/tuning/libtuning/macbeth.py\n@@ -403,10 +403,15 @@ def find_macbeth(img, mac_config):\n # nothing more is tried as this is a high enough confidence to ensure\n # reliable macbeth square centre placement.\n \n+ # Keep a list that will include this and any brightened up versions of\n+ # the image for reuse.\n+ all_images = [img]\n+\n for brightness in [2, 4]:\n if cor >= 0.75:\n break\n img_br = cv2.convertScaleAbs(img, alpha=brightness, beta=0)\n+ all_images.append(img_br)\n cor_b, mac_b, coords_b, ret_b = get_macbeth_chart(img_br, ref_data)\n if cor_b > cor:\n cor, mac, coords, ret = cor_b, mac_b, coords_b, ret_b\n@@ -456,23 +461,24 @@ def find_macbeth(img, mac_config):\n w_inc = int(w * pair['inc'])\n h_inc = int(h * pair['inc'])\n \n- loop = ((1 - pair['sel']) / pair['inc']) + 1\n+ loop = int(((1 - pair['sel']) / pair['inc']) + 1)\n # For each subselection, look for a macbeth chart\n- for i in range(loop):\n- for j in range(loop):\n- w_s, h_s = i * w_inc, j * h_inc\n- img_sel = img[w_s:w_s + w_sel, h_s:h_s + h_sel]\n- cor_ij, mac_ij, coords_ij, ret_ij = get_macbeth_chart(img_sel, ref_data)\n-\n- # If the correlation is better than the best then record the\n- # scale and current subselection at which macbeth chart was\n- # found. Also record the coordinates, macbeth chart and message.\n- if cor_ij > cor:\n- cor = cor_ij\n- mac, coords, ret = mac_ij, coords_ij, ret_ij\n- ii, jj = i, j\n- w_best, h_best = w_inc, h_inc\n- d_best = index + 1\n+ for img_br in all_images:\n+ for i in range(loop):\n+ for j in range(loop):\n+ w_s, h_s = i * w_inc, j * h_inc\n+ img_sel = img_br[w_s:w_s + w_sel, h_s:h_s + h_sel]\n+ cor_ij, mac_ij, coords_ij, ret_ij = get_macbeth_chart(img_sel, ref_data)\n+\n+ # If the correlation is better than the best then record the\n+ # scale and current subselection at which macbeth chart was\n+ # found. Also record the coordinates, macbeth chart and message.\n+ if cor_ij > cor:\n+ cor = cor_ij\n+ mac, coords, ret = mac_ij, coords_ij, ret_ij\n+ ii, jj = i, j\n+ w_best, h_best = w_inc, h_inc\n+ d_best = index + 1\n \n # Transform coordinates from subselection to original image\n if ii != -1:\n", "prefixes": [ "v2", "01/25" ] }