-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix bug of hyper-parameter broken when have not succeeded trial #1460
Conversation
merge master
Update filter description and fix typo
Quick fix nnictl schema TPE issue
…icrosoft#1412) * Eject from react-scripts-ts-antd * test whether it can pass CI without ugilfy * temporarily disable uglify
const temp = value.split(''); | ||
for (let m = 16; m < temp.length; m += 17) { | ||
temp[m] += '\n'; | ||
if (value !== undefined) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The type annotation says it can't be undefined
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, will add it.
for (let m = 16; m < temp.length; m += 17) { | ||
temp[m] += '\n'; | ||
} | ||
return temp.join(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's better to split the string into 17-char chunks with substring
/slice
and then join them with \n
.
Just a personal opinion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I will try it in next version.
No description provided.