Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Polygon lighting issue #2399

Open
GatorScott opened this issue Jan 14, 2015 · 7 comments
Open

Polygon lighting issue #2399

GatorScott opened this issue Jan 14, 2015 · 7 comments

Comments

@GatorScott
Copy link

image

[
    {
        "id": "document",
        "name": "TopicID=33",
        "version": "1.0"
    },
    {
        "id": "aPolygon",
        "polygon": {
            "positions": {
                "cartographicDegrees": [
                     0.0,  0.0, 100000.0,
                     0.0, 10.0, 500000.0,
                    10.0, 10.0, 100000.0,
                    10.0,  0.0, 500000.0 ]
            },
            "perPositionHeight": true
        }
    },
   {
        "id": "LowerLeft",
        "position": {
            "cartographicDegrees": [ 0.0, 0.0, 100000.0 ]
        },
        "point": {
            "color": {
                "rgba": [ 30, 144, 255, 255 ]
            },
            "pixelSize": 10
        }
    },
    {
        "id": "UpperLeft",
        "position": {
            "cartographicDegrees": [ 0.0, 10.0, 500000.0 ]
        },
        "point": {
            "color": {
                "rgba": [ 30, 144, 255, 255 ]
            },
            "pixelSize": 10
        }
    },
    {
        "id": "UpperRight",
        "position": {
            "cartographicDegrees": [ 10.0, 10.0, 100000.0 ]
        },
        "point": {
            "color": {
                "rgba": [ 30, 144, 255, 255 ]
            },
            "pixelSize": 10
        }
    },
    {
        "id": "LowerRight",
        "position": {
            "cartographicDegrees": [ 10.0, 0.0, 500000.0 ]
        },
        "point": {
            "color": {
                "rgba": [ 30, 144, 255, 255 ]
            },
            "pixelSize": 10
        }
    }
]```
@mramato
Copy link
Contributor

mramato commented Jan 14, 2015

This is most likely the same thing as #1825, but we'll keep this open until we can confirm both are fixed.

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 14, 2015

I suspect this is different than #1825 and is due to the normal computation.

@YouLu
Copy link

YouLu commented Jan 14, 2015

Hello all,

Is there a time frame when this bug is going to be fixed?
Thanks,

Steven

@pjcozzi
Copy link
Contributor

pjcozzi commented Jan 14, 2015

Hi Steven - we're heads down on KML support at the moment so I can't say that any of us are going to be available to look at it before the Cesium 1.7 release on March 2.

@hpinkos
Copy link
Contributor

hpinkos commented Oct 14, 2015

@pjcozzi I can't reproduce this. I was using this example:

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

viewer.entities.add({
    polygon : {
        hierarchy : Cesium.Cartesian3.fromDegreesArrayHeights([0.0,  0.0, 100000.0,
                     0.0, 10.0, 500000.0,
                    10.0, 10.0, 100000.0,
                    10.0,  0.0, 500000.0]),
        extrudedHeight: 0,
        perPositionHeight : true,
        material : Cesium.Color.WHITE
    }
});

viewer.zoomTo(viewer.entities);

Regardless, if you're using perPositionHeights and your polygon is not planar, there is no guarantee that it will triangulate the way you would expect it to, and that may lead to some weird issues.

@pjcozzi
Copy link
Contributor

pjcozzi commented Oct 14, 2015

@hpinkos your example looks good, but if you drag and drop the CZML in the first comment, there is still an issue:
ok

@mramato
Copy link
Contributor

mramato commented Apr 26, 2016

We recently found and fixed an issue with perPositionHeight normals, but it does not appear to fix this issue. (See above link).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants