-
Notifications
You must be signed in to change notification settings - Fork 932
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
[WIP] Edit User Guide to offer more information about usability and use cases #627
Draft
willingc
wants to merge
18
commits into
pypa:main
Choose a base branch
from
willingc:doc-sci
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
188b160
Add note on navigation and prose edits
willingc 00a7174
simplify section title
willingc 2fb27be
add basics of installation
willingc 41a47a2
move creating documentation from tutorials to guides
willingc c3abe08
update local table of contents
willingc 07434ee
Refine note box links and headings
willingc 5794b4c
add skeleton for basics of installation
willingc 56d4e4f
add name to authors
willingc c9c54dc
add sections to basic doc
willingc 2d921df
add note for target audience of basics document
willingc 94ad36e
fix typo
willingc bb8db4d
add todo comment
willingc 1495f1a
edit prose to simplify for user reading
willingc cbc3c61
Rename discussions to concepts for better discovery
willingc d52a8b3
rename guides to tasks
willingc e272fed
replace box with a table
willingc cdae51c
add overview and prereqs to sci pkg doc
willingc 71dad94
add section headings and content to science doc
willingc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,49 @@ | ||
====================== | ||
Basics of Installation | ||
====================== | ||
|
||
.. Editors, see notes at the bottom of the document for maintenance info. | ||
|
||
.. contents:: Contents | ||
:local: | ||
|
||
Jargon | ||
------ | ||
|
||
- System Python | ||
- Selected Python | ||
- Environments (None, virtual, conda) | ||
- Package manager/tool (pipenv, pip, conda, miniconda, others) | ||
- Advanced installation cases | ||
|
||
Installation using PyPA Tools | ||
----------------------------- | ||
|
||
|
||
Third Party Installation Tools | ||
------------------------------ | ||
|
||
Best Practices Checklist | ||
------------------------ | ||
|
||
- Review the installation documentation for the package | ||
- Install the package by following the installation documentation | ||
- Most packages will be installed with pipenv or pip. | ||
- [Science and Data Science Packages] The installation instructions may include | ||
references to conda and conda-forge. Follow those instructions. | ||
|
||
Wrap up | ||
------- | ||
|
||
|
||
|
||
.. Editing notes: | ||
|
||
This document targets a novice audience especially scientists who may | ||
not have a computer development background. It's expected | ||
that most developers finding this document will have experience installing | ||
applications and apps, but may not have installed a Python package before | ||
or be familiar with the command line. | ||
|
||
For specific how-to information refer to the documentation of the desired | ||
package for installation. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,21 @@ | ||
Discussions | ||
########### | ||
Concepts | ||
######## | ||
|
||
**Discussions** are focused on providing comprehensive information about a | ||
specific topic. If you're just trying to get stuff done, see | ||
:doc:`/guides/index`. | ||
The **Concepts** section discusses fundamental information about specific | ||
topics. | ||
|
||
The goals of this section include providing background information, describing | ||
important technical concepts, and highlighting usage tradeoffs. | ||
|
||
Contents include: | ||
|
||
.. toctree:: | ||
:maxdepth: 1 | ||
|
||
deploying-python-applications | ||
pip-vs-easy-install | ||
install-requires-vs-requirements | ||
pip-vs-easy-install | ||
wheel-vs-egg | ||
deploying-python-applications | ||
|
||
Alternatively, if you're trying to get a specific task done, see | ||
:doc:`/guides/index`. |
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I do not particularly want to court controversy here, but I think we may want to be very careful about the degree to which
pipenv
is included or featured in this documentation. There has already been a lot of confusion sewn about how "official"pipenv
is.I think putting
pipenv
beforepip
in this sentence will unfortunately be seized upon by the sort of "kremlinologists" who may scrutinize documentation in some sort of legalistic manner for the "word from on high".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.
I can flip them.
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.
FYI. The basics doc is still notes and not even to draft state yet. 😉
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.
I'd phrase this a bit differently:
setup.py
directly, it is likely just outdated information).As much as I'd like to be able to tell users "trust the documentation of the packages you're using", we unfortunately can't do that: