try updating confirmed values in hyperparams

AutomatingSciencePipeline · Nov 17, 2024 · 2f19319 · 2f19319
1 parent 912edfc
commit 2f19319
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/apps/frontend/app/components/Parameter.tsx b/apps/frontend/app/components/Parameter.tsx
@@ -15,9 +15,13 @@ const Parameter = ({ form, type, index, confirmedValues, setConfirmedValues, ...
 	};
 	const Component = remains[type];
 
-	const updateConfirmedValues = (index) => {
+	const updateConfirmedValues = (index, newValues) => {
 		//get values from form
 		const values = form.values.hyperparameters[index].values;
+
+		//set the new values
+
+		form.replaceListItem(`hyperparameters[${index}].values`, newValues);
 		//update confirmed values
 		let updatedValues = confirmedValues?.map((item) =>
 			item.index === index ? { ...item, values: values } : item

diff --git a/apps/frontend/app/components/flows/AddExperiment/stepComponents/ParamStep.tsx b/apps/frontend/app/components/flows/AddExperiment/stepComponents/ParamStep.tsx
@@ -28,7 +28,7 @@ export const ParamStep = ({ form, confirmedValues, setConfirmedValues, ...props
 										form.insertListItem('hyperparameters', {
 											name: '',
 											default: '',
-											...((type === 'stringlist') && { values: [] }),
+											...((type === 'stringlist') && { values: [''] }),
 											...((type === 'integer' || type === 'float') && {
 												min: '',
 												max: '',