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

fix: issues with quick transition between LOD - Scene - LOD - Scene #2462

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

fcolarich
Copy link
Collaborator

What does this PR change?

Added a small time before we switch from a scene/scene promise to a LOD, reducing the visual glitches and reducing the edge cases caused by quickly panning around the camera.

How to test the changes?

Go to -150,0, zoom into 1st person view, then approach the lonely house at -150,-3 until it changes from LOD to Scene. Then zoom out. It should take about 5seconds until it turns into LOD again. If you zoom in, it will turn into a scene again.
This delay should work for all scenes allowing to pan the camera around and leave a scene and come back without it changing too quickly to LODs.
Please let me know if the time set is OK or if we need more time for the change to happen.

2024-10-16.13-39-49.mp4

Copy link

github-actions bot commented Oct 16, 2024

badge

Windows and Mac build successfull in Unity Cloud! You can find a link to the downloadable artifact below.

Name Link
Commit 5866f3f
Logs https://github.com/decentraland/unity-explorer/actions/runs/11364977748
Download Windows https://github.com/decentraland/unity-explorer/suites/29690230161/artifacts/2063299849
Download Mac https://github.com/decentraland/unity-explorer/suites/29690230161/artifacts/2063647692
Built on 2024-10-16T13:49:38Z

Copy link
Contributor

@anicalbano anicalbano left a comment

Choose a reason for hiding this comment

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

🟢 Checked and working as expected! Alongside the test, the following smoke was performed:

✔️ Backpack/Emotes/Wearables
✔️ Social interaction on Chat/My Profile
✔️ Teleport with /goto and Map

16.10.2024_11.25.43_REC.mp4

Leaving a commend and awaiting for Mac test for fully approval 😄

Copy link
Collaborator

@DafGreco DafGreco left a comment

Choose a reason for hiding this comment

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

Implementation done on macOS ✅

  • Backpack
  • Scenes
    -Teleport

@anicalbano anicalbano self-requested a review October 16, 2024 15:31
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.

3 participants