Rework @turf/dissolve & new modules proposal
Open
@DenisCarriere is already working on this.
Since Oct 17, 2017.
new-module
refactoring
- Dominant language
- TypeScript
- Stars
- 10.5k
- Forks
- 1k
- Avg merge
- 1h 11m
- Merged PRs (30d)
- 4
Description
Rework module @turf/dissolve
New module proposal @turf/line-dissolve
New module proposal @turf/polygon-dissolve
New module proposal @turf/homogenous
This proposal is be aimed for post
v5.0release.
Been looking at converting geojson-dissolve to ES modules and diving into the source code and many things could be dropped for performance and geojson-linestring-dissolve can be converted to an internal TurfJS module.
Dependencies
-
topojson-server(ES module compatible) -
topojson-client(ES module compatible) -
geojson-linestring-dissolve(should be internalized to TurfJS => zero external dependencies required) -
geojson-flatten(changed to@turf/metaflattenEach()which will boost performance on large dataset).
@turf/homogenous
Or could be built-in to
@turf/invariantor@turf/helpersashomogenous()
Returns a string Geometry type, if multiple or no geometry types are found then it will return null.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.