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

[v2.0] Refactor code hierarchy (part 1) #2962

Merged
merged 12 commits into from
Oct 19, 2020
Merged

Conversation

liuzhe-lz
Copy link
Contributor

@liuzhe-lz liuzhe-lz commented Oct 15, 2020

This is the first part of code hierarchy refactor. The PRs are split to make review easier.

To install NNI from this branch:

$ make
$ python3 -m pip install -U -e .

What part 1 includes:

  • Move Python codes into one directory
  • Install scripts for Linux and Mac

What part 1 breaks:

  • Release build
  • Windows build
  • Unit tests

What part 1 not includes:

  • Refactor code hierarchy inside nni Python package
  • Rewrite building scripts in Python
  • Update doc

Moved directories:

source destination
Files no more needed archive-del
Python unit tests archive-ut
Docs in moved packages archive-doc
src/sdk/pynni/nni nni
src/sdk/pycli/nnicli nni/nnicli
tools/nni_annotation nni/nni_annotation
tools/nni_cmd nni/nni_cmd
tools/nni_gpu_tool nni/nni_gpu_tool
tools/nni_trial_tool nni/nni_trial_tool
src/nni_manager nni_node/nni_manager
src/webui nni_node/webui
src/nasui nni_node/nasui

@liuzhe-lz liuzhe-lz changed the base branch from v2.0 to master October 15, 2020 11:04
@liuzhe-lz liuzhe-lz changed the base branch from master to v2.0 October 15, 2020 11:04
@liuzhe-lz liuzhe-lz closed this Oct 15, 2020
@liuzhe-lz liuzhe-lz reopened this Oct 15, 2020
@liuzhe-lz
Copy link
Contributor Author

/AzurePipelines run microsoft.nni

@azure-pipelines
Copy link
Contributor

No pipelines are associated with this pull request.

@liuzhe-lz liuzhe-lz changed the base branch from v2.0 to master October 15, 2020 11:08
@liuzhe-lz liuzhe-lz closed this Oct 15, 2020
@liuzhe-lz liuzhe-lz reopened this Oct 15, 2020
@liuzhe-lz liuzhe-lz changed the base branch from master to v2.0 October 15, 2020 14:45
@liuzhe-lz liuzhe-lz changed the base branch from v2.0 to master October 16, 2020 10:15
@liuzhe-lz liuzhe-lz changed the base branch from master to v2.0 October 16, 2020 11:07
@liuzhe-lz liuzhe-lz merged commit f98ee67 into microsoft:v2.0 Oct 19, 2020
Copy link

@Tudor33 Tudor33 left a comment

Choose a reason for hiding this comment

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

@QuanluZhang
Copy link
Contributor

@liuzhe-lz @SparkSnail suggest to change nni_cmd to nnictl

@liuzhe-lz liuzhe-lz deleted the v2.0-init branch October 27, 2020 18:57
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.

5 participants