This repository has been archived by the owner on Sep 18, 2024. It is now read-only.
Remove Tuner's unnecessary ClassArg requirements #903
Assignees
Labels
Comments
|
Good suggestion! |
|
#916 will fix this issue. Since we don't need to know the optimize_mode in random, remove "optimize_mode" in Random tuner. |
xuehui1991
added a commit
that referenced
this issue
Mar 27, 2019
* update readme in ga_squad * update readme * fix typo * Update README.md * Update README.md * Update README.md * update readme * update * fix path * update reference * fix bug in config file * update nni_arch_overview.png * update * update * update * update home page * fix issue: #902, random parameter * update doc about random * change Random tuner test * update metric_test for Random tuner * update random config in test * update Evolution_SQuAD doc
|
Already merge into the master branch. Close this issue. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Currently Tuner has some ClassArg that is not necessary.
For example:
tuner: builtinTunerName: Random classArgs: optimize_mode: maximizeIn fact, random doesn‘t need optimize_mode.
This option is required, that is to say, if you don't add this item, you will get an error.
This is neither reasonable nor affects the user experience. It is recommended to change to optional.
The text was updated successfully, but these errors were encountered: