topaz icon indicating copy to clipboard operation
topaz copied to clipboard

empty zones that are "asleep" will miss updates that happen onGameDay etc (the objects in the zones miss it)

Open topaz-next-bot opened this issue 4 years ago • 3 comments

Issue by TeoTwawki Saturday Oct 17, 2020 at 20:51:44 Originally opened as: project-topaz/topaz - Issue 1384


I have:

  • [x] searched existing issues (http://project-topaz.com/issues/) to see if the issue has already been opened
  • [x] checked the commit log to see if the issue has been resolved since my server was last updated

Additional Information (Steps to reproduce/Expected behavior) : Example: enter Sacrarium and see the wrong days door/floor configuration because its running whatever the last update it saw was. we need something to update these zones when they awaken from sleep.

topaz-next-bot avatar Dec 17 '20 07:12 topaz-next-bot

Comment by zach2good Saturday Oct 17, 2020 at 20:55:41


I was looking at these systems today, commenting so I'm reminded of this later (Zone OnCountIncrease CreateTimer, add lua binding there)

topaz-next-bot avatar Dec 17 '20 07:12 topaz-next-bot

Comment by cocosolos Saturday Oct 17, 2020 at 21:27:29


Just wanna clarify that the stuff in onGameDay still gets run when the zone is empty, the problem with Sacrarium is that it calls setAnimation which relies on the AI controller to update the entity, which is what gets killed when a zone empties.

topaz-next-bot avatar Dec 17 '20 07:12 topaz-next-bot

Comment by TeoTwawki Saturday Oct 17, 2020 at 21:48:24


Just wanna clarify that the stuff in onGameDay still gets run when the zone is empty, the problem with Sacrarium is that it calls setAnimation which relies on the AI controller to update the entity, which is what gets killed when a zone empties.

Yeah I think before the great AI rewrite when a zone "woke up" the 1st tick was supposed to make all the AI do an immediate update with whatever happened during the zones sleep happening all immediately. No idea if that ever worked correctly, but that was what had been intended.

topaz-next-bot avatar Dec 17 '20 07:12 topaz-next-bot