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

DOC: Remove unfinished Linear Algebra section from Quickstart tutorial #18448

Merged
merged 2 commits into from
Feb 21, 2021

Conversation

rossbar
Copy link
Contributor

@rossbar rossbar commented Feb 19, 2021

The Quickstart tutorial has an incomplete Linear Algebra section that has been marked as a WIP since at least 1.13. This PR suggests replacing this section with a link to the svd tutorial which is much more complete and covers a lot of the same material.

Alternatively, this section (or at the very least the sentence about being a WIP) could be removed entirely so it no longer stands out as incomplete.

Replace WIP section with link to well-developed svd tutorial.
@rossbar rossbar changed the title Remove unfinished Linear Algebra section from Quickstart tutorial DOC: Remove unfinished Linear Algebra section from Quickstart tutorial Feb 19, 2021
@mattip
Copy link
Member

mattip commented Feb 21, 2021

I wonder if we shouldn't remove the "linear algebra" section altogether and add the link to the "further reading" section, which could also get a link to the fft and random packages. Also, near the beginning the document states "This is a quick overview of algebra and arrays in NumPy", we should probably remove the "algebra" from that.

 * Remove reference to algebra in the intro text.
 * Link to linalg tutorial in Further Reading
 * Remove linalg heading + WIP from quickstart.
@mattip mattip merged commit 60d46ea into numpy:master Feb 21, 2021
@mattip
Copy link
Member

mattip commented Feb 21, 2021

Thanks @rossbar

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

Successfully merging this pull request may close these issues.

None yet

2 participants