-
Notifications
You must be signed in to change notification settings - Fork 29
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
Terrain Support? #110
Comments
Already document at: https://visgl.github.io/react-map-gl/examples/terrain This code can work:
|
Hi @hecodeit. I don't think So for a small scene or a fairly flat surface, you can just pass the For a big scene, you decide how you want to adapt your scene. Maybe by moving the objects But I think it is fair that that's outside of the scope of this project, unless somebody wants to add a component that does exactly this and PR it (😉 😉 ) as an add-on rather than be an integrated part of the current components. |
Thanks for reply @RodrigoHamuy. I think the map itself, used as a background canvas. The Three.js part should include For example, a small town has some buildings at lower altitudes and others on hilltops. This town is a Three.js scene. Therefore, the background map will not affect the Three.js scene in any way. However, in this case, displaying the map as terrain is a better choice. Do you think this understanding is correct? |
Even if we lift the scene based on the origin coordinates, it will only be at the origin point, so that' won't be enough for most cases. Like in the image you provide, you need to know the altitude at certain intervals and decide if you want to morph the vertices via shader or manually set the geometry correctly. That's why I think this is out of the R3M scope. |
First of all, thank you very much for this project. It only took an afternoon to get the React Three Fiber and Mapbox project up and running.
Since I am new to this project, I noticed that the map does not include altitude information. Does it support Mapbox's terrain feature: https://docs.mapbox.com/mapbox-gl-js/example/add-terrain/
The text was updated successfully, but these errors were encountered: