Skip to content
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

Structure for NAS algorithms #1125

Closed
andreyvelich opened this issue Apr 7, 2020 · 7 comments · Fixed by #1143
Closed

Structure for NAS algorithms #1125

andreyvelich opened this issue Apr 7, 2020 · 7 comments · Fixed by #1143

Comments

@andreyvelich
Copy link
Member

/kind feature

Since I am planning to support DARTS in Katib, I have few thoughts and questions about current NAS algorithms.

  1. Do we want to name each NAS algorithm starting with nas. For example: nasdarts? Or we can name it with algorithm name, e.g darts.
  2. Rename nasrl to enas. I think it will be more clear to user, since we are using ENAS implementation.
  3. Do we want to create nas directory under suggestion folder. User can easily see what HP and NAS algorithms do we support.

What do you think @gaocegege @johnugeorge ?

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the labels:

Label Probability
question 0.83

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@gaocegege
Copy link
Member

SGTM. I will add it into 2020 roadmap.

/priority p1

@andreyvelich
Copy link
Member Author

/area nas

@andreyvelich
Copy link
Member Author

What do you think about 1st and 3rd step? @gaocegege @johnugeorge
Should we name nas algorithms starting with nas or just like name of the algorithm?

@johnugeorge
Copy link
Member

3 looks ok as it is just directory refactoring.

For 1, is it really required? If it needs to be added, I feel that we have to tag existing hp algorithms too.

@andreyvelich
Copy link
Member Author

andreyvelich commented Apr 10, 2020

For 1, is it really required? If it needs to be added, I feel that we have to tag existing hp algorithms too.

No, it is not required. I am just worrying how users will distinguish HP and NAS algorithms.

@andreyvelich
Copy link
Member Author

/assign @andreyvelich

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants