drawing with fill or stroke=None produces black fill/stroke

[celiagg]

drawing with fill or stroke=None produces black fill/stroke

[jwiggins]

Thanks for the report! It's a good question whether this is a bug or a feature. It's certainly confusing, so I would lean towards bug. Looking at the C++ code clarified things a bit for me.

Basically, passing None is not the way to achieve the result you want. Rather, you should set state.drawing_mode = celiagg.DrawingMode.DrawFill or state.drawing_mode = celiagg.DrawingMode.DrawStroke. It just so happens that DrawingMode.DrawFillStroke is the default, and passing None for fill or stroke results in celiagg.SolidPaint(0,0,0) being passed through to the underlying C++ code.

Do you think a descriptive exception would help here if the drawing mode is "incompatible" with the given stroke/fill paint?

[zpincus]

Alternately, perhaps the default could be celiagg.SolidPaint(0,0,0) instead of None? That might clarify thing. (An addition to the docstring mentioning the use of DrawingMode to disable fill / stroke would also be helpful.)

Or is there a reason that None is necessary instead of a specific paint default?

[jwiggins]

I've updated the documentation. The docstrings now reflect better what is happening internally.

[zpincus]

Looks very clear now! Thanks :)