Skip to content

Fix elements sometimes being visible from other dimensions in the current dimension #4184

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

Merged
merged 2 commits into from
May 4, 2025

Conversation

FileEX
Copy link
Contributor

@FileEX FileEX commented Apr 28, 2025

Fixes #1245 (and maybe #4165?)

The bug is that the player's dimension changes after one second, but during that time not all vehicles have been streamed-in yet. As a result, some vehicles get streamed while the player is already in the new dimension. If for example, increase the delay in the test script attached to the mentioned issue, the problem no longer occurs because all vehicles have already been streamed.
The solution is to cancel the streaming of an element if the element's dimension is different from its streamer's dimension (e.g., vehicle streamer)

@Dutchman101 Dutchman101 merged commit 9af03b3 into multitheftauto:master May 4, 2025
3 of 6 checks passed
MTABot pushed a commit that referenced this pull request May 4, 2025
9af03b3 Fix elements sometimes being visible from other dimensions in the current dimension (#4184)
@FileEX FileEX deleted the bugfix/dimension_stream branch May 4, 2025 20:45
@Nico8340
Copy link
Contributor

Nico8340 commented May 5, 2025

This pull request breaks the -1 dimension for client-sided objects

@Nico8340
Copy link
Contributor

Nico8340 commented May 5, 2025

cc @FileEX

tederis pushed a commit that referenced this pull request May 5, 2025
MTABot pushed a commit that referenced this pull request May 5, 2025
tederis added a commit that referenced this pull request May 5, 2025
Dutchman101 added a commit that referenced this pull request May 5, 2025
It will be picked up on for reintroduction tomorrow by either FileEX or Nico, for logic fixes.
Until then, we will value a stable build.
MTABot pushed a commit that referenced this pull request May 5, 2025
@tederis tederis mentioned this pull request May 6, 2025
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.

Vehicles from other dimensions may appear in the current one
3 participants