Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Tuners should have precondition check #1667

Closed
ultmaster opened this issue Oct 29, 2019 · 3 comments
Closed

Tuners should have precondition check #1667

ultmaster opened this issue Oct 29, 2019 · 3 comments
Assignees
Labels
bug Something isn't working fixed downstream nnidev
Milestone

Comments

@ultmaster
Copy link
Contributor

Tuners should throw error in case of invalid search space, such as: {"_type": "randint", "_value": [1, 2, 3]}, or {"_type": "randint", "_value": [0.5, 1.5]}.

@chicm-ms
Copy link
Contributor

@ultmaster What is current Tuners behaviour in case of invalid search space?

@ultmaster
Copy link
Contributor Author

Sometimes they get processed as is. Sometimes tuners fail. Tuners' behaviors are undefined.

@scarlett2018 scarlett2018 added the enhancement New feature or request label Nov 18, 2019
@scarlett2018 scarlett2018 added bug Something isn't working and removed enhancement New feature or request labels Dec 9, 2019
@scarlett2018
Copy link
Member

Assign to @QuanluZhang for further triage.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working fixed downstream nnidev
Projects
None yet
Development

No branches or pull requests

4 participants