{"id":17021,"url":"https://patchwork.libcamera.org/api/patches/17021/?format=json","web_url":"https://patchwork.libcamera.org/patch/17021/","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":"<20220807203204.152358-4-utkarsh02t@gmail.com>","date":"2022-08-07T20:32:03","name":"[libcamera-devel,v6,3/4] qcam: CamSelectDialog: Display Capture script path","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"c107c11b53085514a4d695d5ff8b52476e9c9010","submitter":{"id":114,"url":"https://patchwork.libcamera.org/api/people/114/?format=json","name":"Utkarsh Tiwari","email":"utkarsh02t@gmail.com"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/17021/mbox/","series":[{"id":3385,"url":"https://patchwork.libcamera.org/api/series/3385/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3385","date":"2022-08-07T20:32:00","name":"Introduce capture scripts to qcam","version":6,"mbox":"https://patchwork.libcamera.org/series/3385/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/17021/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/17021/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 1812BBE173\n\tfor <parsemail@patchwork.libcamera.org>;\n\tSun,  7 Aug 2022 20:32:28 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id CBE076332D;\n\tSun,  7 Aug 2022 22:32:27 +0200 (CEST)","from mail-pg1-x52e.google.com (mail-pg1-x52e.google.com\n\t[IPv6:2607:f8b0:4864:20::52e])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id B424C63327\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSun,  7 Aug 2022 22:32:26 +0200 (CEST)","by mail-pg1-x52e.google.com with SMTP id q16so6933479pgq.6\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSun, 07 Aug 2022 13:32:26 -0700 (PDT)","from localhost.localdomain ([2404:bd00:3:d2bc:8eec:115d:8ea8:b700])\n\tby smtp.gmail.com with ESMTPSA id\n\tx29-20020aa7941d000000b0052d194a6602sm7006334pfo.102.2022.08.07.13.32.23\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tSun, 07 Aug 2022 13:32:24 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1659904347;\n\tbh=mDM/AqDCeM00UHYfpX5f1RIyHRRuplHtHJbTxOHBOE0=;\n\th=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=zx46pBe9D3wWUmitRSbd0Lcy+lvliMiZhKK2g5Wz2706ba7zdhLWpq87f/TEgI1mt\n\tc6htj8b3A+tPvyA0SvKtw1Ta/ZJHJnCzNl5WnVHNj2Wx9f+xnCR5Y33pgKYfh8p3DD\n\tNdlHwQ7MfWjQK8wpimqRIKj2r/rZutUxOkTwT/UGbNAHMhAKEV7i32YLc0v2K7F19t\n\tNutOU0AIqhwZZstIgy5RKiWqpoyzVnfQai3JVwLP72XAzH17dh3RN1RHPmhVlUluXg\n\tn/PX0NmsJzFRe8e9tdxaSNj9eCSGJnlvtaqsLO72LYYQIGaPA8+1WauoQWzjrx0OMu\n\t6vYgJfxKDtXGw==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112;\n\th=content-transfer-encoding:mime-version:references:in-reply-to\n\t:message-id:date:subject:cc:to:from:from:to:cc;\n\tbh=kanfHGJITr6TvtYEOvkfhBTL2ZgRI2xaXV9xshtHLHQ=;\n\tb=aFVbkFsR/DYbdVHhyDWiW5hHP+g7mUlZxQekw7t41QsUjQkxsfHOPQQg+t6hxW58Lk\n\tgByMVMwE47rVWK1f/whu4cJzcVLGIzPdpCLcLvzGlkEPRLfMIkLylo7jZ7wTNFF0GYjK\n\tULqvDxvckHtbraOQ6daEN+f1JXF/RHTblTdl/3iFnxzpuY/kRNm6e0NAuz31HNW/VNOM\n\t1jb9aMGf8NrJ0/C1yguyPccY9yTpSsQGfRa9OPZK1WQJHReZrww/HKNjE+WhCnd9rumY\n\tHQHKvX5hGcWTdVWofWD2LSZAoEZu6H+5ooZJ/zZ1zhXsvGDxEwtwy1I6rh7plVEEC000\n\teVcA=="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"aFVbkFsR\"; dkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=content-transfer-encoding:mime-version:references:in-reply-to\n\t:message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc;\n\tbh=kanfHGJITr6TvtYEOvkfhBTL2ZgRI2xaXV9xshtHLHQ=;\n\tb=wZCaODH2PXU0qWMqMc5t7w785N4oaKIsFxM7yQiTq4g/RmeDh02LjD/XgO7rJVFkPR\n\ts/vUcyBp6qQE2J5x0Ws5m98hFWKCCLsb/bK8ElPq89NoQ4XUFs8T3kLXhYp/OJAW42C8\n\tEdj29LN+8Hvg5JTn6AnhNvGLF1ghX4R4IQiOzJy5PxEZoscHy22O03b02G+Aq7llAAtA\n\t8fvfv82qPEPHLjWX8HW8AMlRu3nC7pEF8h94RnkTTdllv0C0uBwWZMSbQjOzVtL2nrUL\n\t1zKra/xxW5lsLMuSgDAIswBGH5OpGoKKQWMI6b4vzLzgfVZ0Ymkwy1n9vFctsQagjCYj\n\tlVSg==","X-Gm-Message-State":"ACgBeo0aVa6d7PIv9hFbj8TYr5ilO9RH51Nq88C6u0lzkWMcVhHWwhTl\n\ttnK81ERdHw3+AbgtsJjPFx1hjd3xKII=","X-Google-Smtp-Source":"AA6agR6QOQwiYt+oqsL4/JMcMLTIDiDkeT23pakktQs0fs2aSXzZ4JhTHMNuaRVqwi7pyBKEObzuOg==","X-Received":"by 2002:aa7:9157:0:b0:52f:20d6:e4f6 with SMTP id\n\t23-20020aa79157000000b0052f20d6e4f6mr4616705pfi.40.1659904344877; \n\tSun, 07 Aug 2022 13:32:24 -0700 (PDT)","To":"libcamera-devel@lists.libcamera.org","Date":"Mon,  8 Aug 2022 02:02:03 +0530","Message-Id":"<20220807203204.152358-4-utkarsh02t@gmail.com>","X-Mailer":"git-send-email 2.25.1","In-Reply-To":"<20220807203204.152358-1-utkarsh02t@gmail.com>","References":"<20220807203204.152358-1-utkarsh02t@gmail.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v6 3/4] qcam: CamSelectDialog: Display\n\tCapture script path","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>","From":"Utkarsh Tiwari via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>","Reply-To":"Utkarsh Tiwari <utkarsh02t@gmail.com>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"},"content":"Display the path of the selected capture script in a thinner font.\n\nSigned-off-by: Utkarsh Tiwari <utkarsh02t@gmail.com>\n---\nDifference from V5:\n\tcame to existance.\n src/qcam/cam_select_dialog.h | 41 +++++++++++++++++++++++++++++++-----\n src/qcam/main_window.cpp     |  3 ++-\n 2 files changed, 38 insertions(+), 6 deletions(-)","diff":"diff --git a/src/qcam/cam_select_dialog.h b/src/qcam/cam_select_dialog.h\nindex fc29b46a..7dc44794 100644\n--- a/src/qcam/cam_select_dialog.h\n+++ b/src/qcam/cam_select_dialog.h\n@@ -19,18 +19,22 @@\n #include <QDialog>\n #include <QDialogButtonBox>\n #include <QFileDialog>\n+#include <QFont>\n #include <QFormLayout>\n #include <QLabel>\n #include <QPushButton>\n #include <QString>\n+#include <QVBoxLayout>\n+#include <QWidget>\n \n class CamSelectDialog : public QDialog\n {\n \tQ_OBJECT\n public:\n \tCamSelectDialog(libcamera::CameraManager *cameraManager,\n-\t\t\tbool isScriptRunning, QWidget *parent)\n-\t\t: QDialog(parent), cm_(cameraManager), isScriptRunning_(isScriptRunning)\n+\t\t\tbool isScriptRunning, std::string scriptPath, QWidget *parent)\n+\t\t: QDialog(parent), cm_(cameraManager),\n+\t\t  isScriptRunning_(isScriptRunning), scriptPath_(scriptPath)\n \t{\n \t\t/* Use a QFormLayout for the dialog. */\n \t\tQFormLayout *camSelectDialogLayout = new QFormLayout(this);\n@@ -51,14 +55,31 @@ public:\n \t\t\t\t\tcameraIdComboBox_->currentText().toStdString()));\n \t\t\t});\n \n+\t\t/* Setup widget for capture script button. */\n+\t\tQWidget *captureWidget = new QWidget;\n+\t\tcaptureWidgetLayout_ = new QVBoxLayout(captureWidget);\n+\t\tcaptureWidgetLayout_->setMargin(0);\n+\n \t\tcaptureScriptButton_ = new QPushButton;\n \t\tconnect(captureScriptButton_, &QPushButton::clicked,\n \t\t\tthis, &CamSelectDialog::handleCaptureScriptButton);\n+\t\tcaptureWidgetLayout_->addWidget(captureScriptButton_);\n+\n+\t\t/* Use a thinner font to indicate script info. */\n+\t\tQFont smallFont;\n+\t\tsmallFont.setWeight(QFont::Thin);\n+\n+\t\tscriptPathLabel_ = new QLabel;\n+\t\tscriptPathLabel_->setFont(smallFont);\n+\t\tscriptPathLabel_->setWordWrap(true);\n \n \t\t/* Display the action that would be performed when button is clicked. */\n-\t\tif (isScriptRunning_)\n+\t\tif (isScriptRunning_) {\n \t\t\tcaptureScriptButton_->setText(\"Stop\");\n-\t\telse\n+\n+\t\t\tscriptPathLabel_->setText(QString::fromStdString(scriptPath_));\n+\t\t\tcaptureWidgetLayout_->addWidget(scriptPathLabel_);\n+\t\t} else\n \t\t\tcaptureScriptButton_->setText(\"Open\");\n \n \t\t/* Setup the QDialogButton Box */\n@@ -75,7 +96,7 @@ public:\n \t\tcamSelectDialogLayout->addRow(\"Camera: \", cameraIdComboBox_);\n \t\tcamSelectDialogLayout->addRow(\"Location: \", cameraLocation_);\n \t\tcamSelectDialogLayout->addRow(\"Model: \", cameraModel_);\n-\t\tcamSelectDialogLayout->addRow(\"Capture Script: \", captureScriptButton_);\n+\t\tcamSelectDialogLayout->addRow(\"Capture Script: \", captureWidget);\n \t\tcamSelectDialogLayout->addWidget(dialogButtonBox);\n \t}\n \n@@ -148,6 +169,8 @@ public:\n \t\t\tQ_EMIT stopCaptureScript();\n \t\t\tisScriptRunning_ = false;\n \t\t\tcaptureScriptButton_->setText(\"Open\");\n+\n+\t\t\tcaptureWidgetLayout_->removeWidget(scriptPathLabel_);\n \t\t} else {\n \t\t\tscriptPath_ = QFileDialog::getOpenFileName(this, \"Run Capture Script\",\n \t\t\t\t\t\t\t\t   QDir::currentPath(), \"Capture Script (*.yaml)\")\n@@ -155,6 +178,11 @@ public:\n \n \t\t\tif (!scriptPath_.empty())\n \t\t\t\tcaptureScriptButton_->setText(\"Loaded\");\n+\t\t\telse\n+\t\t\t\treturn;\n+\n+\t\t\tscriptPathLabel_->setText(QString::fromStdString(scriptPath_));\n+\t\t\tcaptureWidgetLayout_->addWidget(scriptPathLabel_);\n \t\t}\n \t}\n \n@@ -171,5 +199,8 @@ private:\n \tQComboBox *cameraIdComboBox_;\n \tQLabel *cameraLocation_;\n \tQLabel *cameraModel_;\n+\n+\tQVBoxLayout *captureWidgetLayout_;\n \tQPushButton *captureScriptButton_;\n+\tQLabel *scriptPathLabel_ = new QLabel;\n };\ndiff --git a/src/qcam/main_window.cpp b/src/qcam/main_window.cpp\nindex e0219e5e..e354ba10 100644\n--- a/src/qcam/main_window.cpp\n+++ b/src/qcam/main_window.cpp\n@@ -332,7 +332,8 @@ void MainWindow::loadCaptureScript()\n std::string MainWindow::chooseCamera()\n {\n \tbool scriptRunning = script_ != nullptr;\n-\tcamSelectDialog_ = new CamSelectDialog(cm_, scriptRunning, this);\n+\tcamSelectDialog_ = new CamSelectDialog(cm_, scriptRunning,\n+\t\t\t\t\t       scriptPath_, this);\n \n \tconnect(camSelectDialog_, &CamSelectDialog::stopCaptureScript,\n \t\tthis, &MainWindow::stopCaptureScript);\n","prefixes":["libcamera-devel","v6","3/4"]}