Repository navigation
tight_layout() doesn't work properly on plots with a WCSAxes projection #8703
Description
Activity
Is this related to #8678 (comment) ?
I don't think so, that is only a problem on Matplotlib dev (post version
3.0.3), whereas this is an issue on a version of Maptlotlib (3.0.3) that predates that change.I have some ideas :)
A workaround for now is to call:
fig.canvas.draw()before
fig.tight_layout()Basically the fundamental issue is that
get_tightbboxactually requires the figure to be drawn once before it returns correct results. We should probably find a way to avoid this - but in the mean time we could overloadtight_layoutinWCSAxesand just force a draw first.Reacted by P. L. LimFor reference, some simpler code to reproduce this:
import matplotlib.pyplot as plt from astropy.wcs import WCS from astropy.io import fits from astropy.utils.data import get_pkg_data_filename filename = get_pkg_data_filename('galactic_center/gc_msx_e.fits') hdu = fits.open(filename)[0] wcs = WCS(hdu.header) fig = plt.figure(figsize=(8, 6)) axs = [fig.add_subplot(2, 1, i, projection=wcs) for i in [1, 2]] fig.tight_layout() plt.show()
Reacted by P. L. Lim and Vital FernándezAs I slowly descend into madness trying to work this out, a note that the end goal to get this working is to have
TickLabelsandAxisLabelshave their ownget_tight_bbox()methods that can calculate their own Bboxes. This is going to need a lot of refactoring, but I suspect in the long run it should significantly tidy up the WCSAxes code.Reacted by P. L. Lim
A Matpltolib
Axescreated withprojection=wcsdoes not play well with Matplotlib'stight_layout()method, that is supposed to re-size elements of a figure such that there are no overlapping labels/axes etc.Consdier the following example:
projection=None, no tight_layout:projection=None, with tight_layout:(note how this re-organises things nicely)
projection='wcs', no tight_layout:projection='wcs', with tight_layout:Note how things actually get worse!