MNT: remove FeatureArtist.draw args and kwargs

[rcomer]

Rationale

I was going to propose deprecating these so I added some deprecation warnings. However, when I tried to trigger those warnings I found that you just can't pass extra parameters here. The method has Matplotlib's allow_rasterization decorator which has not passed args and kwargs since matplotlib/matplotlib#20331. Since Matplotlib v3.5 is now our minimum supported version, there is no way these can be used any more. So I propose to just remove them. I had hoped this would lead to simpler code but, unless I'm missing a trick, it doesn't make much difference.

Implications

This removes our internal use of the style module so we could consider deprecating that if we wanted.

[greglucas]

I think this makes sense, nice job tracking that down.

One thought for simplification is whether we need to track both stylized and un-stylized paths? Could we instead add an empty style dictionary if there is no style applied? Then we wouldn't need to keep track of both paths.

If we aren't using the style module ourselves it seems like it would be worth deprecating that too.

[greglucas]

Do we need OrderedDict anymore? The versions of Python we support are now insertion order guaranteed.

[rcomer]

Could we instead add an empty style dictionary if there is no style applied?

Done. I had already been in two minds about whether to do that as it makes the no-styler case itself more complicated but as you say it reduces paths through the code.