Ruth Comer comments

Results 454 comments of


                                            Ruth Comer

Introduce `path_to_shapely` and `shapely_to_path`

I also tested this with my user code that generates many contour plots. I got identical results with my branch and with `main`.

Introduce `path_to_shapely` and `shapely_to_path`

Broke the minimum versions tests 😢

Introduce `path_to_shapely` and `shapely_to_path`

Most of the tests were fixed by bumping minimum Shapely to v2.0. I think that’s reasonable to do since v2 will be 2 years old in December. However, I think...

[Bug]: Calling axes.clear() on a shared y or x-axis issues an unexpected warning with log scale

Maybe caused by #27825

DOC: clarify that secondary_[xy]axis returns an Axes subclass

Fair enough. I don't really have bandwidth to propose a better change right now, so let's just close this.

Make `geos_to_path` return one path per geometry

[Progress so far](https://github.com/SciTools/cartopy/compare/main...rcomer:cartopy:geos-to-path-return-path) which I did at the time of #2325 and just now rebased.

Make `geos_to_path` return one path per geometry

I now have the tests passing in my branch :tada: However, this function is public and a quick search of GitHub shows projects are using it. In all the examples...

Make `geos_to_path` return one path per geometry

A disadvantage of making a new function is that we would lose the naming symmetry with `path_to_geos` 🤔

Make `geos_to_path` return one path per geometry

Good point. `path_to_geos` also always returns a list. The list may contain * one or more `Polygon`s (which were closed paths) or * a single `Linestring` or `MultiLinestring` (which were...

Make `geos_to_path` return one path per geometry

Yeah, I had a look at their plotting module (which is marked experimental). AFAICS they don't have anything that returns the path here - just makes a `PathPatch` and plots...