Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Model Store document #1220

Closed
wants to merge 1 commit into from
Closed

Update Model Store document #1220

wants to merge 1 commit into from

Conversation

IsaacYangSLA
Copy link
Contributor

Update document on UI changes.
Replace old screenshot with new UI.

Copy link
Member

@lukeyeager lukeyeager left a comment

Choose a reason for hiding this comment

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

Wordsmithery

@@ -27,12 +27,13 @@ The new page shows models available in model stores.

![List models](images/model-store-list.png)

Hover the Note field to show complete text.
Hover on Note will show the complete notes.
Copy link
Member

Choose a reason for hiding this comment

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

How about:

Hover over "Note" to show the complete notes.

Enter keyword in 'Filter list by' to limit results.
Click 'Update model list' button will retrieve the latest model list (see limitation).
Click 'Import' will import that model into DIGITS (may takes a few seconds, depends on network speed).
Click 'Import' icon will import that model into DIGITS (may takes a few seconds, depends on network speed).
Copy link
Member

Choose a reason for hiding this comment

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

How about:

Click "Import" to import that model into DIGITS (may take a few seconds, depending on network speed).

@@ -69,4 +70,4 @@ The `license.txt` inside that subdirectory will be displayed when users clicking
## Limitation
Some web server may limit frequent requests from the same machine to stop malicious activities.
Therefore, DIGITS implemented a cache mechanism to reduce server-to-server communication.
The button, 'Update model list', invalidates cache and retrieve meta data from all models.
The button, 'Update model list', invalidates cache and retrieve meta data from all models from each Model Store.
Copy link
Member

Choose a reason for hiding this comment

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

How about:

invalidates the cache and retrieves metadata from

@IsaacYangSLA
Copy link
Contributor Author

Close this one in favor of #1346.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants