Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SegFault for initializing cartopy crs for Southern Hemisphere LambertAzimuthalEqualArea #1099

Closed
mjbrodzik opened this issue Jul 31, 2018 · 1 comment

Comments

@mjbrodzik
Copy link

Description

Hello, I am trying to initialize a LambertAzimuthalEqualArea centered at the South Pole, but this generates a SegFault and crashes python.

I get this error on Mac OS X (High Sierra, 10.13.5) and Mac OS X (Sierra, 10.12.6).

I believe this may be only be a problem for the Southern hemisphere exactly at the South Pole, because it works correctly if I set it to the North Pole (central_latitude=90.0), and does not seg fault of I try other latitudes in the Southern Hemisphere besides the Pole.

Code to reproduce

(cartopy) $ python
Python 3.6.5 | packaged by conda-forge | (default, Apr  6 2018, 13:44:09) 
[GCC 4.2.1 Compatible Apple LLVM 6.1.0 (clang-602.0.53)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import cartopy.crs as ccrs
>>> e2s = ccrs.LambertAzimuthalEqualArea(central_latitude=-90.0)
IllegalArgumentException: Points of LinearRing do not form a closed linestring
Segmentation fault: 11
$ 
Full environment definition

Operating system

Mac OS X

Cartopy version

cartopy 0.16.0 py36_0 conda-forge

conda list

I am attaching output from my conda list command.  I've made a fresh conda env using the environment recommended at the recent SciPy2018 tutorial.
[cartopy_env.txt](https://github.com/SciTools/cartopy/files/2246495/cartopy_env.txt)


@ajdawson
Copy link
Member

ajdawson commented Aug 1, 2018

Thanks for the report @mjbrodzik, this looks like a problem in the projection class when it defines the boundary, a bug fix will be necessary (see #1100).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants