Skip to content

Commit

Permalink
Fix: train model with changed hyperparameters
Browse files Browse the repository at this point in the history
  • Loading branch information
taewhi committed May 27, 2024
1 parent 2a3f9e6 commit 228b877
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions bi-tool/client/app/pages/tdb-workbenches/WorkbenchSource.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,9 @@ function WorkbenchSource(props) {
const cleanedValue = changedValue !== undefined ? changedValue : undefined;
values[0][key] = cleanedValue !== undefined ? cleanedValue : item.default_value;
})
console.log('data :', data)
console.log('values :', values)
console.log('changedData :', changedData)
// console.log('data :', data)
// console.log('values :', values)
// console.log('changedData :', changedData)
return values;
};

Expand Down Expand Up @@ -181,14 +181,14 @@ function WorkbenchSource(props) {
table_name: targetTable,
// modeltype_name: modelTypes[0].modeltype_name,
columns: JSON.stringify(targetColumns),
model_options: JSON.stringify({}),
model_options: JSON.stringify(changedHyperParameters),
};
updates.modeltype_name = modelTypes.length > 0 ? modelTypes[0].modeltype_name : null;

setWorkbench(extend(workbench.clone(), updates));
updateWorkbench(updates, { successMessage: null }); // show message only on error

}, [targetTable, targetColumns, modelTypes, defaultHyperParameters])
}, [targetTable, targetColumns, modelTypes, defaultHyperParameters, changedHyperParameters])

useEffect(() => {
// TODO: ignore new pages?
Expand Down

0 comments on commit 228b877

Please sign in to comment.