CesiumGS / CesiumGS/cesium-native

Implicit Tiling: Consolidate redundancies in availability calculations

Open
#427 0 comments 0 reactions 0 assignees View on GitHub
quality
Dominant language
C++
Stars
623
Forks
277
PR merge metrics
No merged PRs in 30d

Description

A quick glance at [`QuadtreeAvailability.cpp`](https://github.com/CesiumGS/cesium-native/blob/de933c29dd36d9582059239fb166af666cf14b41/CesiumGeometry/src/QuadtreeAvailability.cpp) and [`OctreeAvailability.cpp`](https://github.com/CesiumGS/cesium-native/blob/de933c29dd36d9582059239fb166af666cf14b41/CesiumGeometry/src/OctreeAvailability.cpp) will show a bunch of redundant code that could be extracted out given some untangling.

@javagl started to do this as mentioned in this comment: https://github.com/CesiumGS/cesium-native/pull/420#issuecomment-1018622521

We should continue his work there, doing something similar in `OctreeAvailability.cpp` and potentially consolidating commonalities between those two files as well.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.