Skip to content

added Grounded Skybox option#4604

Merged
elalish merged 4 commits intomasterfrom
skybox
Dec 22, 2023
Merged

added Grounded Skybox option#4604
elalish merged 4 commits intomasterfrom
skybox

Conversation

@elalish
Copy link
Contributor

@elalish elalish commented Dec 21, 2023

The three.js ground-projected-skybox example I found did not work well in practice, so I made my own simpler version. This goes along nicely with our new UltraHDR support because now we can transmit high-fidelity HDR lighting environments to use as skyboxes.

As with ground-projected skyboxes generally, it'll only look decent if the image is taken on fairly flat ground without nearby stuff that sticks up. The skybox-height attribute is used to get the proper scale of the image on the ground. If you know the height at which the image was taken, simply put that in - otherwise experiment until it looks reasonable. Ideally you'll want a height that is on the same order as the size of the model for it to look good.

@elalish elalish self-assigned this Dec 21, 2023
@elalish elalish requested a review from diegoteran December 21, 2023 04:13
diegoteran
diegoteran previously approved these changes Dec 21, 2023
Copy link
Collaborator

@diegoteran diegoteran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, I'll keep looking into it

@elalish elalish merged commit a35cafc into master Dec 22, 2023
@elalish elalish deleted the skybox branch December 22, 2023 02:48
JL-Vidinoti pushed a commit to vidinoti/model-viewer that referenced this pull request Apr 22, 2024
* added GroundedSkybox

* updated docs

* more docs

* added tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants