Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit a54c2c7

Browse files
anntzeranntzer
committed
Cleanup cairo + gtk compatibility matrix.
1 parent 0d71eba commit a54c2c7

5 files changed

Lines changed: 58 additions & 50 deletions

File tree

‎doc/sphinxext/mock_gui_toolkits.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44

55
class MyCairoCffi(MagicMock):
6-
version_info = (1, 4, 0)
6+
pass
77

88

99
class MyPyQt4(MagicMock):

‎lib/matplotlib/backends/backend_cairo.py‎

Lines changed: 25 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,17 @@
2727
"is installed")
2828
else:
2929
HAS_CAIRO_CFFI = False
30+
if cairo.version_info < (1, 11, 0):
31+
# Introduced create_for_data for Py3.
32+
raise ImportError(
33+
"cairo {} is installed; cairo>=1.11.0 is required"
34+
.format(cairo.version))
3035
else:
3136
HAS_CAIRO_CFFI = True
3237

33-
if cairo.version_info < (1, 4, 0):
34-
raise ImportError("cairo {} is installed; "
35-
"cairo>=1.4.0 is required".format(cairo.version))
3638
backend_version = cairo.version
3739

38-
from matplotlib import cbook
40+
from .. import cbook
3941
from matplotlib.backend_bases import (
4042
_Backend, FigureCanvasBase, FigureManagerBase, GraphicsContextBase,
4143
RendererBase)
@@ -65,6 +67,23 @@ def _premultiplied_argb32_to_unmultiplied_rgba8888(buf):
6567
return rgba
6668

6769

70+
if HAS_CAIRO_CFFI:
71+
# Convert a pycairo context to a cairocffi one.
72+
def _to_context(ctx):
73+
if not isinstance(ctx, cairo.Context):
74+
ctx = cairo.Context._from_pointer(
75+
cairo.ffi.cast(
76+
'cairo_t **',
77+
id(ctx) + object.__basicsize__)[0],
78+
incref=True)
79+
return ctx
80+
else:
81+
# Pass-through a pycairo context.
82+
def _to_context(ctx):
83+
return ctx
84+
85+
86+
@cbook.deprecated("3.0")
6887
class ArrayWrapper:
6988
"""Thin wrapper around numpy ndarray to expose the interface
7089
expected by cairocffi. Basically replicates the
@@ -348,21 +367,9 @@ def draw_image(self, gc, x, y, im):
348367
im = im[:, :, (2, 1, 0, 3)]
349368
else:
350369
im = im[:, :, (3, 0, 1, 2)]
351-
if HAS_CAIRO_CFFI:
352-
# cairocffi tries to use the buffer_info from array.array
353-
# that we replicate in ArrayWrapper and alternatively falls back
354-
# on ctypes to get a pointer to the numpy array. This works
355-
# correctly on a numpy array in python3 but not 2.7. We replicate
356-
# the array.array functionality here to get cross version support.
357-
imbuffer = ArrayWrapper(im.ravel())
358-
else:
359-
# py2cairo uses PyObject_AsWriteBuffer to get a pointer to the
360-
# numpy array; this works correctly on a regular numpy array but
361-
# not on a memory view.
362-
imbuffer = im.ravel()
363370
surface = cairo.ImageSurface.create_for_data(
364-
imbuffer, cairo.FORMAT_ARGB32,
365-
im.shape[1], im.shape[0], im.shape[1]*4)
371+
im.ravel().data, cairo.FORMAT_ARGB32,
372+
im.shape[1], im.shape[0], im.shape[1] * 4)
366373
ctx = gc.ctx
367374
y = self.height - y - im.shape[0]
368375

‎lib/matplotlib/backends/backend_gtk3agg.py‎

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,21 @@
11
import numpy as np
22
import warnings
33

4-
from . import backend_agg, backend_gtk3
5-
from .backend_cairo import cairo, HAS_CAIRO_CFFI
4+
from . import backend_agg, backend_cairo, backend_gtk3
5+
from ._gtk3_compat import gi
6+
from .backend_cairo import cairo
67
from .backend_gtk3 import _BackendGTK3
78
from matplotlib import transforms
89

9-
if not HAS_CAIRO_CFFI:
10-
warnings.warn(
11-
"The Gtk3Agg backend is known to not work on Python 3.x with pycairo. "
12-
"Try installing cairocffi.")
10+
# The following combinations are allowed:
11+
# gi + pycairo
12+
# gi + cairocffi
13+
# pgi + cairocffi
14+
# (pgi doesn't work with pycairo)
15+
# We always try to import cairocffi first so if a check below fails it means
16+
# that cairocffi was unavailable to start with.
17+
if gi.__name__ == "pgi" and cairo.__name__ == "cairo":
18+
raise ImportError("pgi and pycairo are not compatible")
1319

1420

1521
class FigureCanvasGTK3Agg(backend_gtk3.FigureCanvasGTK3,
@@ -36,11 +42,7 @@ def on_draw_event(self, widget, ctx):
3642
else:
3743
bbox_queue = self._bbox_queue
3844

39-
if HAS_CAIRO_CFFI and not isinstance(ctx, cairo.Context):
40-
ctx = cairo.Context._from_pointer(
41-
cairo.ffi.cast('cairo_t **',
42-
id(ctx) + object.__basicsize__)[0],
43-
incref=True)
45+
ctx = backend_cairo._to_context(ctx)
4446

4547
for bbox in bbox_queue:
4648
area = self.copy_from_bbox(bbox)
@@ -51,12 +53,9 @@ def on_draw_event(self, widget, ctx):
5153
width = int(bbox.x1) - int(bbox.x0)
5254
height = int(bbox.y1) - int(bbox.y0)
5355

54-
if HAS_CAIRO_CFFI:
55-
image = cairo.ImageSurface.create_for_data(
56-
buf.data, cairo.FORMAT_ARGB32, width, height)
57-
else:
58-
image = cairo.ImageSurface.create_for_data(
59-
buf, cairo.FORMAT_ARGB32, width, height)
56+
image = cairo.ImageSurface.create_for_data(
57+
buf.ravel().data, cairo.FORMAT_ARGB32,
58+
width, height, width * 4)
6059
ctx.set_source_surface(image, x, y)
6160
ctx.paint()
6261

‎lib/matplotlib/backends/backend_gtk3cairo.py‎

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,24 @@
11
from . import backend_cairo, backend_gtk3
2-
from .backend_cairo import cairo, HAS_CAIRO_CFFI
2+
from ._gtk3_compat import gi
3+
from .backend_cairo import cairo
34
from .backend_gtk3 import _BackendGTK3
45
from matplotlib.backend_bases import cursors
56

67

8+
# The following combinations are allowed:
9+
# gi + pycairo
10+
# gi + cairocffi
11+
# pgi + cairocffi
12+
# (pgi doesn't work with pycairo)
13+
# We always try to import cairocffi first so if a check below fails it means
14+
# that cairocffi was unavailable to start with.
15+
if gi.__name__ == "pgi" and cairo.__name__ == "cairo":
16+
raise ImportError("pgi and pycairo are not compatible")
17+
18+
719
class RendererGTK3Cairo(backend_cairo.RendererCairo):
820
def set_context(self, ctx):
9-
if HAS_CAIRO_CFFI and not isinstance(ctx, cairo.Context):
10-
ctx = cairo.Context._from_pointer(
11-
cairo.ffi.cast(
12-
'cairo_t **',
13-
id(ctx) + object.__basicsize__)[0],
14-
incref=True)
15-
16-
self.gc.ctx = ctx
21+
self.gc.ctx = backend_cairo._to_context(ctx)
1722

1823

1924
class FigureCanvasGTK3Cairo(backend_gtk3.FigureCanvasGTK3,

‎tutorials/introductory/usage.py‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -481,11 +481,8 @@ def my_plotter(ax, data1, data2, param_dict):
481481
# GTK and Cairo
482482
# -------------
483483
#
484-
# Both `GTK2` and `GTK3` have implicit dependencies on PyCairo regardless of the
485-
# specific Matplotlib backend used. Unfortunately the latest release of PyCairo
486-
# for Python3 does not implement the Python wrappers needed for the `GTK3Agg`
487-
# backend. `Cairocffi` can be used as a replacement which implements the correct
488-
# wrapper.
484+
# `GTK3` backends (*both* `GTK3Agg` and `GTK3Cairo`) depend on Cairo
485+
# (pycairo>=1.11.0 or cairocffi).
489486
#
490487
# How do I select PyQt4 or PySide?
491488
# --------------------------------

0 commit comments

Comments
 (0)