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

Coordinates should accept list or tuple #2

Open
Taxuspt opened this issue Sep 12, 2017 · 1 comment
Open

Coordinates should accept list or tuple #2

Taxuspt opened this issue Sep 12, 2017 · 1 comment

Comments

@Taxuspt
Copy link

Taxuspt commented Sep 12, 2017

The asserts on lines 30 and 63, i.e.
assert isinstance(coordinates, list)
should also accept tuples, i.e.
assert isinstance(coordinates, (list,tuple))

Since tuples are a valid format on GeoJson and some libraries (e.g. shapely) map shapes to tuples instead of lists.

@connor
Copy link

connor commented Jan 27, 2018

Added in #5 (though not yet merged).

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

No branches or pull requests

2 participants