BenLubar / df-ai

Dwarf Fortress + Artificial Intelligence
https://benlubar.github.io/df-ai/
zlib License
248 stars 26 forks source link

Lockstep enabled prevents trade #34

Closed mattregul closed 6 years ago

mattregul commented 6 years ago

DF & AI will just wait at the trade window until ai disable lockstep is preformed

BenLubar commented 6 years ago

Just so I'm sure I'm checking the right version, what does ai version say?

mattregul commented 6 years ago

Dwarf Fortress 0.43.05 Windows 64-bit df-ai 0.43.05-r2-54-ged61dab4 code ed61dab468ef93c174a9bc3131c5fcf6bc5aa15e DFHack 0.43.05-r2-48-g0ee421a9 library 0ee421a9d40881a05aa5b382637148bdbe66ecb9 structures 3322beb2e7f4b28ff8e573e9bec738c77026b8e9

BenLubar commented 6 years ago

Hmm, do you have any tweak commands in your dfhack.init? If possible, could you post your dfhack.init here?

mattregul commented 6 years ago

dfhack.init - https://gist.github.com/mattregul/3db77f1d815496bafc675abb788d7113#file-dfhack-init my launcher - https://gist.github.com/mattregul/3db77f1d815496bafc675abb788d7113#gistcomment-2240134

mattregul commented 6 years ago

I do have this in my onMapLoad_PeridexisErrant.init repeat -time 1 -timeUnits months -command [ quicksave ] -name autosave Quicksaving every month to help recover from crashes

BenLubar commented 6 years ago

Hmm, lots of tweaks in there. I'll look into it later today.

BenLubar commented 6 years ago

I wasn't able to reproduce a freeze when trading with lockstep enabled, but I did notice that the default onLoad.init-example that comes with DFHack causes some dialogue boxes to appear when dwarves are thirsty or stuck in trees. Are there any messages in the DFHack console related to an unexpected viewscreen or a pause of some kind?

mattregul commented 6 years ago

Two items:

  1. I updated to the latest version of df-ai, now I don't seem to be hanging on trade now. Either its fixed, or its intermittent. Maybe move the bug to a "monitoring" state?
  2. Unsure, as I found the "thirsty" dialog annoying, so I removed the command for onLoad after the first time seeing it.
fadedDexofan commented 6 years ago

All works fine on latest builds with enabled lockstep