Skip to content

Conversation

xezon
Copy link

@xezon xezon commented Sep 29, 2025

This change decouples tree sway, topple and sink time step from render update.

Note that there are 2 kinds of trees:

  • Optimized: using W3DTreeBuffer
  • Unoptimized: using Object with logic modules, for example Palm Trees

Only optimized trees will enjoy full decoupling.

The unoptimized trees use the following modules:

  • SwayClientUpdate: for client swaying
  • ToppleUpdate: for logic topple
  • SlowDeathBehavior: for logic death

We cannot decouple ToppleUpdate and SlowDeathBehavior from logic.

TODO

  • Replicate in Generals

@xezon xezon added Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker Gen Relates to Generals ZH Relates to Zero Hour Rendering Is Rendering related labels Sep 29, 2025
@xezon xezon added this to the Decouple logic and rendering milestone Sep 29, 2025
@xezon xezon force-pushed the xezon/rendering-3-tree-updates branch from 38d536d to 4e24452 Compare October 6, 2025 20:39
@xezon xezon force-pushed the xezon/rendering-3-tree-updates branch from 4e24452 to 25727a1 Compare October 14, 2025 17:20
@xezon
Copy link
Author

xezon commented Oct 14, 2025

Updated.

Skyaero42
Skyaero42 previously approved these changes Oct 14, 2025
Copy link

@Skyaero42 Skyaero42 left a comment

Choose a reason for hiding this comment

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

Looks fine to me

@Skyaero42 Skyaero42 dismissed their stale review October 14, 2025 18:01

Want to test first

Copy link

@Skyaero42 Skyaero42 left a comment

Choose a reason for hiding this comment

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

Trees seem to sway correctly

@xezon xezon force-pushed the xezon/rendering-3-tree-updates branch from 25727a1 to 210e8f9 Compare October 15, 2025 20:29
@xezon
Copy link
Author

xezon commented Oct 15, 2025

Replicated SwayClientUpdate in Generals without conflicts.

Things in W3DTreeBuffer not replicated because Generals version of it is old.

@xezon xezon merged commit 9ba92fa into TheSuperHackers:main Oct 15, 2025
18 checks passed
@xezon xezon deleted the xezon/rendering-3-tree-updates branch October 15, 2025 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Is new feature or request Gen Relates to Generals Minor Severity: Minor < Major < Critical < Blocker Rendering Is Rendering related ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trees sway too fast

3 participants