-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
ENH: Improved documentation of parameters 'scaling' and 'mode' of signal.spectrogram() #16000
Conversation
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.
Otherwise LGTM
89fa5db
to
9195288
Compare
The content in #14903 (comment) seems useful -- would you be up for adding some variant of this to the |
9195288
to
42691ff
Compare
I initially created the table for the documentation. I decided to leave it out, since in IMHO the same information is already in the documentation of the Nevertheless, I think that both the |
Sounds good. Incremental progress is better than no progress here! And happy to look at follow-up PRs that make things more consistent, usable, and/or complete. |
In the section 'Notes': * Note that it's a wrapper to the stft() * Practical usage of studying evolution fo spectra * Investigate power and maximum behavior for single frequency signal.
42691ff
to
e72bf43
Compare
Can be closed due to #17408 being merged. |
This PR tries to improve the docstring of signal.spectrogram().
Closes #14903