James Gaboardi
James Gaboardi
@gegen07 This is absolutely worth a discussion IMHO.
@gegen07 It might be helpful to open a dedicated [Discussion](https://github.com/pysal/libpysal/discussions) for this.
~~This seems to be happening in `momepy` actually so I'll transfer this issue over to there.~~ I have no idea why I thought this was opened in `tobler`... LOL
~~Seems I don't have the proper permissions for the transfer.~~ Couldn't transfer because it's already here...
Yes, more checks & informative messages are always great ideas. @ale-v -- Any chance you can tease out the subset of offending data? We can use that in a test.
Boiled down example this is ***not*** reproducible: ```python Python 3.12.7 | packaged by conda-forge | (main, Oct 4 2024, 15:57:01) [Clang 17.0.6 ] Type 'copyright', 'credits' or 'license' for more...
Ideas about what to do different here to reproduce?
Good call. The following reproduces the error: ```python In [1]: import momepy, numpy, shapely ...: ...: line_1 = shapely.LineString((shapely.Point(1, 1), shapely.Point(2, 2))) ...: line_2 = shapely.LineString((shapely.Point(2, 2), shapely.Point(3, 3))) ...:...
> You probably had mixed LineString and LineString Z geometries after the concatenation. > > I am curious if we should do some check internally here to avoid these cryptic...