-
Notifications
You must be signed in to change notification settings - Fork 40
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
Edit user plugin installation page to simplify instructions #586
base: main
Are you sure you want to change the base?
Conversation
c776bfc
to
8ffef2d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great changes!
Really appreciate the addition of more examples :)
docs/plugins/start_using_plugins/finding_and_installing_plugins.md
Outdated
Show resolved
Hide resolved
docs/plugins/start_using_plugins/finding_and_installing_plugins.md
Outdated
Show resolved
Hide resolved
docs/plugins/start_using_plugins/finding_and_installing_plugins.md
Outdated
Show resolved
Hide resolved
docs/plugins/start_using_plugins/finding_and_installing_plugins.md
Outdated
Show resolved
Hide resolved
```{admonition} Example | ||
If you want to install `napari-svg` from a specific release, enter `napari-svg==0.1.0` in the text field. | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
```{admonition} Installing with Optional Dependency Groups | |
To install a plugin with a group of option dependencies, use the optional group in brackets. If you want to install `napari-svg` with the optional testing group, enter `napari-svg[testing]` in the text field. | |
``` | |
I find this might be the most common use of the test field with users I know -- because optional groups otherwise can't be installed from GUI.
Maybe napari-svg isn't the best example because its only optional is testing. Something like napari-nD-annotator[bbox]
might be more informative?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding your existing suggestion to the batch for commit. I think your suggestion about the annotator is a good one. Happy to commit if you add it as a second example.
Co-authored-by: Tim Monko <timmonko@gmail.com>
@TimMonko Thanks for the user-focused review suggestions. |
References and relevant issues
Closes #224.
Addresses #582.
Together with with #581 improves the user's understanding of plugins, their discovery, and installation.
Description
This PR simplifies the prose using concepts from Google's Docs Style Guide