1
86%
master: 86%

Ran 27 Dec 2019 07:32PM UTC

Files 55

Run time 5s

Badge

Embed ▾

Committed 27 Dec 2019 06:58PM UTC coverage: 89.298% (-0.2%) from 89.472%

Job # GEO_TYPES_FEATURES=""

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Merge #402

402: Simplify indices r=urschrei a=urschrei

This PR adds a new trait to the `simplify` and `simplifyVW` modules: `SimplifyIdx` and `SimplifyVwIdx`.
Both traits do the same thing: they run the respective simplification algorithm, but return the _indices_ of the simplified `LineString`, instead of the simplified geometry. This is mostly in response to ongoing requests from FFI consumers: because their use-cases don't necessarily map onto the Simple Features geometry model, returning geometries is less useful than returning indices, which can be used as mask values / input into native filter functions.

The only current drawback to these new traits is that because they return `Vec<usize>`, they're only implemented for `LineString`. If there's a more flexible way to do it, I'd love to extend it to some other geometries.

This change introduces some additional indirection for the RDP implementation, but `cargo bench` hasn't shown a perf hit…

Co-authored-by: Stephan Hügel <shugel@tcd.ie>
Co-authored-by: Stephan Hügel <urschrei@gmail.com>

Run Details

4731 of 5298 relevant lines covered (89.3%)

1.26 hits per line

georust / geo / 1784 / 1
86%
master: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1784.1 (GEO_TYPES_FEATURES="")

georust / geo / 1784 / 1 86% master: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1784.1 (GEO_TYPES_FEATURES="")

georust / geo / 1784 / 1
86%
master: 86%

README BADGES
x