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

Do not add a feature if the polygon is empty. Do not an empty polygon… #20

Merged
merged 1 commit into from
Nov 5, 2020

Conversation

nems808
Copy link
Contributor

@nems808 nems808 commented Nov 3, 2020

… to coordinates collection

Sometimes contourf returns an empty polygon, if this is the case, the feature should not be added. Also, sometimes the polygon starts with an empty coordinates (path). If this is the case, the isobands won’t display in Leaflet

… to coordinates collection

Sometimes contourf returns an empty polygon, if this is the case, the feature should not be added. Also, sometimes the polygon starts with an empty coordinates (path). If this is the case, the isobands won’t display in Leaflet
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 68.605% when pulling 92bbf36 on uhsealevelcenter:master into 26b7301 on bartromgens:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 68.605% when pulling 92bbf36 on uhsealevelcenter:master into 26b7301 on bartromgens:master.

@bartromgens bartromgens merged commit 1e3a7f3 into bartromgens:master Nov 5, 2020
@bartromgens
Copy link
Owner

👍 Thanks!

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

Successfully merging this pull request may close these issues.

3 participants