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

Refactor builtin/customized tuner installation #3134

Merged
merged 129 commits into from
Dec 11, 2020

Conversation

chicm-ms
Copy link
Contributor

@chicm-ms chicm-ms commented Nov 28, 2020

register all builtin tuners/assessors/advisors into deployment/registered_algorithms.yml and copy it into nni config dir in setup.py
nni config dir is the following suggested path:

Environment "data_files" Path Suggested Path
Linux/global /usr/nni ~/.config/nni
Linux/user ~/.local/nni ~/.config/nni
Linux/conda (env_root)/nni (env_root)/nni
Windows/global Program Files/Python38/nni %appdata%/nni
Windows/user %appdata%/python/nni %appdata%/nni
Windows/conda (env_root)/nni (env_root)/nni

nnictl package command is replaced with nnictl algo

TODO:

  • extras_require for SMAC
  • SMAC/BOHB/PPOTuner dependencies install.
  • SMAC/BOHB/PPOTuner import check.
  • SMAC/BOHB/PPOTuner doc
  • nni config dir for windows.

chicm-ms added 30 commits August 6, 2019 11:19
Filter prune algo implementation (microsoft#1655)
document the dispatcher working dir (microsoft#1866)
@liuzhe-lz liuzhe-lz mentioned this pull request Nov 30, 2020
77 tasks
@QuanluZhang QuanluZhang requested a review from ultmaster December 5, 2020 07:30
parser_package_install = parser_package_subparsers.add_parser('install', help='install packages')
parser_package_install.add_argument('source', nargs='?', help='installation source, can be a directory or whl file')
parser_package_install.add_argument('--name', '-n', dest='name', help='package name to be installed', required=False)
parser_package_install.set_defaults(func=package_install)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

package_install this function is not removed?

@QuanluZhang
Copy link
Contributor

looks great!

@liuzhe-lz liuzhe-lz merged commit 0d3f13a into microsoft:master Dec 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants