wxGUI: failed to show legend when PIL is missing

[martinl]

It's not possible to show legend when PIL package is not available, see traceback bellow

 File "/opt/src/grass_trunk/dist.x86_64-unknown-linux-gnu/etc/gui/wxpython/mapdisp/frame.py", line 187, in <lambda>
    overlay.overlayChanged.connect(lambda: self.MapWindow2D.UpdateMap(render=False, renderVector=False))
  File "/opt/src/grass_trunk/dist.x86_64-unknown-linux-gnu/etc/gui/wxpython/mapwin/buffered.py", line 720, in UpdateMap
    for img in self.GetOverlay():
  File "/opt/src/grass_trunk/dist.x86_64-unknown-linux-gnu/etc/gui/wxpython/mapwin/buffered.py", line 601, in GetOverlay
    img = utils.autoCropImageFromFile(overlay.mapfile)
  File "/opt/src/grass_trunk/dist.x86_64-unknown-linux-gnu/etc/gui/wxpython/core/utils.py", line 1103, in autoCropImageFromFile
    from PIL import Image

[annakrat]

I added special handling of missing PIL in r58256 (with advice to install it), however we should decide if PIL is a official requirement for wxGUI. Now it's used for cropping overlays (legend and so) so that you can move more decorations easily.

[hamish]

Replying to annakrat:

however we should decide if PIL is a official requirement for wxGUI.Now it's used for cropping overlays (legend and so) so that you can move more decorations easily.

fwiw it is already "encouraged", since if PIL is there the wxGUI PsMap and Animation tools like to use it too.

Hamish

[annakrat]

Replying to hamish:

Replying to annakrat:

however we should decide if PIL is a official requirement for wxGUI.Now it's used for cropping overlays (legend and so) so that you can move more decorations easily.

fwiw it is already "encouraged", since if PIL is there the wxGUI PsMap and Animation tools like to use it too.

Animation tool uses PIL for the same reason as map Window now - trim legend so that you can move it around the screen easily.

Can we close this?

[martinl]

Replying to annakrat:

Can we close this?

For me - yes.

[annakrat]

I also updated REQUIREMENTS.html and gui/wxPython/README (r58282).