CesiumGS / CesiumGS/cesium

EllipseGeometryUpdater crashes when positions is defined but positions.getValue(time) is undefined

Open
#3,796 0 comments 0 reactions 0 assignees View on GitHub
category - graphics type - bug
Dominant language
JavaScript
Stars
15.7k
Forks
3.9k
Avg merge
4d 6h
Merged PRs (30d)
34

Description

reported on the forum: https://groups.google.com/forum/?hl=en#!topic/cesium-dev/6qQ9NzKNzWk

A `DeveloperError` is being thrown from `EllipseGeometry` because the center position is undefined

``` javascript
var viewer = new Cesium.Viewer('cesiumContainer');

viewer.entities.add({
position : new Cesium.SampledPositionProperty(),
ellipse : {
semiMinorAxis : 3000.0,
semiMajorAxis : 3000.0,
material : Cesium.Color.GREEN
}
});
```

Contributor guide

Open the contributing guide

Research direction

Start by locating EllipseGeometryUpdater and reproduce the provided entity example with a SampledPositionProperty whose getValue(time) is undefined. Trace how the undefined center reaches EllipseGeometry, then verify that the example no longer throws a DeveloperError while valid ellipse positions continue to work.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
computer-graphics
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.