Skip to content

Conversation

@tederis
Copy link
Member

@tederis tederis commented May 6, 2025

The reimplementation of #4184(thanks to @FileEX) and includes fix #4198(thanks to @Nico8340) with changes which make the whole thing a little bit more performant(it prevents models from loading instead of unloading them straight after the load).

@tederis tederis added the bugfix Solution to a bug of any kind label May 6, 2025
@Dutchman101
Copy link
Member

Thanks! This will be merged in one of the next rounds of builds that are intended for extensive testing. Streamer is always a sensitive part to modify, as we also just got reminded of.

(Please don't merge it before i do)

@Dutchman101 Dutchman101 requested a review from TheNormalnij May 7, 2025 20:01
@Dutchman101
Copy link
Member

I've requested a review from TheNormalnij as well, he has experience with streamer stuff.

@TheNormalnij
Copy link
Member

The logic looks correct

@blow1d

This comment was marked as spam.

@botder botder merged commit 1dff560 into multitheftauto:master May 17, 2025
6 checks passed
@botder botder added this to the 1.6.1 milestone May 17, 2025
MTABot pushed a commit that referenced this pull request May 17, 2025
1dff560 Fix model streaming (PR #4201)
fd955a8 Update crowdin-translators.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Solution to a bug of any kind

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants