Skip to content

added Grounded Skybox option#4604

Merged
elalish merged 4 commits into
masterfrom
skybox
Dec 22, 2023
Merged

added Grounded Skybox option#4604
elalish merged 4 commits into
masterfrom
skybox

Conversation

@elalish

@elalish elalish commented Dec 21, 2023

Copy link
Copy Markdown
Contributor

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

@diegoteran diegoteran left a comment

Copy link
Copy Markdown
Collaborator

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