Skip to content

GitLab

Commit b56e4a17 authored by Lijiao's avatar Lijiao Committed by xuehui
Browse files

fix bug of hyper-parameter graph when search space has loguniform type (#1399)

parent 5b62d048
Show whitespace changes
Inline Side-by-side
Showing with 15 additions and 11 deletions
src/webui/src/components/trial-detail/Para.tsx
View file @ b56e4a17
...@@ -87,13 +87,10 @@ class Para extends React.Component<ParaProps, ParaState> { ...@@ -87,13 +87,10 @@ class Para extends React.Component<ParaProps, ParaState> {
let temp: Array<number> = []; let temp: Array<number> = [];
for (let i = 0; i < dimName.length; i++) { for (let i = 0; i < dimName.length; i++) {
if ('type' in parallelAxis[i]) { if ('type' in parallelAxis[i]) {
temp.push( temp.push(eachTrialParams[item][dimName[i]].toString());
eachTrialParams[item][dimName[i]].toString()
);
} else { } else {
temp.push( // default metric
eachTrialParams[item][dimName[i]] temp.push(eachTrialParams[item][dimName[i]]);
);
} }
} }
paraYdata.push(temp); paraYdata.push(temp);
...@@ -199,11 +196,18 @@ class Para extends React.Component<ParaProps, ParaState> { ...@@ -199,11 +196,18 @@ class Para extends React.Component<ParaProps, ParaState> {
break; break;
// support log distribute // support log distribute
case 'loguniform': case 'loguniform':
if (lenOfDataSource > 1) {
parallelAxis.push({ parallelAxis.push({
dim: i, dim: i,
name: dimName[i], name: dimName[i],
type: 'log', type: 'log',
}); });
} else {
parallelAxis.push({
dim: i,
name: dimName[i]
});
}
break; break;
default: default:
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment