The Forums Are Now Closed!

The content will remain as a historical reference, thank you.

The Updated Enhanced AI Mod Thread

By on June 14, 2011 11:41:43 PM from Demigod Forums Demigod Forums

pacov

Join Date 02/2008
+182

Updates:

Public version 1.03 live 1/18/2012 at 6:07 PM EST


 


Download links:

Link to the latest version PUBLIC version of the mod:  http://www.box.net/shared/g8lbbhpojik4tbkjn6su


I created a post that tracks the history of the AI in Demigod and it covers a good chunk of the development that has already been handled by the modding community.  If you are interested, see over here for more details:  https://forums.demigodthegame.com/409712

The Project

Using the last version of peppe’s AI mod (0.26.35), I’ve started development on updated versions of the AI mod.  I’ll be releasing those versions here, tracking bugs, enhancement requests, rebalancing, etc.  I won’t be adding a new version to my combined DG installer until we make a decent amount of progress or come up with some sort of significant enhancement to justify a new release to the community at large.  It took us a long time to get reasonable adoption rate on the existing ai mod – I’d rather not force folks to redownload the bundle over and over.

The Project Team

  • Project lead:  pacov
  • Lead design/coder:  pacov
  • Lead support:  miriyaka – without miri, the majority of the changes we are looking at would not happen
  • Testers:  sledge, plaguewraith, and darkliath 
  • Contributors:  Lord_Orion

If you’d like to help in any way, we are happy to have you on the team.


Current status (what’s being worked on)

  • Build updates (removing inefficient builds and replacing them with new ones) - complete
  • Reprioritization of citadel upgrades, items, trips to the shop - largely complete - some late game tweaking still possible, but its improved quite a bit
  • Implementing a fix for AI’s with summonable minions so that they do not waste mana summoning more after the have created the max number of minions (miri created some code and I’m looking to make it work)  Resolved in 0.26.41
  • Recoding the ShouldSave functionality to improve the coding efficiency and gives us additional flexibility.  If we get this working, it will be a very solid improvement and provide more options.  - this has been implemented

Bug list (unless otherwise noted, these are all bugs from the original 0.26.35 version)

  • Stuck AI – the AI can get stuck on occasion for > 30 seconds
  • Poor scaling based on AI difficulty - this has been improved but there is still room for improvement
  • Not enough shopping trips for the AI unless it is killed or sent back the base with low HP (eg an AI could have 10k gold and still not attempt to shop and get better items) - 1 forced shopping trip at wr 4 has been added... might be good to do more, but much better now.
  • AI is not always getting fs1 in a timely manner (resolved in 0.26.39)
  • Some abilities are not being used as often as they should (a sedna, for instance, with pounce as her only ability would use it once in a blue moon) - continuing to make improvements (pent/pounce/grasp/deep freeze have all been fixed)
  • Some abilities are being used at inappropriate items and wasting mana as a result
  • Customized settings often throw the AI off.  For instance, if you set a game to being at WR10, the AI don’t shop (not sure if we want to try to code this to cover all possible settings) - improved some.
  • AI standing around at the shop at the beginning of the game (introduced in 0.26.38 – happens randomly and seems to only impact 1 ai on a team) - resolved in 0.26.40
  • Not truly a bug, but something I want to change - need to make it so the high gold dg can still buy things for themselves (resolved in 0.26.49 "siesta")
  • AI considers minions as part of a DG's army... this is good and bad - its good to target healers from time to time, but if I can make an enemy target a shambler while I'm wailing on him...

Enhancement requests (these are changes that folks would like to see happen with the AI)

  • Better healer distribution.  The over all goal would be to ensure that monks/clerics/etc are sent to appropriate players.  The basic code works fine, so this is a lower priority change imo.
  • Improve da’s shadow swap usage – plaguewraith suggests removing it completely from his build (might be a good temporary solution until we can tackle this) (removed 0.26.55)
  • Better creep farming with skills.  Plaguewraith indicated that he never saw a tb use circle of fire to farm.  This is something we could tweak, but we’d need to think it through
  • Lord-Orion suggests we remove all ub builds except for Spit/Ooze life - REMOVED 1 OF THE BUILDS 0.26.40 - still 2 left
  • Lord-Orion suggests we optimize all builds for skill dmg output as the AI cannot currently rationalize like a human can. 
  • Lord-Orion suggests we go with a basic item prioritization of 4 life/1 helm builds – I agree to some extent… (pretty much done as of 0.26.52)
  • Come up with a way for the ai’s to intelligently chain stuns or interrupts to go for kills.  In theory, the AI could be even better than a human at this if it was tied to precise timing
  • Teach the AI to teleport for ganks (this could be tricky – would probably result in a lot of pointless tp’s)
  • Find a way to help the AI manage mana better.  Early game the ai’s are almost always out of mana
  • Teach the AI when to shop!  (big improvement with 0.26.49 "siesta")
  • Have the AI always engage a simple enemy opponent if on a flag if hp is good, etc
  • Have the AI choose to engage towers on a regular basis (improved a lot since 0.26.54)
  • Have the AI refuse to run past towers in most circumstances and instead TARGET THE TOWER if no dgs around
  • Improve cataract map logic – the ai often goes into the middle of the map instead of staying in a lane.  This is probably due to the proximity of the creeps when the ai is working on capturing the middle flag (or if all flags are controlled by their team).  Ideal behavior would be to try to capture the middle flag and if its already controlled, then get into a lane on HP or Mana.
  • Override the flee functionality if there is an enemy dg and its likely possible to get a kill
  • Better decision making for flags – Ai heading over from mana side all the way to Hp without capturing mana.  this is likely due to flag weights… just not sure why it didn’t evaluate hp as the priority initially – could be some conflicting code?
  • Send the AI back to base if its low mana (sledge) – We might do something for this, but it would have to be a conditional statement – perhaps add a check to see if they are at low mana around X Ws have > X money, then run back and prioritze getting a helm as an override
  • Plaguewraith suggests removing mist from erb’s build – I’m not sure about this yet.  He says remove it or build a routine to cause erb to use it just to get rid of negative buffs – I agree that we could use a routine like that.  - mist removed in 0.26.40
  • Encourage the AI to react quickly if they are losing a flag that is near 2 or more towers (eg gold flag)
  • Increase the frequency that the AI picks up sigils – not sure about this one – I agree a good player should generally have one, but the AI might just blow a lot of money casting when it doesn’t need to and fall behind as a result.  Perhaps this becomes a conditional statement to raise the priority after ws X (0.26.47 - implemented)
  • Add an override so the ai will only go for flags like gold if the tower is down.  Add another override so the ai doesn’t go for portals early (probably WS related)
  • plaguewraith – delete the tb builds and just stick with 1 fire tb build - REMOVED 1 BUILD - only pure fire and pure ice remain 0.26.40

Change log

version 1.03
-Created new UID
-Adjusted hero/squad targeting values to increase AI skill use aggression
-Adjusted range cutoff multiliers to mitigate the chance of the AI from running past towers to cap flags
-Adjusted/fixed errors in Oak, Queen of Thorns, Unclean Beast, Regulus and Demon Assassin AI builds
-Re-enabled Sedna Pounce build and Queen of Thorns Shield_Spike build
-Homogenized AI build names so it is obvious what skills the AI is using.

version 0.28.00 BETA
- removed unclean beasts AA build as the Spit ooze build is generally a little better

version 1.02
- created new UID and incorporated all changes since version 1.01

version 0.27.09 BETA
- Disabled a substantial amount of logging (will result in a substantial performance boost for many)
- added miri's scenario name capture function to CommonUtils.lua (doesn't work now, but isn't being called)
- began to tweak ub's usage of ooze.  Reduced health activation from >= 40% hp to >= 30%.  Also reduced the deactivation health value from < 40% to < 30%
- reenabled the attack override in herogoap
- updated the flee mastergoal to set at 50% HP instead of the current 75% hp
- Added an action time to health pot usage to hopefully keep the AI from using a pot at lower HP, having the pot bring them up to full strength and then having the ai immediately sigil

version 0.27.08 BETA
- Added new action and instant status function in useitemactions to keep the AI from "double locking" flags (eg wasting locks on a flag that is locked)

version 0.27.07 BETA
- increased sigil activation health % from 45% to 50%
- added new hammerslam calculate rates function - should increase the odds that rook will slam if the unit is stunned (should work for any type of stun)
- rebalanced weights of rook's actions to bring them more into logical numbers
- rebalanced weights of erb's  actions to bring them more into logical numbers

version 0.27.06 BETA
- reduced sigil activation health % from 50% to 45%
- changed orb of defiance usage check so that it will consider using it before sigils
- removed grunt check on orb of defiance (previous the AI would refuse to use use the orb if the threat level was < 15)
- added nearby enemy hero check to orb of defiances - if no enemies nearby, then orb will not be used
- reduced the value of narmoth's ring on the AA ub build so that it is not choosen as the only item at the start of a game on nightmare difficulty
- reduced the captureflag override at the start of the game from 60 seconds to 40 seconds
- disabled the attack override to allow the ai to make its own decisions based on weight
- Reduced the reteat values if there are nearby enemy heroes and towers from 85% to 75%
- modified rules for dg vs dg fights.  AI will run if there are more enemies than allies present

 version 0.27.05 BETA
- removed the nonworking per map flag settings
- revised the generic flag settings
- disabled all existing rook builds
- added new "more balanced" hammer slam tower build

version 0.27.04 BETA
- continued to enhance the documentation in heroGOAP
- added new logic to provide a count of heroes/enemies in heroGOAP for decision making
- added rule so that the AI's goal will flee if 3 or more enemies are present vs 1 ai
- changed default value of gold and portal flags to 0.5
- increased unit.movecutoffrange from 1.2 to 2.5 in attackactions
- fixed a problem with da's new swap logic
- continuing to test out flagassets.lua - I don't think values are being loaded for each map

version 0.27.03 BETA
- removed erb's desire to cast stun as an interrupt as its not possible
- increased erb's desire to bite
- increased oak's desire to use surge to kill units
- removed sedna's desire to use silence as an interrupt as its not possible
- added comments to heroGOAP to try to track where the AI is getting stuck (NOTE - this could slow down some lower end pcs)
- added new logic to count the number of grunts near a hero for decision making purposes - previous check was based on threatlevel
- changed the balancing capture flag logic so that the AI will re-prioritize capturing flags if there is a difference of 50 in warscore
- changed AI's desire to buy capture locks from WR 4 to WR 6 - AI will not purchase them prior to WR 6

version 0.27.02 BETA
- major revamp to DA's swap ability - da will now only swap if the number of allies is > enemies near da
- disabled existing DA build
- enabled STANDARD_ASSASSIN da build (eg what most players use when playing da) now that swap is working as desired
- disabled pounce sedna build

 version 0.27.01 BETA
- updated TB's frost nova so that it is used more often

version 0.27.00 BETA
- created new UID - this is done so folks can still keep the release version 1.0 installed and try out new "beta" versions and help with testing, etc
- added additional documentation to AIGlobals.lua
- adjusted the saving routine so that angels are not saved for until ws 7
- changed the way the AI evaluates additional shopping trips.  Now based on warscore
- fixed a minor bug with reg's mark of the betrayer squad target
- added additional shopping trips (see details below)
    # SHOP PERIODS
    # Warscore >= 300, AI with most money, possible to buy fs1, at least 600 gold
    # Warrank >= 3, AI with most money, possible to buy cur1, at least 1800 gold
    # Warscore between 2450-2575, NOT AI with most money, at least 1500 gold
    # Warscore between 3100-3225, AI with most money, at least 1500 gold
    # Warscore between 3800-3925, AI with most money, at least 1500 gold
    # Warscore between 4150-4275, NOT AI with most money, at least 1500 gold
    # Warrank 8 OR AI already bought the upgrade, priest/angel/cats available, AI can afford the upgrade
    # Warrank 10, possible to buy giants, AI can afford the upgrade

version 1.00
- created new UID
- removed any "pacov" labeling
- changed name to Enhanced AI (peppe's original version was Enhanced_AI
- updated version name to 1.00 (numbering convention will be 1.00/1.01/etc going forward)

 version 0.26.56 (misc fixes + pounce sedna build is live)
- removed CaptureFlag goal weight from oak's pent functions
- removed CaptureFlag goal weight from rook's hammerslam functions
- removed Captureflag goal weight from tb's deep freeze functions
- removed CaptureFlag goal weight from ub's grasp functions
- removed ub's mygraspstatusfunction and replaced with DefaultStatusFunction
- removed oak's myPenitenceStatusFunction and replaced with DefaultStatusFunction
- removed sedna's myPounceStatusFunction and replaced with DefaultStatusFunction
- substantially increased sedna's desire to pounce
- re-enabled sedna's pounce_tank build

version 0.26.55
- fixed a bug that still allowed demon assassin to pick up swap
- fixed a bug with unclean beast's grasp code
- enabled new hammerslam/tower rook code and tweaked desire to hammerslam
- increased sedna's desire to pounce (did not re-enable the pounce build yet)

version 0.26.54
- removed the remaining demon assassin build and added a new build without swap per request
- reworked the valor flags weight.  Should be less desirable for AI prior to ws 8
- tweaked deep freeze to be cast much more often
- re-enabled ai priority to attack structures.  Tweaked the formula so the AI will immediately back off if any enemy dgs come into range.  This should reduce the odds of death and also keep the AI from wandering past towers for the gold flag, etc

version 0.26.53
- increased artifact weight so they will be kept if the AI purchases
- added mageslayer to the generic equipment purchase list with a priority of 110, moved godplate to 120
- enabled oak to cast surge when trying to flee

version 0.26.52
- substantially increased the odds that oak will attempt to interrupt
- removed demon assassin speed_spine build
- re-enabled the graveyard level 1 upgrade
- minor misc changes
- changed ai's desire to pick up locks from level 3 to level 4

version 0.26.51
- rebalanced the general equipment builds
- rebalanced the specific demigod equipment builds
- removed AI's desire to purchase any graveyard upgrades
- removed "cloak of invisibility" from artifact prioritization as the item does not exist

version 0.26.50
- Reduced priority from 35 to 20 for boots of speed on UB HP/ooze build to keep the AI from purchasing boots of speed as the first item if the AI is set to normal
- increased priority of grofflings plate in the general build
- substantially increased ub's desire to grasp in game
- updated flag goal for cataract to reduce the AI's desire to grab the valor flag early (eg the AI running to the middle of the map)
- increased ice tb's goal to make it use deep freeze more often (tb's abilities all need a bit of an overhaul)
- various item selection tweaks

version 0.26.49
- changed desire for flag locks to increase at wr3 instead of wr4
- adjusted AA ub build so that it will never choose mana items (unbreakable is still acceptable, though)
- implemented the "siesta."  At warrank 4, any ai (not the high gold AI), will return to base to shop as long as they have 1500 gold.  Then, towards the end of wr4, the AI that is the highgold AI will shop alone. 

version 0.26.48
- added miri's check to force the tb to stay in whatever mode its build is designed for.  This should improve the AI's usage of abilities tied to the pure ice or fire builds.  Confirmed that fire tb will stay in fire form and ice in ice form based on build.
- continued to balance item selections
- Changed Rook's favor item to blood of the fallen
- TEMPORARILY turned off the attack structure code

version 0.26.47
- made MANY balance changes - all changes are noted in the files, but too many to detail here (so I'll cover highlights)
- changed ideologies for the mod.  Before the goal was to force the AI to do everything I wanted it to do (simply buy cit upgrades) - now I'm planning on having it scale that back and focus on becoming an arse kicker
- odds your ai will have a sigil is MUCH higher - this improves survival odds ALOT
- ai will always purchase fs1/cur1/priest/angel/cats/giants - that's it.  AI will no longer get any levels of experience.
- ai will VERY OFTEN have locks - I still need to teach the AI how to use locks better though... so at least for now, it will have them...
- MANY item prioritization changes - If you understand the modding side of things a wee bit, there are 2 ways a dg chooses equipment/items:  1 - a general list that contains all recommendations.  2 - a list that is specific to demigod build.  I spent quite a bit of time improving the general list today and started working on the 2nd method that includes build specific items.  I've only started on QoT, but I will likely get her to mimic my standard QoT build for items.
- removed some additional checks to help reduce overhead of the AI

* note - I have not begun to force shopping trips on the AI outside of fs1/cur1/priest/angel/cats/giants.  I'll be looking to start phasing in shopping trips to encourage the ai to get even better items as appropriate.  AI will still shop if it gets its arse kicked, but I want to schedule some trips so the AI will get stronger at different intervals even if its doing fine.

version 0.26.46 (the getting back on track version)
- added miriyaka's changes to the save function so that we can evaluate based on warscore as well
- added saving for fs1 as a priority after ws 200
- updated value for fs1 to increase its priority to 200 after ws 300
- removed log writes from miri's savefor, etc to see if that helps the lag issue that's been mentioned
- confirmed - the generals will now choose monks if they are a normal ai with default settings (instead of saving for fs1)

version 0.26.45 (the lesser of two evils/back to basics version)
- the title gives you an idea - if you play on normal settings with 1 general ai, they will NOT purchase monks at the start.  This is not desired behavior.  But in allowing this, the AI does not freeze between ws2 and 3 for any extended duration.  Sadly, the other option right now is you get monks, but the ai stands around like an idiot at ws2.  
- Added additional documentation for the herogoap file
- disabled the trip to the shop for ws 5 to get cur2
- developed basic test code to send the ai to the base if its not the high gold ai
- backed out the changes that increased rate of fire for bite/pounce/pent.  We'll visit this again, but I'm concerned that the ai is overriding their flee function due to the high priority I put on these abilities resulting in more pointless deaths.  My goal is to simply make them use their abilities more often... not die like goofs.
- tweaked the rules for purchasing sigils.  AI will not buy them until ws 2 and only if the AI has a max health > 2750
- removed assassin sedna build again until I get back to sorting pounce

 version 0.26.44
- hopefully corrected a bug that would cause the ai wait around to purchase an upgrade if it did not have enough money of the ws for it.  This should give us new opportunies for balancing upgraded in the future.  The short version is I updated HeroGoap perform a check if the ai can afford the upgrade they are saving for before heading off to the shop.  Not enough money = no shop.  I'll be looking into explicitly sending specific ai's to shop on some sort of interval in the future
- (not a real change... but started deep dive into integrating miri's savefor function)
- removed cur2 shopping trip for now - it still gets a high priority, though

version 0.26.43
- added miriyaka's save for gold functionality
- disabled ai goal chat function from herogoap 1198

version 0.26.42
- substantially increased the rate that erebus uses bite
- substantially increased the rate that sedna uses pounce
- re-enabled assassin sedna build now that pounce is used more often
- substantially increased the rate of oak's penitence
- added another shop period at ws 4 - the ai with the most goal should head for base a 4.  This might result in a different dg being choosen for highgold after

version 0.26.41
- added miriyaka's summon shambler fix.  The code would work for QoT, Sedna, and Oculus.  I'm only implementing it for QoT as this would be a disadvantage for the Oculus build and sedna does not use yetis in the AI mod
- Removed anklet of speed from all TB builds and replaced with Blood of the Fallen.  The ai is not smart enough to use a speed fire tb build
- Removed the file mod_units. This contains fixes already in uberfix and is not needed here
- Enabled master goal chat - this is a debug function that broadcasts what the AI's goals are - you might find this annoying...
- Re-enabled code that adds a destroy structure goal for the AI.  Peppe turned this off at some point... probably for a good reason, but I did see the AI being more aggressive attacking towers, so I'm leaving this on for now.
- Made some adjustments to the flag weights on cataract.  This SHOULD end dgs running over the the mana flag at the start and then rushing to HP before they capture the mana flag.  Reduced the weight of the gold flag to encourage the ai to attack structures first.  Reduced the value of portals prior to ws8.
- Removed all oak builds and added a new shield/pent focused build
- Minor change for reg's build - added impedance bolt at level 16 instead of stats 1

version 0.26.40
- removed the code that placed a limitation on what items could be purchased at the start of the game (AIShopUtilities 736-741)
- Changed the priority level trigger from WarScore to WarRank for fs1 (AIGlobals 1433) - this appears to have resolved the ai's standing at mid issue.  The Ai will purchase fs1 at ws2 now correctly
- Changed AI's priority to get xp2-xp4 to 0.  AI will no longer purchase these upgrades
- removed mist and added coven 1 as erb's level 2 skill until we get a chance to write a routine for him to use mist to remove negative buffs
- Updated ai priority values for minotaurs to 0/5/10/15 (eg an ai will never buy the 1st level minotaurs now) - I was seeing the ai pick this up as a cheap filler if they have the money - not worth it
- Updated ai priority values for level 1 archers - dropped from 15 to 0 so the ai will never buy
- updated ai priority values for hauberk of life - dropped from 40 to 35 so that unbreakable would be chosen over this if money was available
- updated ai priority values for unbreakable - removed conditional formula and set to a static 39
- removed ub skill build spit_ooze_mana (essentially bots ub)
- removed hybrid_fire_ice build from tb (bots tb)

 version 0.26.39
pacov is learning things…
- reprioritized AI to purchase fs1 at ws2.  Previously, it only purchased it if made it back to base with enough money; this forces it back to base to get it if no one else has
- raised priority for currency 2.  Logic mirrors currency 1 as I want this purchased every time.  Also sending ai with the most gold to purchase at ws 5.
- removed assassin sedna build until I have a chance to look at the pounce training – assassin sed should be pouncing left and right… right now its like once in a blue moon – heal_tank sedna is solid though

version 0.26.38
removed unitstatussensors code – concerned it might be causing an issue
- tweaked erebus build so that he gets mass charm later – he’s not using it well as is

version 0.26.37
bugfix – just resolving a sytax issue

version 0.26.36
copy of peppe’s version 0.26.35
- removed 3 old sedna ability builds
- added 2 new sedna ability builds
- removed old qot ability builds
- added new qot ability build
- removed 2 old erebus ability builds
- added 2 new erebus ability builds
- uncommented some code peppe developed in UnitStatusSensors that might help resolve the frozen dgs
*note – all builds that have been added by me will be announced in team chat at the start of the game and will say “pacov” folowed by the build name.

Pinned, Locked Post 458 Replies +1
Search this post
Subscription Options


Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 7:02:23 AM from Demigod Forums Demigod Forums

One more thing i´d like to suggest:

Can the difficulty setting for the AI set to nightmare, when a player drops out?

This could help to increase it´s usage in MP games, i think.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 8:55:35 AM from Demigod Forums Demigod Forums

Thats a really good point of Plague, although I think that has nothing to do with the ai mod.

 

Seems to me that the TB probs I talked about yesterday are similar to the qot problem. A hybrid TB build doesnt have to be bad but it seems that the ai changes form very seldom and then only has half of the skill sets available. So either let them in one form only or make them better in changing forms, but its not only qot.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 2:57:13 PM from Demigod Forums Demigod Forums

Quoting Plaguewraith,
Can the difficulty setting for the AI set to nightmare, when a player drops out?

I don't think that's possible, but that shouldn't matter, too much.  They'd start using the enhanced AI logic regardless of what type of AI they are.

edit - updated the OP - now we have good tracking in place.

As you think of enhancements or bugs, please use a simply bullet point format - makes it easier for me to keep the op updated.  Thanks!

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 4:07:30 PM from Demigod Forums Demigod Forums

been toying around with the qot code a bit more.  I've now got it to the point where the log is outputing the summonstatus function info.  Its even successfully counting the max number of shambers as expected - and then also indicating when its at max, but the qot is still successfully resummoning shamblers.  trixie

Think I'm going to put this on the back burner and get back to things i can sort without assistance for a wee bit.  We'll see how much time I get - hopefully bump out a new version tonight.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 5:44:56 PM from Demigod Forums Demigod Forums

Continuing work on balancing.

Only played one game with my latest version, but I'm very pleased with the minor changes and how the AI is handling itself with regards to shopping now.  Going to take a look at what citadel upgrades are purchased next.  There will very likely be a new release tonight. 

Current changelog for 0.26.40:

- removed the code that placed a limitation on what items could be purchased at the start of the game (AIShopUtilities 736-741)
- Changed the priority level trigger from WarScore to WarRank for fs1 (AIGlobals 1433) - this appears to have resolved the ai's standing at mid issue

 

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 6:45:00 PM from Demigod Forums Demigod Forums

Quoting OMG_pacov,
quick note, though. I hooked AIAbilityUtilties as a standalone file. Should I not have done that. Here is EXACTLY what I have in AiAbilityUtilizies:

That's fine, that's how that was meant to be used.

Quoting OMG_pacov,
been toying around with the qot code a bit more. I've now got it to the point where the log is outputing the summonstatus function info. Its even successfully counting the max number of shambers as expected - and then also indicating when its at max, but the qot is still successfully resummoning shamblers.

What did you change to get it to see MaxShamblers?

Try adding a 'return false' after the 'already at max' log message, inside the same else block (line 29 I guess).  It should be doing that already, but whatever.  Is it also listing the 'Fail' message after it says it's at max summons, or does it just stop there?

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 6:51:14 PM from Demigod Forums Demigod Forums

Quoting miriyaka,
What did you change to get it to see MaxShamblers?

I... cheated a bit, methinks.  If I changed the name of in the heroaiaction template (HQueen_AIActions) to Shambler1, I'd get results to the log file.  If I left it as shambler, I never got anything.  I decided to change this to shambler1 when I was testing as I noticed a message pop up in the log indicating that it found duplicate values for shambler... anyway, I'm still at a loss here.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 6:58:03 PM from Demigod Forums Demigod Forums

version 0.26.40
- removed the code that placed a limitation on what items could be purchased at the start of the game (AIShopUtilities 736-741)
- Changed the priority level trigger from WarScore to WarRank for fs1 (AIGlobals 1433) - this appears to have resolved the ai's standing at mid issue.  The Ai will purchase fs1 at ws2 now correctly
- Changed AI's priority to get xp2-xp4 to 0.  AI will no longer purchase these upgrades
- removed mist and added coven 1 as erb's level 2 skill until we get a chance to write a routine for him to use mist to remove negative buffs
- Updated ai priority values for minotaurs to 0/5/10/15 (eg an ai will never buy the 1st level minotaurs now) - I was seeing the ai pick this up as a cheap filler if they have the money - not worth it
- Updated ai priority values for level 1 archers - dropped from 15 to 0 so the ai will never buy
- updated ai priority values for hauberk of life - dropped from 40 to 35 so that unbreakable would be chosen over this if money was available
- updated ai priority values for unbreakable - removed conditional formula and set to a static 39
- removed ub skill build spit_ooze_mana (essentially bots ub)
- removed hybrid_fire_ice build from tb (bots tb)

New version is live.  Pretty good improvements all around.  Download in the OP.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 7:11:20 PM from Demigod Forums Demigod Forums

Oh, I see the problem.  InstantStatusFunction is used for some kind of action planning, but that can still be bypassed, in which case the ActionFunction is used to determine whether or not the ability is ready, and if so, to cast it.

New AIAbilityUtilities.lua: http://pastebin.com/32eaepJp

Keep the InstantStatusFunctions how they are, and point each action's ActionFunction toward AIAbility.SummonActionFunction as well.  Assuming I didn't make any mistakes, this should catch any plan-independent attempts to cast a summon as well.

Quoting OMG_pacov,
I... cheated a bit, methinks. If I changed the name of in the heroaiaction template (HQueen_AIActions) to Shambler1, I'd get results to the log file. If I left it as shambler, I never got anything. I decided to change this to shambler1 when I was testing as I noticed a message pop up in the log indicating that it found duplicate values for shambler... anyway, I'm still at a loss here.

The original 'Shambler' action would still exist if you were doing that.  I would highly recommend you use the syntax I provided to modify InstantStatusFunction and ActionFunction, so as to avoid any issues with overwriting the existing action:

Code: c++
  1. HeroAIActionTemplates.Shambler.InstantStatusFunction = AIAbility.SummonStatusFunction
  2. HeroAIActionTemplates.Shambler.ActionFunction = AIAbility.SummonActionFunction
Again, this goes outside of any templates, and you do not need to re-paste or override the original Shambler template in QoT's actions file - just leave it how it is in the original AI mod file, and put this at the end of the file.

And if it doesn't work, post a log.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 7:52:09 PM from Demigod Forums Demigod Forums

Ok, I got a few minutes to test and troubleshoot these things, and holy crap do FindBestBaseItem / FindBestCitadelUpgrade run often.  Like, every 2-3 seconds.  That means the old ShouldSave function was running about, uh, ~200 times per second, on average.  This new SaveForUpgrade function only runs once per query per purchase type (twice every 2-3 seconds), and is much simpler.  So hopefully that'll cut down a bit on AI sim excess for people with slower CPUs.

Haven't had time to fully test SaveForUpgrade's actual functionality, but I have it outputting all relevant info to the log for each run, so it should be easy to watch it manage what each demigod is saving for.  The error it was producing was related to a lack of parentheses I had around some conditional unit.Sync.ItemCostMod accesses - I thought an 'or' would always be processed before math operations, but apparently they're considered in left-to-right order along with high order-of-operation functions (multiplication and division).

The new max-summon prevention system works great.  It runs a bit often for my liking, but I kept it simple, and there's already a ton of overhead inside the action system, so there isn't much to be done about that.


Here's an updated set of FindBestBaseItem / FindBestCitadelUpgrade functions for AIShopUtilities.lua: http://pastebin.com/Vt2EdjS6

And the same SaveForUpgrade set I posted before, to replace ShouldSave in AIGlobals.lua: http://pastebin.com/C8pDzaYy

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 8:04:33 PM from Demigod Forums Demigod Forums

Thanks miri - looking over your code now.  Going to see if I can get the summon thing sorted based on your feedback.  If I can integrate that and test, I'll move on the saveforupgrade stuff.  Thanks!

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 8:13:31 PM from Demigod Forums Demigod Forums

OK - tested and it works.  Do you think I should keep the logging enabled or kill that?  I assumed it was just for testing, but figured I'd see what you think.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 8:59:55 PM from Demigod Forums Demigod Forums

You can just comment out any log messages in the stuff I give you once it's working how you like, unless there's a particular piece of info you want to keep in there for when you're balancing something.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 9:11:40 PM from Demigod Forums Demigod Forums

version 0.26.41
- added miriyaka's summon shambler fix.  The code would work for QoT, Sedna, and Oculus.  I'm only implementing it for QoT as this would be a disadvantage for the Oculus build and sedna does not use yetis in the AI mod
- Removed anklet of speed from all TB builds and replaced with Blood of the Fallen.  The ai is not smart enough to use a speed fire tb build
- Removed the file mod_units. This contains fixes already in uberfix and is not needed here
- Enabled master goal chat - this is a debug function that broadcasts what the AI's goals are - you might find this annoying...
- Re-enabled code that adds a destroy structure goal for the AI.  Peppe turned this off at some point... probably for a good reason, but I did see the AI being more aggressive attacking towers, so I'm leaving this on for now.
- Made some adjustments to the flag weights on cataract.  This SHOULD end dgs running over the the mana flag at the start and then rushing to HP before they capture the mana flag.  Reduced the weight of the gold flag to encourage the ai to attack structures first.  Reduced the value of portals prior to ws8.
- Removed all oak builds and added a new shield/pent focused build
- Minor change for reg's build - added impedance bolt at level 16 instead of stats 1

New version live

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 9:24:01 PM from Demigod Forums Demigod Forums

Quoting miriyaka,
Ok, I got a few minutes to test and troubleshoot these things, and holy crap do FindBestBaseItem / FindBestCitadelUpgrade run often.  Like, every 2-3 seconds.  That means the old ShouldSave function was running about, uh, ~200 times per second, on average.  This new SaveForUpgrade function only runs once per query per purchase type (twice every 2-3 seconds), and is much simpler.  So hopefully that'll cut down a bit on AI sim excess for people with slower CPUs.

Haven't had time to fully test SaveForUpgrade's actual functionality, but I have it outputting all relevant info to the log for each run, so it should be easy to watch it manage what each demigod is saving for.  The error it was producing was related to a lack of parentheses I had around some conditional unit.Sync.ItemCostMod accesses - I thought an 'or' would always be processed before math operations, but apparently they're considered in left-to-right order along with high order-of-operation functions (multiplication and division).


Here's an updated set of FindBestBaseItem / FindBestCitadelUpgrade functions for AIShopUtilities.lua: http://pastebin.com/Vt2EdjS6

And the same SaveForUpgrade set I posted before, to replace ShouldSave in AIGlobals.lua: http://pastebin.com/C8pDzaYy

So miri - at this point, you've been testing this in Demigod and are seeing some success, right?  I get you haven't fully tested, but your tests have been in DG and not in theory, right?

I'd like some more feedback from you before i start working on integration.  Thanks!

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:05:09 PM from Demigod Forums Demigod Forums

Yes, I tested both the summon code and the gold save code in-game.  I explained in my post what the error was, and why it was occurring.  I also ran the game long enough to see the AI actually save for and purchase Currency I, and to acquire the Priests save-for at WR 6.

Feel free to tweak the WR levels that I have set in the SaveForUpgrades table, just don't pile too many upgrades onto any one WR / enemy WR level, because each AI will only save for one upgrade at a time right now (that can be modified, but I think it will produce undesirable behavior, much like the current ShouldSave system does now).

I just updated the code on Pastebin to make the Rank variable optional as well, so an upgrade can be saved for based only on ERank or EHas now.  This might be desirable for Priests and Angels.  It'll need some testing either way.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:11:11 PM from Demigod Forums Demigod Forums

Quoting miriyaka,
Yes, I tested both the summon code and the gold save code in-game.  I explained in my post what the error was, and why it was occurring.  I also ran the game long enough to see the AI actually save for and purchase Currency I, and to acquire the Priests save-for at WR 6.

Feel free to tweak the WR levels that I have set in the SaveForUpgrades table, just don't pile too many upgrades onto any one WR / enemy WR level, because each AI will only save for one upgrade at a time right now (that can be modified, but I think it will produce undesirable behavior, much like the current ShouldSave system is now).

I just updated the code on Pastebin to make the Rank variable optional as well, so an upgrade can be saved for based only on ERank or EHas now.  This might be desirable for Priests and Angels.  It'll need some testing either way.

OK good stuff.  Not sure if I'll dig into this tonight or later... I've been busy making silent updates to 0.26.41. 

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:19:40 PM from Demigod Forums Demigod Forums

Oh, I just realized you've probably made changes to those two AIShopUtilities functions, and that the copies I gave you might erase some of those (isn't destructive modding great?).  If you don't want to pick out the changes I made, pastebin an updated copy of your AIShopUtilities.lua and I'll handle integrating the changes.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:34:02 PM from Demigod Forums Demigod Forums

Sure - I'll get setup on pastebin now and copy that over.

getting setup for pastebin took all of 17 seconds... heh.  So here:  http://pastebin.com/wasFe6Nm

Sidenote, miri - Just download the exe of the mod in the OP.  If I get anything working I find interesting, I make an update.  That said, looking forward to see what you've got.  Glad I've got your interest in making this better and developing more efficient code.  Thanks!

 

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:37:06 PM from Demigod Forums Demigod Forums

That's AIAbilityUtilities, which I made (so I obviously have the contents of that).  I need AIShopUtilities.

Edit: I just noticed you posted .41.  If you haven't made any changes to that file since then, I'll just use that.

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:41:36 PM from Demigod Forums Demigod Forums

LISTEN - you give me 2 minutes to correct any post I make when I upload the wrong file or I'll get mad. 

Anyway, its already fixed - let me know if you need something else.  Thanks!

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 10:45:57 PM from Demigod Forums Demigod Forums

TESTERS - Please have a go at the latest version.  Please read the changelog for versions 0.26.40 and 0.26.41 before you play a game.  I'm looking for the usual feedback and, of course, comments letting me know if you are seeing improvement in the flag captures, targetting structures, etc. 

I'm hoping you can provide some feedback tomorrow for me.  Thanks!

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 11:13:10 PM from Demigod Forums Demigod Forums

hmm... I'm still trying to pinpoint the stuck dg error.  Just played a test game with all normal ai's.  The tb on my team got stuck at the shop after dying.  I wonder if this problem will still happen when we move over to miri's system for highgold.  Anyway, the tb was the highgold guy, but he didn't have enough money for cur2.  Then ws8 came around.  He was still frozen, and the occ on my team came back and bought priest (eg currency 2 was overwritten as a priority - which is how I thought it worked).  Anyway, I say hmm... I'll buy cur2 and see what happens.  I get cur2 and the tb snaps back to life about 30 seconds later (he had literally been just standing at the shop for 2 minutes prior...

Then, also interesting, my tb, with 5k gold, went and stood next to the cit.  He just stood there for 30 sec.  Occ game back and bought angel.  TB immedaitely bought cats adn left.  Then I picked up giants.  Sounds like we need some sort of clear goal function or something... anything to wake up an ai that is sitting on its butt for too long. 

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 11:18:28 PM from Demigod Forums Demigod Forums

Oh whoops, I missed an AIGlobals function in that pastebin (GetGoldRank).  Added that.

Full AIGlobals.lua: http://pastebin.com/5YYKWnau
Full AIShopUtilities.lua: http://pastebin.com/Esn55ysp

Reason for Karma (Optional)
Successfully updated karma reason!
June 19, 2011 11:44:00 PM from Demigod Forums Demigod Forums

OK - just made the last of my silent 0.26.41 updates.  Last changes only involved ability build tweaking, so no impact to anything you've been working on, miri.  I'm off to sleep.  If you think of anything else that would be worth mentioning, please do.  If things go according to my plans anyway, I'll start working on integrating your changes tomorrow. 

Though frankly, if you just want to work on your own build, I won't be able to touch this again until after 5:30PM EST tomorrow and all of the changes I've been working on at in 0.26.41.  So, if you want to work on building it out using 0.26.41 as your source, knock yourself out.  Otherwise, I'll take a look after work.  Thx

Reason for Karma (Optional)
Successfully updated karma reason!
Stardock Forums v1.0.0.0    #108433  walnut3   Server Load Time: 00:00:00.0000391   Page Render Time: