Changes between Initial Version and Version 1 of Ticket #4142, comment 6


Ignore:
Timestamp:
Sep 18, 2016, 4:45:50 PM (8 years ago)
Author:
elexis

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #4142, comment 6

    initial v1  
    11Thanks for the patch Sandarac! I have committed your patch since it works very well (and before your patch, regicide AI games were guaranteed to be over after 10 minutes in the first attack). Hope to see more of your patches, even if it's hard to get reviews.
     2
     3I have replaced the `hurt()` check with a `healthLevel < 0.8` check, so that it would attack with heroes if they have sufficient HP. Also merged those hero type distinctions for now.
    24
    35* That check for near enemies is unfortunately extremely slow. It was lagging so much when I ran some tests with the patch, but not nearly as much without the patch. It was freezing every few turns for 2 or 3 seconds. After removing the check, it was working fine. Ideally we could use `ExecuteQueryAroundPos` of `RangeManager`, but it is out of reach currently.