Cesium sensors broken in 1.9?

CesiumJS
1

Can anyone else confirm if there is a compatibility issue between Cesium 1.9 and the latest available cesium-sensors? In particular, I am attempting to use RectangularSensor, which was working prior to upgrading to 1.9. Now, I get the following error:

TypeError: Cannot read property ‘length’ on undefined

at RectangularSensorVisualizer._onCollectionChanged

at new RectangularSensorVisualizer

at DataSourceDisplay.defaultVisualizersCallback

…

It appears as though _onCollectionChanged is called with “added” undefined. Did something change in Cesium 1.9 that would cause this issue?

Thanks

2

Hello I made a pull request about this issue : https://github.com/AnalyticalGraphicsInc/cesium-sensors/pull/5

3

Great, thanks. I hadn’t made the connection that this was related to the deprecation of entities.entities.

Is someone handling pull requests for this repository still, or does it need to be forked?

4

I am using the last version of the plugin downloaded from github,
and according to the pull request I changed the line in the file Source/RectangularSensorVisualizer.js from entities to values.

I am still getting the same error.
Are there more files that need to be changed?

Thank you.

5

You need to change the line and also rebuild it so that the change gets propagated to the combined CesiumSensors.js file. Look at the forks of the repo. There may be versions where this was already done.

6

Thank you very much,
I have found the repository's fork -
https://github.com/kaktus40/cesium-sensors/tree/updateToCesium1.9

But I am still receiving the same error.
I think I am missing another step that should be done.
Thank you.

7

That fork updated the source file but still needs to be built before using. In other words, the file under the Build/ directory wasn’t updated. You’ll either need to build it yourself or try using the version here: https://github.com/mgoldgeier/cesium-sensors

8

Hello I made an update of sensors plugin (https://github.com/kaktus40/cesium-sensors) in order to be compliant with cesium 1.14