-
-
Notifications
You must be signed in to change notification settings - Fork 35.5k
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
Grounded Skybox #27448
Grounded Skybox #27448
Conversation
The E2E screenshot needs an update via |
Maybe it makes sense to convert the HDR to .jpg like in #27183 for this particular example? |
Hmm, Meantime, can someone else generate the screenshot? |
Seems like it worked on my machine... fc885c5 🤔 |
@elalish is there a reason for not setting I adjusted depthWrite in this PR as well: |
* initial change * mrdoob approves * updated example * update screenshot
I run a builder type site where people used to be able to tweak the parameters in real time to get their content to display the way they wanted. Is the best way now to just delete/recreate via |
Yes. Remove the skybox from the scene and then call |
Yep, the types showed there was no .dispose(), but after checking out the code and seeing how the new code is actually structured, that's exactly what I'm doing now. Thanks. |
@timothyallan do you have a link to that site? |
It's part of a new suite of 3d tooling being added to an existing SaaS platform. We've been beta testing it all for the past few months, and it's not available to the general public yet. It's good this update happened now actually! |
Fixes #27422
Description
As requested; I've simplified it a bit too, as well as removed the GUI from the example since the parameters are not really meant to be tweaked realtime.