You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Remove extra arguments from WCSAxesdraw() methods #12672
In Matplotlib every draw() method has a two arguments, self and renderer. In WCSAxes this is not the case however, and there are frequently other arguments to draw(). This causes at least a couple of issues when interacting with Matplotlib, most recently #12568. It also reduces legibility of the code in my opinion - drawing an object should only rely on the state of that object at draw time, not any extra parameters.
I am therefore in the process of refactoring WCSAxes to remove extra arguments to the draw() methods. This is going to require quite a bit of surgery, so I'm opening this issue to track the progress I make and PRs I open.
Activity
dstansby commentedon Jan 10, 2024
All done 👍