Skip to content

Tight layout options#215

Merged
lukelbd merged 2 commits intoproplot-dev:masterfrom
zmoon:tight_layout_options
Jul 3, 2021
Merged

Tight layout options#215
lukelbd merged 2 commits intoproplot-dev:masterfrom
zmoon:tight_layout_options

Conversation

@zmoon
Copy link
Contributor

@zmoon zmoon commented Aug 4, 2020

This would add the feature of #64, the ability to set hspace between all rows, wspace between all columns, or both, based on the calculated maximum value in ProPlot's layout routine.

I am not sure if this follows what you were suggesting in this comment. It seems to work, but the gridspec's idea of hspace and wspace don't seem to match how the figure shows up (see my example, maybe I am printing the wrong thing).

See in action here.

@bradyrx bradyrx requested a review from lukelbd October 22, 2020 16:03
@lukelbd lukelbd marked this pull request as ready for review July 3, 2021 16:23
@lukelbd
Copy link
Collaborator

lukelbd commented Jul 3, 2021

10 years later... looks good to me, thanks! Here's an example with this feature turned off and on:

fig, axs = plot.subplots(ncols=3, share=False, wequal=False)
axs[2].format(yformatter='null')
axs[1].format(ylabel='ylabel\nylabel\nylabel')
fig.show()

unequal

fig, axs = plot.subplots(ncols=3, share=False, wequal=True)
axs[2].format(yformatter='null')
axs[1].format(ylabel='ylabel\nylabel\nylabel')
fig.show()

equal

Will add this to the changelog + update the contributors list

@lukelbd lukelbd merged commit ac3f9f7 into proplot-dev:master Jul 3, 2021
@lukelbd
Copy link
Collaborator

lukelbd commented Jul 3, 2021

(Closes #64)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants