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

Remove unused configuration in doc #1841

Merged
merged 22 commits into from
Dec 13, 2019
Merged

Conversation

SparkSnail
Copy link
Contributor

@SparkSnail SparkSnail commented Dec 10, 2019

dataDir and outputDir is not supported in PAI mode.

@SparkSnail SparkSnail requested review from LinquanLiu and QuanluZhang and removed request for LinquanLiu December 10, 2019 07:07
@SparkSnail SparkSnail requested a review from chicm-ms December 10, 2019 07:11
@chicm-ms chicm-ms requested a review from squirrelsc December 10, 2019 08:36
@chicm-ms
Copy link
Contributor

@squirrelsc I rember we should not update Chinese document, right?

@@ -112,10 +112,6 @@ trial:
memoryMB: 32869
#The docker image to run nni job on OpenPAI
image: msranni/nni:latest
Copy link
Contributor

Choose a reason for hiding this comment

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

So how to set our dataDir and outputDir?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So how to set our dataDir and outputDir?

PAI does not support setting outputDir from configuration anymore, the dataDir path NNI used is hdfs://host:port/{username}/nni/{experiments}/{experimentId}/trials/{trialId}/nnioutput, NNI does not support user setting the path. refer

Copy link
Contributor

Choose a reason for hiding this comment

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

@SparkSnail Thanks. But the current usage in PAI seems complicated and troublesome. Can we make it easier?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@SparkSnail Thanks. But the current usage in PAI seems complicated and troublesome. Can we make it easier?

Sure, we are support using NFS for NNI to upload data to PAI, it will be easier to use in the future.

@SparkSnail
Copy link
Contributor Author

@squirrelsc I rember we should not update Chinese document, right?

fixed, removed change on chinese doc.

@SparkSnail SparkSnail merged commit 0bb80f3 into microsoft:master Dec 13, 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