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.

Sign up for GitHub

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

Jump to bottom

Change for a more strict geojson handling. #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Change for a more strict geojson handling. #47

wants to merge 2 commits into from

Commits on Nov 9, 2020

  1. Change for a more strict geojson handling. 

    - Do not handle m coordinate.
- Raise for unknown geojson `type`.
- Allow less strict polygons (not simple). Which is ok per spec.
- Add various tests.
    @BuonOmo
    BuonOmo committed Nov 9, 2020
    Copy the full SHA
    fe72a9a View commit details
    Browse the repository at this point in the history

  2. Enforce right-hand rule when encoding 

    Before (no right-hand rule)
```
Warming up --------------------------------------
        big geometry    32.000  i/100ms
      small geometry    10.379k i/100ms
Calculating -------------------------------------
        big geometry    343.166  (± 2.6%) i/s -      1.728k in   5.039023s
      small geometry    106.304k (± 2.1%) i/s -    539.708k in   5.079398s
```

After (right-hand rule)

```
Warming up --------------------------------------
        big geometry     1.000  i/100ms
      small geometry     2.780k i/100ms
Calculating -------------------------------------
        big geometry      9.085  (± 0.0%) i/s -     46.000  in   5.069703s
      small geometry     28.541k (± 2.9%) i/s -    144.560k in   5.069153s
```
    @BuonOmo
    BuonOmo committed Nov 9, 2020
    Copy the full SHA
    b9bc47a View commit details
    Browse the repository at this point in the history