Greg Lucas

I am guessing that it is just that no one has taken the time to turn the readers into a context manager. PRs are welcome if you'd like to propose...

Currently the gridlines are removed before recreating them, so there is a lot of thrashing going on for you with draw() / remove() both trying to continuously update. https://github.com/SciTools/cartopy/blob/60c8c20b4f1c481468acc6058299fba440020ed0/lib/cartopy/mpl/gridliner.py#L627-L639 It...

🤔 I wonder if Fiona is causing issues? 1.9.6 was released March 8, which is around when we started seeing the flaky issues. https://github.com/Toblerity/Fiona/releases

Closing as this doesn't seem to be the problem.

@dsheward-astro, would you be able to add a quick test for this? Let me know if you want any pointers for where to add it. I also see that you...

@raphaelquast friendly ping in case you had a chance to look into `to_polygons()` and had any thoughts on the other comments.

It looks like the path is allowed to simplify still as it enters that method. If you put in `path.should_simplify = False`, then I think it might work as expected.

> My final question now is if we should "reset" the should_simplify property to its initial value after conversion to avoid changing the properties of the path instance, e.g. something...

We just keep going deeper here :( It looks like the path is cleaned to remove duplicates with the `to_polygons` which must push some of the image checks just over...

> Should we apply the same treatment also to the _ViewClippedPathPatch? (E.g. putting _ensure_path_closed outside the class and then using it for both the spine and the patch) That also...