Ghom bot v2v2

[tonyv82]

Hello, I tried to use this script, but when it starts, it press the "Start Game", then it kinda search for the waypoint and it finds while it's in the loading screen, so before I'm even in the game I get the message "Can't find yourself on minimap!.....", but the problem is that it "find" the waypoint before it is really ingame, at least that's what I think its happening.

Resolution 1366x768

2 changes in file "Kill Ghom All Resolutions v2.1.au3"

replace line 99 from :
;Startgame()

to
Sleep(4000)


and

replace line 147 from:
Sleep(200)

to
Sleep(4000)

currently working on better fix but this will help if the issue is what you say.

[adong06]

witch doctor running 1920x1080. everything runs perfectly but the bot still wont loot anything after killing ghom.

If my character will stand still after killing ghom, he will loot items. but 99% of the time he runs right past ghom after killing him and the items arent on the screen anymore. i've sent you screenshots in a private message.

Thanks!

[maozao]

2 changes in file "Kill Ghom All Resolutions v2.1.au3"

replace line 99 from :
;Startgame()

to
Sleep(4000)


and

replace line 147 from:
Sleep(200)

to
Sleep(4000)

currently working on better fix but this will help if the issue is what you say.

It worked, but now the problem is this:

It enter the boss room, change to "Boss found", then it uses all spells and attack it once, and change to "Start Looting", something like this, I don't remembert exactly the messages, but it's not really killing the boss, just attacking the spells once and then start to loot and try to leave the game, of course it doesn't work because I'm in a fight, so not sure if something else is not working too.

Thanks.

[tonyv82]

It worked, but now the problem is this:

It enter the boss room, change to "Boss found", then it uses all spells and attack it once, and change to "Start Looting", something like this, I don't remembert exactly the messages, but it's not really killing the boss, just attacking the spells once and then start to loot and try to leave the game, of course it doesn't work because I'm in a fight, so not sure if something else is not working too.

Thanks.

it thinks ghom is dead. i need a screenshot after ghom is dead to see what is wrong, it has to have the quest complete dialog box with the purple ghom and the gold/xp reward.

[maozao]

it thinks ghom is dead. i need a screenshot after ghom is dead to see what is wrong, it has to have the quest complete dialog box with the purple ghom and the gold/xp reward.

Well, I got a monitor with the 1920 resolution, but the game still at 1366 and now it is working good, besides the looting part, it kinda mess up while looting, sometimes it do not loot, sometimes it loot 1~2 itens when there are 5 yellow items on the ground, but looks like its walking before loot, so it clicks in the wrong spot, cuz in some runs it "Start Looting" then move to the east where the boss spawn, then it see the yellow item in the ground near the wall and try to click in the edge of the screen, so looks like just a sync problem.

Thank you again.

Edit: Now it droped like 4 yellow and 1 legendary, it picked up 2 yellow items and move to tp back to the town.

[tonyv82]

Well, I got a monitor with the 1920 resolution, but the game still at 1366 and now it is working good, besides the looting part, it kinda mess up while looting, sometimes it do not loot, sometimes it loot 1~2 itens when there are 5 yellow items on the ground, but looks like its walking before loot, so it clicks in the wrong spot, cuz in some runs it "Start Looting" then move to the east where the boss spawn, then it see the yellow item in the ground near the wall and try to click in the edge of the screen, so looks like just a sync problem.

Thank you again.

Edit: Now it droped like 4 yellow and 1 legendary, it picked up 2 yellow items and move to tp back to the town.

this will take all the screenshots i need, let it run for 2 runs and send me the screens in bots directory:

gb241_debug.rar

[maozao]

PRivate mesage sent

[Sphenoidalis]

Is the multi language support released yet?

Greets
Spheno

[tonyv82]

Is the multi language support released yet?

Greets
Spheno

not yet, instead of checking for text i need to check for image, that means i have to change many things and it takes time. and i have finals now so it takes longer. sorry

[tonyv82]

for all the "not looting" bugs out there, the old debug version took screenshots that didn't have the true color of the items so i couldn't use the screenshots to see what's wrong.

this version takes screenshots with full color so i can find the cause of the issue.

note that the looting screenshots now take about 8MB per screenshot.

gb241_debug.zip

[maozao]

PM Sent with some suggestions too.

=)

[tonyv82]

version 242 is out:

fixed quest select after you die (no longer gets stuck). added yellow colors so that it will pick up all yellow items. some more small fixes.
no non-English language support yet.

is player dead detection in 1920x1080 is match better now. to make it better for all resolutions i need "you are dead" screenshots. a true color screenshot tool will be uploaded in few minutes.

[maozao]

Thank you for the update. While you were "afk" I fixed it myself adding more than 1 yellow color too, I'm using like 10 different yellows and it's working great(Didn't lose ANY yellow items, I just need to drop some legendary to check the color to add to the "color list" if necessary too.

I don't want to sound rude or something like that, but I made some improvements in the code here and feel free to use them in this script if you want to, I guess it will help everyone, but it's your script, so it's up to you hahaha

I will explain everything that I changed, so if you think it's usefull somehow, as I said, feel free to use.

First, I made a array of the yellow colors, so we don't need to add a lot of "If LootItem($yellow1) Then", $yellow2, etc,etc

On Constant files, removed the $yellow variable and created an array and a variable to hold the array size, so we dont need to hardcode it in the GhomRun.au3

Ps: I'm not that good with pixels etc, so I kind checked a lot of yellow colors that might appear in the rare items(aka yellow loot), but it still fast to get them and in rare cases it find one of those colors in the ground when you walk to the middle of the room, I didn't removed it because I guess it give us more random actions in our character to help agains a banishiment.

Global Const $YellowSize = 11 ;Size of the array
Global Const $YellowTable[$YellowSize] = [0xFFFF00,   0xEAEA00,   0xE1E100,    0xEEEE01,     0xE8DC3C,    0xDDDD00,   0xE8E800,   0xE9E900,   0xEBEB00,   0xE5E500,  0xFAFA00]
;                                         255-255-0   234-234-0   225-225-0    238-238-01    232-220-60   221-221-0   232-232-0   233-233-0   235-235-0   229-229-9  250-250-0

Loot() function:

If $pickupYellow = 1 Then
		   if $ResolutionRatio = 3 Then
			  If LootItem($yellow_1680) Then
				  $amarello = $amarello + 1
				  ContinueLoop
			   EndIf
			EndIf
			   ;Changes here
			   For $i = 0 to $YellowSize - 1
				  If LootItem($YellowTable[$i]) Then
					 $amarello = $amarello + 1
					 ContinueLoop
				  EndIf
			   Next
		EndIf

Remember to add a "Return True" before the last line of the function Loot(), to be exactly, before the "EndFunc ;==>Loot"

LootItem() function - Just added a Sleep() if it found the color, because sometimes you need to walk to the item, so it will give the function some time to walk->pickup the loot, not really necessary I guess, but might be better if for some reason you are far away from the item:

If Not @error Then
		MouseClick("left", $coords[0], $coords[1], 1, 10)
		Sleep(Random(1000,2000))
		Return True
	Else

In the main function of the "Kill Ghom" file I also added a variable just to hold the Loot() function return, because most of times looks like the script was executing the Loot() then walking to the middle of the room before the Loot() function ends since it had no return(void method), so it would like Try loot->Walk->Found a loot in the first Loot call->Walk Back->Get crazy and get TP, so it is just to really wait the Loot() function return, wich I made return True just in case(talked about that above):

Main loop

$dis_ent = false
   $loot_ok = false ;NEw variable
   local $kill_flag = False
.
.
If FightBoss() Then
				$TotalTime = $TotalTime + TimerDiff($TimeStart)
				$NumOfKills = $NumOfKills + 1
				Sleep(1000)
				$loot_ok = Loot()
				;Send("{PRINTSCREEN}")
				Sleep(1000)
				Gold()
				Sleep(1000)
				$loot_ok = Loot()
				Sleep(1000)
			Else

Last, but not least, it kind worries me about that spell spam that the script keep doing, 4-3-2-1, 4-3-2-1 all the time where we have some skills with a high cooldown(Following the build in the first post, like Akarant skill that it's 90 seconds(at least here in my character), so I created a cooldown checker based in the last time you casted that spell, as I said I'm not that good with this Pixel thing, I'm still doing some researchs about that, so I'm not checking the cooldown skill color, so the only thing that might happen here is, the script press the skill number(1,2,3 or 4) and for some reason it wasn't casted, so it will wait until the "Cooldown" in the script is good to try cast again, to be honest it didn't happened to me anymore after some adjustments that I made, so if you want to, you can use it and maybe make it less hardcoded too and maybe the possibility to edit it in the script GUI.

SO, let's go. In the GhomFights file, before the Func wizard():

If Not IsDeclared("usepotion") Then $usepotion = 1
If Not IsDeclared("$xratio") Then $xratio = 1
If Not IsDeclared("$yratio") Then $yratio = 1

#include <Date.au3> ;Library needed to get the system time

;The spells CD, might be changed, unfortunately I used the spells name, but it can be changed to a "neutral" name, like $Skill1_CD
Const $AKARANT_CD = 90
Const $JUDGMENT_CD = 20
Const $LAWS_OF_VALOR_CD = 30
Const $HEAVENS_FURY_CD = 20
;An default date to initialize the variables below
Const $Default_Date = "1970/01/01 [00:00:00]" ;Do not change this

;Variables that will hold the last time the spell was used - Set the default value to them in the first execution, didn't find anything to help this "shit code" on AutoIt Documentation
$last_Akarant = $Default_Date
$last_Judgment = $Default_Date
$last_Laws = $Default_Date
$last_Heavens = $Default_Date

I've added two new functions to help us to cleanup the crusader() function and to make it more realistic:

;Returns if we can cast the spell, parameters are, the last time we used the spell and the spell cooldown in seconds
Func canCast($lastCast, $cooldown)
   If(_DateDiff('s', $lastCast, _NowCalc()) > $cooldown) Then
	  Return True
   Else
	  Return False
   EndIf
EndFunc

;Function to verify all the 4 spells cooldown and cast them if you can cast, if you can cast it, it will wait some miliseconds to not spam spells and to avoid the next spell to not be casted, it press the Skill button twice because it's more efective after some tests that I made
Func castCrusaderSpells()
   If(canCast($last_Heavens, $HEAVENS_FURY_CD)) Then
	  $last_Heavens = _NowCalc()
	  Send("4")
	  Sleep(200)
	  Send("4")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Laws, $LAWS_OF_VALOR_CD)) Then
	  $last_Laws = _NowCalc()
	  Send("3")
	  Sleep(200)
	  Send("3")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Judgment, $JUDGMENT_CD)) Then
	  $last_Judgment = _NowCalc()
	  Send("2")
	  Sleep(200)
	  Send("2")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Akarant, $AKARANT_CD)) Then
	  $last_Akarant = _NowCalc()
	  Send("1")
	  Sleep(200)
	  Send("1")
	  Sleep(Random(700, 1000))
   EndIf

   Return True
EndFunc

With the two functions above, we can replace all the lines in the crusader() function like this:

Send("4")
Sleep(200)
Send("3")
Sleep(200)
Send("2")
Sleep(200)
Send("1")

To this simple line:

castCrusaderSpells()

An example how is the begining of my cusader() function:

Func crusader() ;Code by wasssaaa and tonyv
   HotKeySet("{F1}", "_Close")
	;Start Fight
	MouseClick("Left", Round(370 * $x_ratio) + Random(-10, 10), Round(600 * $y_ratio) + Random(-10, 10)) ; Move against wall
	Sleep(200)
	MouseMove(Round(728 * $x_ratio) + Random(-10, 10), Round(280 * $y_ratio) + Random(-10, 10)) ; Point mouse straight
	Sleep(5000)

	castCrusaderSpells()
	Sleep(Random(100,300))

	Send("{SHIFTDOWN}")
	Sleep(200)
	MouseDown("Right")
	$rflag = True
	Sleep(5000)

Also I added a lot of Random() codes, to again help us against a banishiment, so we won't always click in the same X/Y of the screen, anyway, I'm uploading the script I'm using atm with my changes, it's based in the version 241, if you think it's unecessary or abusive of my part I can remove the attachment.

Ps: It do NOT has any improvements from the original version 242 from the first post like the new "You are dead" check, so feel free to use it at your own risk.

Thanks and keep doing the great work : )

[soulo]

what setting is this for? ive been using the old one 236 because the more work the more bugged..

Thank you for the update. While you were "afk" I fixed it myself adding more than 1 yellow color too, I'm using like 10 different yellows and it's working great(Didn't lose ANY yellow items, I just need to drop some legendary to check the color to add to the "color list" if necessary too.

I don't want to sound rude or something like that, but I made some improvements in the code here and feel free to use them in this script if you want to, I guess it will help everyone, but it's your script, so it's up to you hahaha

I will explain everything that I changed, so if you think it's usefull somehow, as I said, feel free to use.

First, I made a array of the yellow colors, so we don't need to add a lot of "If LootItem($yellow1) Then", $yellow2, etc,etc

On Constant files, removed the $yellow variable and created an array and a variable to hold the array size, so we dont need to hardcode it in the GhomRun.au3

Ps: I'm not that good with pixels etc, so I kind checked a lot of yellow colors that might appear in the rare items(aka yellow loot), but it still fast to get them and in rare cases it find one of those colors in the ground when you walk to the middle of the room, I didn't removed it because I guess it give us more random actions in our character to help agains a banishiment.

Global Const $YellowSize = 11 ;Size of the array
Global Const $YellowTable[$YellowSize] = [0xFFFF00,   0xEAEA00,   0xE1E100,    0xEEEE01,     0xE8DC3C,    0xDDDD00,   0xE8E800,   0xE9E900,   0xEBEB00,   0xE5E500,  0xFAFA00]
;                                         255-255-0   234-234-0   225-225-0    238-238-01    232-220-60   221-221-0   232-232-0   233-233-0   235-235-0   229-229-9  250-250-0

Loot() function:

If $pickupYellow = 1 Then
		   if $ResolutionRatio = 3 Then
			  If LootItem($yellow_1680) Then
				  $amarello = $amarello + 1
				  ContinueLoop
			   EndIf
			EndIf
			   ;Changes here
			   For $i = 0 to $YellowSize - 1
				  If LootItem($YellowTable[$i]) Then
					 $amarello = $amarello + 1
					 ContinueLoop
				  EndIf
			   Next
		EndIf

Remember to add a "Return True" before the last line of the function Loot(), to be exactly, before the "EndFunc ;==>Loot"

LootItem() function - Just added a Sleep() if it found the color, because sometimes you need to walk to the item, so it will give the function some time to walk->pickup the loot, not really necessary I guess, but might be better if for some reason you are far away from the item:

If Not @error Then
		MouseClick("left", $coords[0], $coords[1], 1, 10)
		Sleep(Random(1000,2000))
		Return True
	Else

In the main function of the "Kill Ghom" file I also added a variable just to hold the Loot() function return, because most of times looks like the script was executing the Loot() then walking to the middle of the room before the Loot() function ends since it had no return(void method), so it would like Try loot->Walk->Found a loot in the first Loot call->Walk Back->Get crazy and get TP, so it is just to really wait the Loot() function return, wich I made return True just in case(talked about that above):

Main loop

$dis_ent = false
   $loot_ok = false ;NEw variable
   local $kill_flag = False
.
.
If FightBoss() Then
				$TotalTime = $TotalTime + TimerDiff($TimeStart)
				$NumOfKills = $NumOfKills + 1
				Sleep(1000)
				$loot_ok = Loot()
				;Send("{PRINTSCREEN}")
				Sleep(1000)
				Gold()
				Sleep(1000)
				$loot_ok = Loot()
				Sleep(1000)
			Else

Last, but not least, it kind worries me about that spell spam that the script keep doing, 4-3-2-1, 4-3-2-1 all the time where we have some skills with a high cooldown(Following the build in the first post, like Akarant skill that it's 90 seconds(at least here in my character), so I created a cooldown checker based in the last time you casted that spell, as I said I'm not that good with this Pixel thing, I'm still doing some researchs about that, so I'm not checking the cooldown skill color, so the only thing that might happen here is, the script press the skill number(1,2,3 or 4) and for some reason it wasn't casted, so it will wait until the "Cooldown" in the script is good to try cast again, to be honest it didn't happened to me anymore after some adjustments that I made, so if you want to, you can use it and maybe make it less hardcoded too and maybe the possibility to edit it in the script GUI.

SO, let's go. In the GhomFights file, before the Func wizard():

If Not IsDeclared("usepotion") Then $usepotion = 1
If Not IsDeclared("$xratio") Then $xratio = 1
If Not IsDeclared("$yratio") Then $yratio = 1

#include <Date.au3> ;Library needed to get the system time

;The spells CD, might be changed, unfortunately I used the spells name, but it can be changed to a "neutral" name, like $Skill1_CD
Const $AKARANT_CD = 90
Const $JUDGMENT_CD = 20
Const $LAWS_OF_VALOR_CD = 30
Const $HEAVENS_FURY_CD = 20
;An default date to initialize the variables below
Const $Default_Date = "1970/01/01 [00:00:00]" ;Do not change this

;Variables that will hold the last time the spell was used - Set the default value to them in the first execution, didn't find anything to help this "shit code" on AutoIt Documentation
$last_Akarant = $Default_Date
$last_Judgment = $Default_Date
$last_Laws = $Default_Date
$last_Heavens = $Default_Date

I've added two new functions to help us to cleanup the crusader() function and to make it more realistic:

;Returns if we can cast the spell, parameters are, the last time we used the spell and the spell cooldown in seconds
Func canCast($lastCast, $cooldown)
   If(_DateDiff('s', $lastCast, _NowCalc()) > $cooldown) Then
	  Return True
   Else
	  Return False
   EndIf
EndFunc

;Function to verify all the 4 spells cooldown and cast them if you can cast, if you can cast it, it will wait some miliseconds to not spam spells and to avoid the next spell to not be casted, it press the Skill button twice because it's more efective after some tests that I made
Func castCrusaderSpells()
   If(canCast($last_Heavens, $HEAVENS_FURY_CD)) Then
	  $last_Heavens = _NowCalc()
	  Send("4")
	  Sleep(200)
	  Send("4")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Laws, $LAWS_OF_VALOR_CD)) Then
	  $last_Laws = _NowCalc()
	  Send("3")
	  Sleep(200)
	  Send("3")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Judgment, $JUDGMENT_CD)) Then
	  $last_Judgment = _NowCalc()
	  Send("2")
	  Sleep(200)
	  Send("2")
	  Sleep(Random(700, 1000))
   EndIf


   If(canCast($last_Akarant, $AKARANT_CD)) Then
	  $last_Akarant = _NowCalc()
	  Send("1")
	  Sleep(200)
	  Send("1")
	  Sleep(Random(700, 1000))
   EndIf

   Return True
EndFunc

With the two functions above, we can replace all the lines in the crusader() function like this:

Send("4")
Sleep(200)
Send("3")
Sleep(200)
Send("2")
Sleep(200)
Send("1")

To this simple line:

castCrusaderSpells()

An example how is the begining of my cusader() function:

Func crusader() ;Code by wasssaaa and tonyv
   HotKeySet("{F1}", "_Close")
	;Start Fight
	MouseClick("Left", Round(370 * $x_ratio) + Random(-10, 10), Round(600 * $y_ratio) + Random(-10, 10)) ; Move against wall
	Sleep(200)
	MouseMove(Round(728 * $x_ratio) + Random(-10, 10), Round(280 * $y_ratio) + Random(-10, 10)) ; Point mouse straight
	Sleep(5000)

	castCrusaderSpells()
	Sleep(Random(100,300))

	Send("{SHIFTDOWN}")
	Sleep(200)
	MouseDown("Right")
	$rflag = True
	Sleep(5000)

Also I added a lot of Random() codes, to again help us against a banishiment, so we won't always click in the same X/Y of the screen, anyway, I'm uploading the script I'm using atm with my changes, it's based in the version 241, if you think it's unecessary or abusive of my part I can remove the attachment.

Ps: It do NOT has any improvements from the original version 242 from the first post like the new "You are dead" check, so feel free to use it at your own risk.

Thanks and keep doing the great work : )

[tonyv82]

what setting is this for? ive been using the old one 236 because the more work the more bugged..

what doesn't work for you in 242 but works in 236?