MEP12 of showcase's examples + other folders.#7952
MEP12 of showcase's examples + other folders.#7952anntzer merged 8 commits intomatplotlib:masterfrom NelleV:MEP12_showcase
Conversation
NelleV
changed the title
| @@ -1,62 +0,0 @@ | |||
| """ | |||
There was a problem hiding this comment.
I removed this example is being broadly a duplicate of interpolation_methods.
There was a problem hiding this comment.
It does point out an important difference between the "none" and "nearest" methods though. This information needs to remain available somewhere.
There was a problem hiding this comment.
You mean the text or the example itself?
There was a problem hiding this comment.
Both of the text and the images IMO, though not necessarily as an example (eg. it could be a FAQ entry instead).
There was a problem hiding this comment.
I think it is very unlikely that anyone goes to that example to understand the difference between None and nearest. I think the interpolation methods (accompanied by more text explanation) is more than enough to understand the difference.
There was a problem hiding this comment.
It does not have to be an example but I would be very sorry to see that figure go. I have on several occasions benefitted from having this figure to explain the difference. A graphical explanation is so much more powerfull than text only. The interpolation_methods.py example does nothing to explain this as None none and nearest are rendered identically
There was a problem hiding this comment.
You mean interpolation_methods.py? Right now it certainly doesn't contain all this information.
There was a problem hiding this comment.
No, but too much information is also not useful. We need to drastically cut down the number of examples we have because we dilute the information too much. I can add more text on the interpolation_methods.py example but if I have to do this for all examples we'll never have a gallery or even documentation that is in any way useful.
There was a problem hiding this comment.
I've added some detailed text to interpolation methods, mostly copy pasting from the example I removed.
I am stressing once again that we need to cut back the number of examples. Most are extremely poorly designed, with unclear message and can be merged into one single example. It drowns the information too much.
To be specific, we have more than 500 examples right now, many displaying several images. There is no way anyone can sort or process any kind of information from these. If it helps convince you, I can have a communication expert look at the examples and the text and help us with the process of cleaning up our gallery.
There was a problem hiding this comment.
I completely agree with your assessment that we have too many examples and that many are poorly designed. However this specific example is exactly addressing a very specific question (the apparent duplication of interpolation methods between "none" and "nothing"). Thus my suggestion to move it to a FAQ entry.
|
http://matplotlib.org/examples/pylab_examples/manual_axis.html seems worth removing as well. |
| This example displays the difference between interpolation methods for imshow | ||
| and matshow. | ||
|
|
||
| If `intperpolation` is None, it defaults to the rc image.interpolation |
|
|
||
| If `intperpolation` is None, it defaults to the rc image.interpolation | ||
| parameter. If the interpolation is `none`, then no interpolation is performed | ||
| for the Agg, ps and pdf backends. Other backends will defult to 'nearest'. |
There was a problem hiding this comment.
in the Agg, ps, and pdf
And "Other backends default to nearest"
| # from the docs: | ||
| For the Agg, ps and pdf backends, interpolation = 'none' works well when a big | ||
| image is scaled down, while interpolation = 'nearest' works well when a small | ||
| image is blown up. |
There was a problem hiding this comment.
large image is scaled down, small image is scaled up. May as well keep the language parallel.
| """ | ||
| Simple demo of a scatter plot. | ||
| ============= | ||
| Scatter plots |
|
I've fixed the typos in the text I added to the interpolation_methods example. That text clearly needs more work, but I think it is out of the scope of this PR. |
story645
changed the title
|
Thanks, merging and opening a separate issue regarding interpolation_none_vs_nearest. |
QuLogic
changed the title
5 participants
Small MEP12-ifying showcase.
refs #7206