Repository navigation
Conversation
f31e844 to
969828e
Compare
QuLogic
left a comment
There was a problem hiding this comment.
I'm not sure what the original goal of #1460 was, but add_subplot accepts keyword arguments and forwards them to the Axes constructor. This means that subplot_kw can accept many more keyword arguments than the 3 listed here. Maybe that's not needed, but it should be well documented.
6710f67 to
402476a
Compare
|
Wow, that is an old issue! I share @QuLogic 's concerns about hard-coding a sub-set of kwargs to be hoisted up to This also has the logical extension that we should route some keys to In short, I am worried that half measures will make things more confusing and inconsistent and the full solution may grow too complex. |
|
I checked |
|
@sabulikia That is a reasonable summary. On one hand The fundamental issue is we have two things that consume all of the "extra" keyword arguments and are user-plug able. I do not think there is a more fundamental solution possible here, but I would be happy to be proved wrong! It seems simpler to say "input to subplots goes here and input to |
Fixes #1460
PR Summary
pyplot.subplots()now accepts subplot arguments through **kwargs without the need of using dictionaries.PR Checklist