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

Support rendering 3d extrusions #684

Closed
amyleew opened this issue Aug 11, 2014 · 11 comments
Closed

Support rendering 3d extrusions #684

amyleew opened this issue Aug 11, 2014 · 11 comments
Assignees

Comments

@amyleew
Copy link

amyleew commented Aug 11, 2014

Please add building height styling options for building data in Mapbox GL.

Specifically building data in CartoCSS can be rendered in a 3d look using building-fill, building-fill-opacity, and building-height.

Can we add this functionality to Mapbox GL?

@kkaefer
Copy link
Member

kkaefer commented Aug 11, 2014

dupe of #292

@kkaefer kkaefer closed this as completed Aug 11, 2014
@trallarn
Copy link

I know this issue is closed, but I can't find any doc on how to render extrusions. Has this feature really been added?

@kkaefer
Copy link
Member

kkaefer commented Sep 15, 2015

No, this feature hasn't been added yet.

@ghost
Copy link

ghost commented Feb 13, 2016

What is the status of development on the 2.5d line extruding? Is it something that I could help with?

@ansis
Copy link
Contributor

ansis commented May 12, 2016

implementing this:

the first part

followup

  • fix antialiasing (draw faded 1px lines along all edges?)
  • fix depth related bugs

@lbud
Copy link
Contributor

lbud commented May 25, 2016

I've roughly recreated last year's demo branch at https://github.com/mapbox/mapbox-gl-js/tree/building-redux (thanks to the codespirit of @ansis 👻 the old branch ported pretty easily).

image

Shaders at https://github.com/mapbox/mapbox-gl-shaders/tree/js-buildings, a spec branch (filled in only by copying fill properties for validation purposes, not definitive spec suggestions) at https://github.com/mapbox/mapbox-gl-style-spec/tree/buildings. I've modified this repo's debug page/style in this branch for the sake of quick iteration, not to be merged.

Still lots of TODOs to clean up rough edges in this branch + will open a spec ticket to discuss what this should look like.

@ghost
Copy link

ghost commented May 28, 2016

Nice work with the development!

I wanted to get a sense of how far along the development progress is - how long do you expect until functionality analogous to http://keks.com/dev/line3/ will be present.

@lbud
Copy link
Contributor

lbud commented Jun 7, 2016

@jonathanward at this time this feature is still very much in development, and spec changes are still not finalized at mapbox/mapbox-gl-style-spec#456, so I really can't give a timeline — though it is looking like at least initially we don't plan to support extruding non-polygon features.

@lbud
Copy link
Contributor

lbud commented Jun 13, 2016

quick status update:

  • most of the spec changes proposed for extrusions are implemented in mapbox-gl-js; mostly finished with adding data-driven properties and then need to figure out antialiasing
  • after that, still need to
    • port to native
    • work with Studio team to integrate extrusion type
    • solidify spec changes and do releases of -js, -native and -spec

@yocontra
Copy link

yocontra commented Oct 8, 2016

@iBud Any docs for enabling this feature? Didn't see any in the PR. Demos look great BTW - thanks a lot for putting in the work!

@lbud
Copy link
Contributor

lbud commented Oct 8, 2016

@contra check out the accompanying style spec PR mapbox/mapbox-gl-style-spec#495 — the fill-extrude-base and fill-extrude-height paint properties enable this, so a fill layer for buildings might include a property function like

"paint": {
    "fill-extrude-height": {
        "stops": [[0, 0], [1000, 1000]],
        "property": "height"
    }
}

if the specified source layer has a property called "height" in its features.
This feature will be in the next Mapbox GL JS library release.

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

8 participants