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

Documentation Errors for specgram #840

Merged
merged 1 commit into from Aug 21, 2012
Merged

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Aug 20, 2012

Im sorry if im breaking some standard of communication with these issues.

Im using matplotlib version 1.2.x in MacOs Lion

The documentation errors are located in http://matplotlib.sourceforge.net/api/pyplot_api.html#matplotlib.pyplot.specgram

The function prototype states that the default parameter for overlap - noverlap is 128 while in the description it is stated that is 0. I think the correct is 128.

  1. The description for the return value at the end states that Pxx is a len(times) x len(freqs) array of powe,r while in reality Pxx is an array with len(freqs) and each subarray is len(bins)

@pelson
Copy link
Member

pelson commented Aug 19, 2012

Thanks @jdcaballero. No, your not breaking any protocols :-) . If you come across anything else, please do let us know.
If your keen to give it ago, we love receiving pull requests even more than issues!

mdboom added a commit that referenced this pull request Aug 21, 2012
Documentation Errors for specgram
@mdboom mdboom merged commit 9ae66ef into matplotlib:master Aug 21, 2012
@mdboom mdboom deleted the issue840 branch March 3, 2015 18:44
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