Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#405 closed defect (fixed)

LOS manager is slow

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

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 (1)

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

Download all attachments as: .zip

Change History (7)

by Philip Taylor, 14 years ago

Attachment: los.patch​ added

a patch from telaviv

comment:1 by Philip Taylor, 14 years ago

(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 by (none), 14 years ago

Milestone: Open Source Release

Milestone Open Source Release deleted

comment:3 by Andrew, 14 years ago

Milestone: → Backlog

comment:4 by fabio, 14 years ago

Milestone: Backlog → Alpha 3

comment:5 by Philip Taylor, 14 years ago

Resolution: → fixed
Status: new → closed

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

comment:6 by Kieran P, 14 years ago

Milestone: Alpha 3 → Alpha 2
Note: See TracTickets for help on using tickets.