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: fix several typos #8537. #8575

Merged
merged 3 commits into from
Feb 10, 2017
Merged

DOC: fix several typos #8537. #8575

merged 3 commits into from
Feb 10, 2017

Conversation

godaygo
Copy link
Contributor

@godaygo godaygo commented Feb 7, 2017

I've made corrections to my previous PR #8544 (which I closed).

I've also noticed that in documentation, it is nothing said that np.int, np.float, etc. returns Python's int and float instead of array scalar. The first time it was pretty confusing to me. If it's OK I will add a commit, but I don`t know the right place in documentation where to add this stuff.

The example with `newaxis` now looks rather silly. I left it
because it is the only place in this tutorial with an example
how to use `newaxis`.
@juliantaylor
Copy link
Contributor

numpy/doc/basics.py might be a good place

@charris
Copy link
Member

charris commented Feb 10, 2017

np.int, np.float

They are aliases of the Python types.

@charris
Copy link
Member

charris commented Feb 10, 2017

Thanks @godaygo . For future reference, github has the quirk that [ci skip] needs to be in all commit messages to avoid the testing.

@charris
Copy link
Member

charris commented Feb 10, 2017

Also, if you had added Closes #8537 in one of the commit messages that issue would automatically have been closed when this was merged.

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

3 participants