As I see it, there are three options to resolve this:

Do not support backends that don't have the __init__(self, figure) signature.
Change, for example, the wxagg backend to pass defaults for its parent and id keyword arguments, thus bringing its call signature inline with __init__(self, figure).
Change the call signature of Figure() to pass id and parent to the backend.

Implementing 1): This is the easiest, but users of these backends do not benefit from the reduced boilerplate code. Having said that, the reduced boilerplate code was designed to achieve two things: a) to make it more convenient to set up figures in an object-oriented fashion; and b) to make the backend as transparent as possible to the user.

Implementing 2): This is the hardest, and is a backwards-incompatible change if not done properly. Also, I am not sure (and anybody that uses the wxagg backend please chime in here) it is possible to know what 'default' id and 'default' parent are. Before #1125, was there some other wx-related widget set up to create a parent and an id?

Implementing 3): This makes the figure less backend-agnostic. Though, it would still allow for less boilerplate than 1). A downside here is the possibility of littering the front-end Figure() method with backend-specific parameters as more and more backends are developed. Having said that, this could be kept to a minimum by letting Figure() take only only one kwarg, backend_kwargs, say. This would be a dictionary of backend-specific parameters (id and parent in the case of wxagg) that are passed to the relevant canvas object. Another downside of this is that the Figure class would become less backend-agnostic, what with having to pass on backend-specific parameters.

Ideally, I'd like every user to benefit from reduced boilerplate code, and option 3) achieves this. However, it might be an idea to implement 1) and perhaps discuss further and what needs to happen to support all backends more cleverly. Perhaps an MEP is needed to make the interface for the backends uniform.

Uh oh!

Uh oh!

backend wx and wxagg broken by #1125 #1218

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions