Sandcastle Polyline example throw exception in Columbus view

[pjcozzi]

@bagnell this is all you...

Open http://localhost:8080/Apps/Sandcastle/index.html?src=Polylines.html

Switch to Columbus view. You'll get

Uncaught TypeError: Cannot read property 'longitude' of undefined

The polyline is passing Cartesian points to a function that wants cartographic. An an aside, we can probably change this function to work in Cartesian space, like I started to do for polygons that cross the IDL.

We should address this soon, but it will not hold up b13 since it is already in b12a (and I suspect others).

[mramato]

I disagree. We shouldn't ship a release with known broken Sandcastle examples.

[pjcozzi]

We already did - several times - perhaps even including releases you did. I just tested carefully enough to find it - others did not. No one reported it in the wild; it is minor. We are releasing.

[mramato]

Just because we already did something doesn't mean it's necessarily okay to do it again. But you're the release-meister this month, so it's your judgment call.

[emackey]

@bagnell Beware of #497 which has a lot of Sandcastle changes coming in.

[bagnell]

This is fixed in the above pull. Closing.