Ticket #405 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

LOS manager is slow

Reported by: Philip Owned by:
Priority: Should Have Milestone: Alpha 2
Component: Core engine Keywords:
Cc:

Description

Profiler indicates the game is spending ~14% of its process's cycles in CLOSManager::GetStatus and CLOSManager::GetUnitStatus, when sitting idle in the default map, which is far too much.

Attachments

los.patch​ (15.9 KB) - added by Philip 3 years ago.
a patch from telaviv

Change History

Changed 3 years ago by Philip

a patch from telaviv

comment:1 Changed 3 years ago by Philip

(I'm keeping the patch here so it doesn't get lost, but don't want to test or apply it now because it'll get broken by the simulation rewrite - it should be revisited once that's done.)

comment:2 Changed 3 years ago by anonymous

  • Milestone Open Source Release deleted

Milestone Open Source Release deleted

comment:3 Changed 3 years ago by wacko

  • Milestone set to Backlog

comment:4 Changed 3 years ago by fabio

  • Milestone changed from Backlog to Alpha 3

comment:5 Changed 3 years ago by Philip

  • Status changed from new to closed
  • Resolution set to fixed

Fixed a while ago by rewriting the whole thing (it doesn't do any work when units are idle now).

comment:6 Changed 3 years ago by k776

  • Milestone changed from Alpha 3 to Alpha 2
Note: See TracTickets for help on using tickets.