X Tutup
Skip to content

DOC: Fix psd_demo_complex similarly to psd_demo3#6843

Merged
tacaswell merged 3 commits intomatplotlib:masterfrom
afvincent:fix_psd_demo_complex
Jul 28, 2016
Merged

DOC: Fix psd_demo_complex similarly to psd_demo3#6843
tacaswell merged 3 commits intomatplotlib:masterfrom
afvincent:fix_psd_demo_complex

Conversation

@afvincent
Copy link
Copy Markdown
Contributor

Reports on psd_demo_complex some fixes similar to the ones done in #6841 on psd_demo3 (from which this example is derived):

  • OO code style
  • set a fixed y-range

NB: as the axes were not shared between the subplots in psd_demo3, I removed the sharing in this PR. It might be reintroduced if needed.

Besides, the 2nd commit halves the x-tick amount, because otherwise the x-tick labels overlap. Currently, the example looks like:
fix_psd_demo_complex
vs the current one in the devdocs gallery:
devdocs_psd_demo_complex

BTW, is it normal for the PNG in the devdocs gallery to be only 550x450 pixels?

@afvincent
Copy link
Copy Markdown
Contributor Author

Now forcing the (pseudo-)random draws to be repeatable.

@tacaswell tacaswell merged commit e6efa8e into matplotlib:master Jul 28, 2016
tacaswell added a commit that referenced this pull request Jul 28, 2016
DOC: Fix `psd_demo_complex` similarly to `psd_demo3`
@tacaswell
Copy link
Copy Markdown
Member

backported to v2.x as d1d7d7a

@QuLogic QuLogic added this to the 2.0 (style change major release) milestone Jul 28, 2016
@afvincent afvincent deleted the fix_psd_demo_complex branch July 28, 2016 07:26
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.

4 participants

X Tutup