Skip to content

MRG, BUG: Fix handling of axes with constrained_layout=True#8344

Merged
hoechenberger merged 2 commits intomne-tools:masterfrom
larsoner:constrained
Oct 6, 2020
Merged

MRG, BUG: Fix handling of axes with constrained_layout=True#8344
hoechenberger merged 2 commits intomne-tools:masterfrom
larsoner:constrained

Conversation

@larsoner
Copy link
Copy Markdown
Member

@larsoner larsoner commented Oct 5, 2020

We should only internally do tight_layout(...) calls when axes were not created using constrained_layout=True. Our current code gets a matplotlib warning about disabling the constrained_layout of the axes.

Also fixes a bug where we would raise as error instead of re-emit as a warning

@larsoner larsoner added the backport-candidate on-merge: backport to maint/1.12 label Oct 5, 2020
@hoechenberger
Copy link
Copy Markdown
Member

One Travis job timed out after successfully running all tests.

@hoechenberger hoechenberger merged commit ce4d514 into mne-tools:master Oct 6, 2020
@hoechenberger
Copy link
Copy Markdown
Member

Thanks @larsoner

Can you do the backporting also?

@larsoner larsoner deleted the constrained branch October 6, 2020 11:23
larsoner added a commit that referenced this pull request Oct 6, 2020
* BUG: Fix handling of axes with constrained_layout=True

* FIX: Version
@larsoner larsoner added backported and removed backport-candidate on-merge: backport to maint/1.12 labels Oct 6, 2020
marsipu pushed a commit to marsipu/mne-python that referenced this pull request Oct 14, 2020
…s#8344)

* BUG: Fix handling of axes with constrained_layout=True

* FIX: Version
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.

2 participants