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

Update shared storage doc #4217

Merged
merged 2 commits into from
Sep 28, 2021
Merged

Update shared storage doc #4217

merged 2 commits into from
Sep 28, 2021

Conversation

J-shang
Copy link
Contributor

@J-shang J-shang commented Sep 26, 2021

No description provided.

@@ -12,6 +12,9 @@ The main differences are as follows:
* Need to provide the model to be compressed, and the model should have already been pre-trained.
* No need to set ``trial_command``, additional need to set ``auto_compress_module`` as ``AutoCompressionExperiment`` input.

.. note::
Auto compression only support TPE Tuner, Random Search Tuner, Anneal Tuner, Evolution Tuner right now.
Copy link
Contributor

Choose a reason for hiding this comment

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

support => supports

Copy link
Contributor Author

Choose a reason for hiding this comment

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

THX, fix it

@@ -13,6 +13,12 @@ And we will develop more practical features in the future based on shared storag
Shared storage is currently in the experimental stage. We suggest use AzureBlob under Ubuntu/CentOS/RHEL, and NFS under Ubuntu/CentOS/RHEL/Fedora/Debian for remote.
And make sure your local machine can mount NFS or fuse AzureBlob and the machine used in training service has ``sudo`` permission without password. We only support shared storage under training service with reuse mode for now.

.. note::
What is the difference between training service native storage and shared storage? Training service native storage is usually provided by the specific training service.
e.g., the local storage on remote machine in remote mode, the provided storage in openpai mode. These storages might not easy to use, e.g., users have to upload datasets to all remote machines to train the model.
Copy link
Contributor

Choose a reason for hiding this comment

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

e.g. => E.g.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@acured acured requested a review from SparkSnail September 27, 2021 02:42
@acured acured merged commit 8700102 into microsoft:master Sep 28, 2021
@J-shang J-shang deleted the ss-doc branch October 25, 2021 03:25
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.

3 participants