CesiumGS / CesiumGS/cesium

Remove scene.groundPrimitives

Open
#7,639 1 comment 0 reactions 0 assignees View on GitHub
type - cleanup
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

I don't think there's a technical reason for `scene.primitives` and `scene.groundPrimitives` to be different collections anymore. It might just be legacy from when the classification pass didn't exist.

Contributor guide

Open the contributing guide

Research direction

Start by tracing the definitions and references to scene.primitives and scene.groundPrimitives, then inspect how the classification pass uses them. Done means removing the separate groundPrimitives collection without breaking classification behavior and updating affected tests or callers.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
computer-graphics
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.