CustomSensorCone time-varying outer half angle issue

CesiumJS
1

Try plugging in the CZML below into the Cesium Viewer (sorry, my proxy won’t let me attach files.) I’m doing something similar on my project and every machine I’ve tested it on, I get a flickering effect at one of the clock angle slices. I’m assuming it’s either the first or last slice. Does anyone else see this issue? It goes away if I set the outerHalfAngle property to a single value instead of being time-varying.

{

“id”:“cone”,

“availability”:“2012-05-22T19:22:02Z/2012-05-22T19:24:02Z”,

“cone”:{

“minimumClockAngle”:0.0,

“maximumClockAngle”:6.283185307179586,

“show”:true,

“showIntersection”:true,

“innerHalfAngle”:0.0,

“outerHalfAngle”:{

“epoch”:“2012-05-22T19:22:02Z”,

“number”:[

0.0,0.09034602912017678,

60.0,0.09034819716130152,

120.0,0.0903503813944031

]

},

“radius”:6.263947203246858E7

},

“position”:{

“cartesian”:[

-1592611.894963102,4.215996588670453E7,-16865.102873710017

]

},

“orientation”:{

“unitQuaternion”:[

-0.682951222618297,0.07886654195625965,0.0833065496966768,0.7213998300953987

]

}

}

2

Hey Eric,

I'll definitely check this out today, if it's a CZML processing issue,
it should be an easy fix.

Matt

3

Eric,

I can reproduce this easily with your czml, thanks. I see what the
problem is (it's not always rendering the last triangle), but I don't
know why.

The good news is that I have a workaround in the mean time.

Change line 33 of DynamicConeVisualizerUsingCustomSensor.js from

        var angleStep = CesiumMath.toRadians(2.0);
to
        var angleStep = CesiumMath.toRadians(3.0);

And the problem goes away. This leads me to believe it is some sort
of precision or sampling error, but I can't be for sure. I opened a
new GitHub issue to keep track of it:

4

Thanks for looking into that so quickly Matt, I’ll plug that in. Just in time for the demo on Monday :slight_smile:

5

Eric, just an FYI that this has been fixed in pull request 152. It
should be in master by Monday, if not sooner.

https://github.com/AnalyticalGraphicsInc/cesium/pull/152

6

Kevin’s fix is in master now.

Patrick