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(txd): bump txd pool size #2777

Closed
wants to merge 1 commit into from
Closed

Conversation

SKITTLE6969
Copy link

The primary purpose of this commit is to prevent crashes in the two most recent DLCs caused by addon TXDs. After version 2802, the limits were not updated. As a result, many servers are unable to upgrade to the latest DLCs due to TXD limitations. This bump will extend support up to version 2944 with a reasonable set of TXDs. However, further increases will be necessary if we plan to support versions 3095 and 3258.

solves #2724

Code compiles and has been tested successfully.
Code explains itself well and/or is documented.
My commit message explains what the changes do and what they are for.
No extra compilation warnings are added by these changes.

The primary purpose of this commit is to prevent crashes in the two most recent DLCs caused by addon TXDs. After version 2802, the limits were not updated. As a result, many servers are unable to upgrade to the latest DLCs due to TXD limitations. This bump will extend support up to version 2944 with a reasonable set of TXDs. However, further increases will be necessary if we plan to support versions 3095 and 3258.

solves citizenfx#2724

 Code compiles and has been tested successfully.
 Code explains itself well and/or is documented.
 My commit message explains what the changes do and what they are for.
 No extra compilation warnings are added by these changes.
@github-actions github-actions bot added the triage Needs a preliminary assessment to determine the urgency and required action label Sep 7, 2024
@Korioz
Copy link
Contributor

Korioz commented Sep 7, 2024

PR #2639 is already pending, it is addressing the same issue.

@SKITTLE6969
Copy link
Author

Elements kindly merge this or the other to the main :(

@Legacy-TacticalGamingInteractive

@prikolium-cfx had mentioned this before I think here #2392 if this is related idk

@AvarianKnight
Copy link
Contributor

You can do this for canary on the latest server build

See: https://github.com/citizenfx/fivem-docs/pull/481/files

@SKITTLE6969
Copy link
Author

Thanks guys <3

closing in favour of citizenfx/fivem-docs#481

@Legacy-TacticalGamingInteractive

You can do this for canary on the latest server build

See: https://github.com/citizenfx/fivem-docs/pull/481/files

To be clear, so we could add say increase_pool_size "TxdStore" 26000 and that would add an additional 26000 to clients?

@prikolium-cfx
Copy link
Contributor

You can do this for canary on the latest server build
See: https://github.com/citizenfx/fivem-docs/pull/481/files

To be clear, so we could add say increase_pool_size "TxdStore" 26000 and that would add an additional 26000 to clients?

Absolutely right. But keep in mind that it will work only for Canary users. ETA for delivering this feature to Production is ~6 weeks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Needs a preliminary assessment to determine the urgency and required action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants