From d3f4b792780a7662330fb3c6869dc2812e1c249e Mon Sep 17 00:00:00 2001 From: Samir Kamal <1954121+skamril@users.noreply.github.com> Date: Wed, 29 Nov 2023 17:18:27 +0100 Subject: [PATCH] fix(ui-lancher): cores limit cannot be break --- webapp/src/components/App/Studies/LauncherDialog.tsx | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/webapp/src/components/App/Studies/LauncherDialog.tsx b/webapp/src/components/App/Studies/LauncherDialog.tsx index ceecde25c4..1419bfe6ac 100644 --- a/webapp/src/components/App/Studies/LauncherDialog.tsx +++ b/webapp/src/components/App/Studies/LauncherDialog.tsx @@ -289,9 +289,13 @@ function LauncherDialog(props: Props) { type="number" variant="filled" value={options.nb_cpu} - onChange={(e) => - handleChange("nb_cpu", parseInt(e.target.value, 10)) - } + onChange={(e) => { + const newValue = parseInt(e.target.value, 10); + handleChange( + "nb_cpu", + Math.min(Math.max(newValue, cores.min), cores.max), + ); + }} inputProps={{ min: cores.min, max: cores.max,