beyond-all-reason / beyond-all-reason/RecoilEngine

Remove redundant events from `wupget:UnsyncedHeightMapUpdate`

Open
#1,839 1 comment 0 reactions 0 assignees View on GitHub
area: Lua API enhancement
Dominant language
C++
Stars
679
Forks
290
Avg merge
3d 2h
Merged PRs (30d)
40

Description

If a small rectangle completely fits within a big one it's sometimes sent redundantly anyway. Perhaps batch it or something.

What should happen to near-overlap (e.g. whether or not red+green should be merged to blue in the example below) is left unspecified.
![image](https://github.com/user-attachments/assets/1e9410da-6d45-47dc-93d3-e3bf282b35cb)

Contributor guide

Open the contributing guide

Research direction

Start by tracing the `wupget:UnsyncedHeightMapUpdate` event and how rectangle updates are queued or batched. Reproduce a case where a small rectangle is fully contained by a larger one, then define completion as eliminating the redundant event while preserving the currently unspecified near-overlap behavior until clarified.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.