-
Notifications
You must be signed in to change notification settings - Fork 909
Cht add pcolormesh #91
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
Conversation
dfe4f4c
to
bd6a6ca
Compare
bd6a6ca
to
c692d8e
Compare
New version: |
I don't really understand why the trim trailing whitespace is failing... |
There are 4 space characters in line 14. Might have snuck in when pre-commit checks were not looking... cheatsheets/.circleci/config.yml Lines 1 to 18 in 297ef35
|
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 agree that pcolormesh is more relevant than xcorr.
pcolormesh
does not appear in the cheatsheet, yet it is often more useful thanimshow
. Suggest replacingxcorr
(which is pretty obscure, and plotted in a digital manner, which is not how I've ever seen it done) withpcolormesh
. One could argue that adding astem
plot instead ofxcorr
may be helpful, however if we did that I'd probably get rid ofeventplot
.