Drillpick broken in Cesium 14?

CesiumJS
1

I download the newest Cesium and when I try to perform a drillpick I get an error saying it cannot compute ".pass" from undefined.

I think it may be the line 1084 in scene.js:

        var length = commandList.length;
        for (var i = 0; i < length; ++i) {
            var command = commandList[i];
            var pass = command.pass;

            if (pass === Pass.COMPUTE) {
                computeList.push(command);

It should have a check to make sure command is defined:

        var length = commandList.length;
        for (var i = 0; i < length; ++i) {
            var command = commandList[i];
            if (command) {
                var pass = command.pass;

                if (pass === Pass.COMPUTE) {
                    computeList.push(command);

The code used to have the check in previous versions then the check was removed. Was it removed for a reason?

- V

2

I don’t believe command should ever be undefined, which is why the check was removed.

Does the Drill Down picking example in Sandcastle work for you?

http://cesiumjs.org/Cesium/Apps/Sandcastle/index.html?src=Picking.html&label=Showcases

If it does, can you post a small Sandcastle snippet that triggers the problem on your end? Perhaps it’s only a problem in certain situations or with certain primitive types.