Skip to content

Async animals alert - #983

Open
still222 wants to merge 3 commits into
rwmt:devfrom
still222:AsyncAnimalAlert
Open

Async animals alert#983
still222 wants to merge 3 commits into
rwmt:devfrom
still222:AsyncAnimalAlert

Conversation

@still222

@still222 still222 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Vanilla uses (mostly) needs.food.TicksStarving to determine if animals are hungry/starving. Which in async results in constant warnings, when viewed from another, higher tick map.

Meanwhile vanilla humanlike trackers use needs.food.Starving, no idea why it wasn't done to the animals in the first place.
The most non-intrusive way I could come up with, was using postfix to filter the results and ensure animals actually hungry.

There is a separate bug for (async?) alerts, which results in them "jumping" around when hovered with a mouse. Not sure what causes it, but it's not this commit. With this commit if you see the alert, that means 100% there are starving animals, though sometimes (dont know exactly when or why) to get correct, non-jumpy tooltip, you have to find a correct map first. Without this commit alert will always show, sometimes jumpy, sometimes not, but they will be wrong most of the time.

@still222

still222 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

I've tested it as an external patch, cannot build real multiplayer dll for some reason.

If you can test it would be cool - host async server with 2 maps, have some pen animals (that have food need) in a pen and drift another map at least 30000 ticks (half a day) ahead.

Without commit you will see "Hungry pen animals" and "Animals starving" alerts when viewed from another map, with commit you should see them only if any animals have their need.food <= 0f.

When not in async postfix does nothing to save on iterations.

@still222
still222 marked this pull request as ready for review August 19, 2026 05:32
Renaming patch classes to be inline with the rest of the code
@notfood notfood added fix Fixes for a bug or desync. async Bugs or issues only in async time mode. 1.6 Fixes or bugs relating to 1.6 (Not Odyssey). labels Aug 19, 2026
@notfood notfood moved this to In review in 1.6 and Odyssey Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.6 Fixes or bugs relating to 1.6 (Not Odyssey). async Bugs or issues only in async time mode. fix Fixes for a bug or desync.

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants