[Bug]: (Tight) layout engine breaks for 3D patches #27361

thrien · 2023-11-22T19:23:45Z

Bug summary

The tight layout sometimes breaks for plots with 3D patches because 'PathPatch3D' object has no attribute '_path2d'.

Code for reproduction

import matplotlib.pyplot as plt
import matplotlib.patches as pat
import mpl_toolkits.mplot3d.art3d as art3d

fig, ax = plt.subplots(subplot_kw={"projection": "3d"}, layout="tight") # , dpi=300)

p = pat.Rectangle((0, 0), 1.0, 1.0)
ax.add_patch(p)
art3d.pathpatch_2d_to_3d(p)

# fig.set_dpi(300)
# fig.tight_layout()

fig.savefig("test.pdf")  #, dpi=300)

Actual outcome

Traceback (most recent call last):
  File "/home/tobias/mne3.py", line 11, in <module>
    fig.savefig("test.pdf")  #, dpi=300)
    ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/figure.py", line 3343, in savefig
    self.canvas.print_figure(fname, **kwargs)
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/backends/backend_qtagg.py", line 75, in print_figure
    super().print_figure(*args, **kwargs)
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/backend_bases.py", line 2342, in print_figure
    self.figure.draw(renderer)
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/artist.py", line 95, in draw_wrapper
    result = draw(artist, renderer, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/artist.py", line 72, in draw_wrapper
    return draw(artist, renderer)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/figure.py", line 3134, in draw
    self.get_layout_engine().execute(self)
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/layout_engine.py", line 178, in execute
    kwargs = get_tight_layout_figure(
             ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/_tight_layout.py", line 266, in get_tight_layout_figure
    kwargs = _auto_adjust_subplotpars(fig, renderer,
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/_tight_layout.py", line 82, in _auto_adjust_subplotpars
    bb += [martist._get_tightbbox_for_layout_only(ax, renderer)]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/artist.py", line 1415, in _get_tightbbox_for_layout_only
    return obj.get_tightbbox(*args, **{**kwargs, "for_layout_only": True})
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/mpl_toolkits/mplot3d/axes3d.py", line 3189, in get_tightbbox
    ret = super().get_tightbbox(renderer,
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/axes/_base.py", line 4408, in get_tightbbox
    bbox = a.get_tightbbox(renderer)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/artist.py", line 367, in get_tightbbox
    bbox = self.get_window_extent(renderer)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/matplotlib/patches.py", line 604, in get_window_extent
    return self.get_path().get_extents(self.get_transform())
           ^^^^^^^^^^^^^^^
  File "/home/tobias/.local/lib/python3.11/site-packages/mpl_toolkits/mplot3d/art3d.py", line 420, in get_path
    return self._path2d
           ^^^^^^^^^^^^
AttributeError: 'PathPatch3D' object has no attribute '_path2d'

Expected outcome

The following code actually produces (something that is at least close enough to) the expected outcome. It does NOT produce an error even though I would expect it to be equivalent.

import matplotlib.pyplot as plt
import matplotlib.patches as pat
import mpl_toolkits.mplot3d.art3d as art3d

fig, ax = plt.subplots(subplot_kw={"projection": "3d"})

p = pat.Rectangle((0, 0), 1.0, 1.0)
ax.add_patch(p)
art3d.pathpatch_2d_to_3d(p)

fig.tight_layout()

fig.savefig("test.pdf")

The result is different compared to the case without the tight layout engine, so I assume it is doing something. However I am not sure if it is fully doing it's job.

Additional information

Whether I get an error or not depends on some weird circumstances (e.g. the behaviour depends on dpi settings). I left some possible modifications as comments in the minimal example above. Here is a short summary of some test I did:

layout="tight" works with fig.savefig("test.png") but NOT with fig.savefig("test.png", dpi=300).
Similarly if I use plt.subplots(..., dpi=300) or fig.set_dpi(300) the call to fig.tight_layout() already gives an error and NOT the call to fig.savefig().
In most cases where it breaks for layout="tight" it also breaks for "compressed" or "constrained".

Operating system

Arch

Matplotlib Version

3.7.1

Matplotlib Backend

QtAgg

Python version

Python 3.11.5

Jupyter version

No response

Installation

Linux package manager

The text was updated successfully, but these errors were encountered:

rcomer · 2023-11-23T21:19:49Z

Thanks for the report @thrien. I confirm that I have reproduced this with main.

The problem is that the _path2d attribute is currently only created at draw, by calling the do_3d_projection method on the artist. That method depends on Axes3D.M, which is also currently only created at draw.

A workaround is to call fig.draw_without_rendering before saving.

oscargus · 2023-11-23T21:51:52Z

Quick reply, but I think I have fixed a similar bug earlier. Hence, it may be good to check all 3D-objects for similar problems.

oscargus · 2023-11-24T07:42:33Z

The PR I was thinking about, #23562, didn't solve exactly the same problem, but a similar approach to solving it can be used: check if the attribute exists, if not, create it. Would you like to give it a go @thrien?

rcomer · 2023-11-24T07:49:26Z

check if the attribute exists, if not, create it

What happens if the attribute exists but the view angle changed since it was created? I wonder if we need to just always re-create. That seems to be what happens within draw.

oscargus · 2023-11-24T07:56:05Z

Good question. It can probably be wrong (but not crash...). Maybe one need some sort of "stale" behavior here and keep track of if one need to do the 3D-projection again? (Without checking the exact code, I see a risk of doing the 3D-projection more than once...)

I guess one way can be to add an optional argument to all the get-methods which controls if the 3D-projection should be done before getting the result. True by default, but can be set to False if performance becomes a problem and one know that it is not required.

thrien · 2023-11-24T17:30:04Z

The PR I was thinking about, #23562, didn't solve exactly the same problem, but a similar approach to solving it can be used: check if the attribute exists, if not, create it. Would you like to give it a go @thrien?

That's seems like an easy fix. However I cannot build myself because of some annoying issues with the setup. (I'm trying this for the first time). So I can't do it myself (yet).

thrien · 2023-12-02T11:59:04Z

I fixed my issues and was able to build a version with the simple fix following the suggestion of @oscargus. It does not address the "stale" issue mentioned above. The commit is here. I was able to build and then run the new test that I've implemented. However I was not able to run all the tests.

Should I already create a pull request? It is only a partial solution but it is still an improvement since as far as I understand at draw the projection will always be called again so everything that worked before should still work the same.

(Maybe someone can comment on whether the docstring is actually inherited because I have no idea how this works.)

oscargus · 2023-12-03T12:28:49Z

I think that patch is good to open a PR with.

Yes, the doc-string should be inherited.

rcomer added the status: confirmed bug label Nov 23, 2023

scottshambaugh added the topic: mplot3d label Nov 26, 2023

thrien mentioned this issue Dec 4, 2023

Fix get_path for 3d artists #27440

Merged

3 tasks

ksunden closed this as completed in #27440 Dec 4, 2023

QuLogic added this to the v3.9.0 milestone Dec 4, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[Bug]: (Tight) layout engine breaks for 3D patches #27361

[Bug]: (Tight) layout engine breaks for 3D patches #27361

thrien commented Nov 22, 2023 •

edited by oscargus

Loading

rcomer commented Nov 23, 2023

Uh oh!

oscargus commented Nov 23, 2023

Uh oh!

oscargus commented Nov 24, 2023

Uh oh!

rcomer commented Nov 24, 2023

Uh oh!

oscargus commented Nov 24, 2023

Uh oh!

thrien commented Nov 24, 2023 •

edited

Loading

Uh oh!

thrien commented Dec 2, 2023

Uh oh!

oscargus commented Dec 3, 2023

Uh oh!

Uh oh!

[Bug]: (Tight) layout engine breaks for 3D patches #27361

[Bug]: (Tight) layout engine breaks for 3D patches #27361

Comments

thrien commented Nov 22, 2023 • edited by oscargus Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Bug summary

Code for reproduction

Actual outcome

Expected outcome

Additional information

Operating system

Matplotlib Version

Matplotlib Backend

Python version

Jupyter version

Installation

rcomer commented Nov 23, 2023

Uh oh!

oscargus commented Nov 23, 2023

Uh oh!

oscargus commented Nov 24, 2023

Uh oh!

rcomer commented Nov 24, 2023

Uh oh!

oscargus commented Nov 24, 2023

Uh oh!

thrien commented Nov 24, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

thrien commented Dec 2, 2023

Uh oh!

oscargus commented Dec 3, 2023

Uh oh!

thrien commented Nov 22, 2023 •

edited by oscargus

Loading

thrien commented Nov 24, 2023 •

edited

Loading