Add optimized bboxFill function
Open
@nrabinowitz is already working on this.
Since Dec 21, 2018.
enhancement
- Dominant language
- C
- Stars
- 6.5k
- Forks
- 627
- Avg merge
- 3d 21h
- Merged PRs (30d)
- 6
Description
Filling a bounding box with hexagons currently requires polyfill, which does an additional point-in-poly check that shouldn't be necessary for bounding boxes. If the use case is common enough, we ought to be able to provide a bboxFill function that would be faster for just filling bboxes. The case that's come up a few times here is getting all of the H3 indexes within a map viewport.
Additionally, bboxFill could handle bounding boxes wider than 180 degrees longitude, a current restriction of polyfill, because we'd have an explicit east/west signal.
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.