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

Docs improvement: configurations and more #1823

Merged
merged 12 commits into from
Dec 16, 2019

Conversation

ultmaster
Copy link
Contributor

No description provided.

@xuehui1991 xuehui1991 closed this Dec 5, 2019
@xuehui1991 xuehui1991 reopened this Dec 5, 2019
@@ -1,6 +1,6 @@
# NNI on Windows (experimental feature)

Currently we support local, remote and pai mode on Windows. Windows 10.1809 is well tested and recommended.
Running NNI on Windows is an experimental feature. Windows 10.1809 is well tested and recommended.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is NNI on Windows still experimental?

Copy link
Contributor

Choose a reason for hiding this comment

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

@SparkSnail I think our windows version only contains part of the function of nni?

Copy link
Contributor

Choose a reason for hiding this comment

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

@SparkSnail I think our windows version only contains part of the function of nni?

Yes, NNI only support local/remote/pai on windows, I don't know if we will support all of platform in the feature. In this point, maybe it's reasonable that we call it an experimental feature.

Copy link
Contributor

Choose a reason for hiding this comment

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

could remove this sentence. could be replaced with We have supported NNI on Windows.

Copy link
Contributor

Choose a reason for hiding this comment

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

But we did not support all functions in Windows. We should point out which functions we support, which not.

@@ -1,6 +1,6 @@
# NNI on Windows (experimental feature)

Currently we support local, remote and pai mode on Windows. Windows 10.1809 is well tested and recommended.
Running NNI on Windows is an experimental feature. Windows 10.1809 is well tested and recommended.
Copy link
Contributor

Choose a reason for hiding this comment

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

@SparkSnail I think our windows version only contains part of the function of nni?

@scarlett2018 scarlett2018 mentioned this pull request Dec 6, 2019
44 tasks

#### Windows
Simply filling the `machineList` section and then run:
Files in `codeDir` will be automatically uploaded to the remote machine. You can run NNI on different operating systems (Windows, Linux, MacOS) to spawn experiments on the remote machines:
Copy link
Contributor

Choose a reason for hiding this comment

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

better to mention that only linux is allowed on remote machines

#The hdfs directory to store data on OpenPAI, format 'hdfs://host:port/directory'
dataDir: hdfs://10.10.10.10:9000/username/nni
#The hdfs directory to store output data generated by nni, format 'hdfs://host:port/directory'
outputDir: hdfs://10.10.10.10:9000/username/nni
Copy link
Contributor

Choose a reason for hiding this comment

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

no need to specify dataDir?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, the configuration is removed.


* __local__ run an experiment on local ubuntu machine.
* TODO: explain frameworkcontroller.
Copy link
Contributor

Choose a reason for hiding this comment

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

frameworkcontroller submit trial jobs to kubernetes. It is built to orchestrate all kinds of applications (e.g., Tensorflow models, PyTorch models, Python scripts) on Kubernetes by a single controller. For more details of configuration for frameworkcontroller mode, please refer to FrameworkController Docs

@ultmaster ultmaster merged commit 843b642 into microsoft:master Dec 16, 2019
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