Hi everyone,

I was just wondering whether someone would be willing to have a look at this PR? I feel like it's in a complete or nearly-complete state but need some guidance - the two failing CI jobs seem to be:

• PR cleanliness:
  The added unit test was written first by@Abitamim, and then expanded by myself. When I expanded the test to check the newly-added reversed arrow heads, I changed both the test name (temp_test_boxarrow →test_boxarrow_adjustment) and the name of the output image (roadsign_test_image.png →boxarrow_adjustment_test_image.png) to reflect this. The 'file both added and deleted' error given seems to be a result of adding then deletingroadsign_test_image.png.
• Python 3.11 on macos-13:
  The failing test here (test_blitting_events) is marked as flaky, with the comment# subprocesses can struggle to get the display, so rerun a few times. I feel it may be that this was just a random instance of the test failing multiple times in a row; unfortunately I don't have access tomacos-supported hardware for rerunning this locally, so I was wondering if someone would be able to take a look.

Any help would be greatly appreciated. Thanks!

PR-cleanliness: This is a check so that we don't merge commits with unneeded images, which would increase repo size. You can either squash and force-push. Or ignore this and we'll squash-merge at the end.

Ignore the macos test failure. It's unrelated.

Thank you for getting back so quickly! Would it be okay if you handled the squashing - I'm relatively new to git, and I'm not sure of the best method to use for squashing to keep a relatively clean but traceable commit history?

Since I'm going to need to make some relatively major changes to the contents of this PR (a redo of the arrow drawing code and a rewrite of its unit test), would it be better for me to amend my last commit (d9b0b32) as suggested in thecontributing guide, or make a few new commits? I'm not quite sure how to add more commits to an open PR - do I just push the changes to my development branch?

@CharlieThornton33 yes you can just push more commits to your current branch and they will appear in this PR. Since you already have several commits they will ultimately need squashing one way or the other (and I see you already asked that we handle that), so I do not think you gain anything from the—amend option.

I've just redone the padding (only forLArrow andRArrow for the moment), and I think the issue with the text escaping the outline has been fixed. I set the arrow body to dynamically increase in length to make sure there's always a gap of at leastpad between the edge of the text closest to the arrow head and the position where that edge would first intercept the outline if moved towards the tip:

import numpy as npimport matplotlib.pyplot as pltfig, ax = plt.subplots(figsize=(3, 8))t0 = ax.text(0.5, 0.9, "Text",            ha="center", va="center", rotation=0, size=30,            bbox=dict(boxstyle="rarrow,pad=0.3,head_width=1.5,head_angle=-90",                      fc="lightblue", ec="steelblue", lw=2))t1 = ax.text(0.5, 0.7, "Text",            ha="center", va="center", rotation=0, size=30,            bbox=dict(boxstyle="rarrow,pad=0.3,head_width=1.5,head_angle=70",                      fc="lightblue", ec="steelblue", lw=2))t2 = ax.text(0.5, 0.5, "Text",            ha="center", va="center", rotation=0, size=30,            bbox=dict(boxstyle="rarrow,pad=0.3,head_width=1.5,head_angle=100",                      fc="lightblue", ec="steelblue", lw=2))t3 = ax.text(0.5, 0.3, "Text",            ha="center", va="center", rotation=0, size=30,            bbox=dict(boxstyle="rarrow,pad=0.3,head_width=1.5,head_angle=130",                      fc="lightblue", ec="steelblue", lw=2))t4 = ax.text(0.5, 0.1, "Text",            ha="center", va="center", rotation=0, size=30,            bbox=dict(boxstyle="rarrow,pad=0.3,head_width=1.5,head_angle=170",                      fc="lightblue", ec="steelblue", lw=2))plt.show()

It's also a bit concerning that the CI workflow seems to run on every commit to this PR - am I okay to disable tests in my commit messages until the one where I write the new test to save on CI resources?

I've just noticed - all of thepytest test runs are about to fail because I forgot to remove the flawedtest_boxarrow_adjustment test. Is there any way to cancel a CI test run?

@CharlieThornton33 I wouldn't worry about the CI failing, but if they are still running when you push another commit they will be interrupted and re-started.

Hi,

I'm a bit stuck with how to handle cases where an arrow head extends behind the opposite end of the body; something like

I'm not sure whether the arrow head should 'stick out' through the other end of the body, or should cut off at that point.

I'd really appreciate any advice - I don't know if there's any sort of precedent in Matplotlib for this kind of thing.

I don't think there's precedent. While such cases should be rare, my gut feeling is that the tip should not stick out.