1784
86%
master: 86%

Ran 27 Dec 2019 07:32PM UTC

Jobs 4

Files 165

Run time 11min

Badge

Embed ▾

pending completion

Build # 1784

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

14193 of 15894 relevant lines covered (89.3%)

5.04 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	1784.1 (GEO_TYPES_FEATURES="")	27 Dec 2019 07:32PM UTC	89.3	Travis Job 1784.1
2	1784.2 (GEO_TYPES_FEATURES="--features serde")	27 Dec 2019 07:32PM UTC	89.3	Travis Job 1784.2
3	1784.3 (GEO_TYPES_FEATURES="--features rstar")	27 Dec 2019 07:34PM UTC	89.3	Travis Job 1784.3
4	1784.4 (GEO_FEATURES="")	27 Dec 2019 07:43PM UTC	89.3	Travis Job 1784.4

Source Files on build 1784

Detailed source file information is not available for this build.

georust / geo / 1784
86%
master: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1784

georust / geo / 1784 86% master: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1784

georust / geo / 1784
86%
master: 86%

README BADGES
x