Repository navigation
Conversation
There was a problem hiding this comment.
this probably should be _extend_axes_pad
|
Sorry if I am stepping on any toes by commenting on this code. This is similar in spirit to #2276 where I do something very similar (change a scalar parameter -> n-D) where I do it in the way I mention in the code comment (by adding extra scalar parameters). |
|
@leejjoon - pinging you on this one. I'm in favour of this change, but I haven't read it completely, and I suspect you should actually press the merge button once your happy with it. Cheers, |
There was a problem hiding this comment.
_init_axes_pad creates new Size.Fixed instances while set_axes_pad should not. Although I have not tested, I guess calling "set_axes_pad" will have no effect with this change.
There was a problem hiding this comment.
Looking at this again, I no longer understand my comment as my claim that they are identical is clearly wrong.
There was a problem hiding this comment.
I believe it is fixed now
|
I have used this feature successfully when writing a scientific paper, see Fig. 6 |
|
@matejak This will need a re-base as it no longer merges cleanly. |
|
I have not followed the advice on the web when making the fork and the pull request (I was not aware of those at that time), so I will drop this request and create a new one. Sorry about that. |
|
@matejak What is the state of this? |
|
The state is good, there is another pull request #2731 |
This feature is useful in cases such as showing the same section of an image with different colorbar ranges.
Since colorbars have to be labelled, it is not possible to get things right when using the same horizontal and vertical spacing.
I attach an image (you can obtain it by running the demo script or by generating the documentation, the new functionality is the image to the right).
I think that I haven't broken anything since the documentation generation did not complain. However, I don't know how to write tests for this feature.