PixelGetColor makes comparing time with random delay all the time

[SkyDancer]

Const $pixel_check_x = 1030
Const $pixel_check_y = 409
Const $pixel_check_color = 13093063
Const $send_delay_ms = 520


Do
    $pixel = PixelGetColor($pixel_check_x, $pixel_check_y)
Until $pixel = $pixel_check_color

Sleep($send_delay_ms)

Send("{SPACE}")

Exit

I have this code but delay is always random time after color matched, how to make color waiting with a static time then make some static delay and then pres Space button? Thank you.

[Nine]

How do you know it is random ? Sleep () works perfectly.  It is your loop that takes a random time to find the right color. IMO.

[SkyDancer]

5 minutes ago, Nine said:

It is your loop that takes a random time to find the right color. IMO.

Exectly. I need to search color with a static time. Is it posible to do it in AutoIt?

[Nine]

yes, just remove the loop, it will be static...

[SkyDancer]

But how to wait for specific lolor then? I need to wait till one pixel on the screen changed the color then I need static delay and then Press Space button. So the basic question is how to wait for the pixel color changet with a static time, the pixel position is always static so it has to be static time for search it.

[Nine]

4 minutes ago, SkyDancer said:

I need to wait till one pixel on the screen changed the color

If that change happens randomly, it will be random, you cannot have both.  And may I ask what kind of application you are trying to automate ?  Could help if we know what you are attempting to do 

[SkyDancer]

Im trying to make a unit test for my own web application. And I know the pixel color time is random but after pixel changed, from this moment next steps have to be with a static time but seems like the time for pixel search is random, but it has to be static since the pixel coordinates are always the same or I understand pixel search algorithm wrong?

[Nine]

Try this :

Const $pixel_check_x = 1030
Const $pixel_check_y = 409
Const $pixel_check_color = 13093063
Const $send_delay_ms = 520

$timer = TimerInit ()
Do
    $pixel = PixelGetColor($pixel_check_x, $pixel_check_y)
Until $pixel = $pixel_check_color

Sleep($send_delay_ms - TimerDiff($Timer))

Send("{SPACE}")

It will be static unless the pixel doesn't show before end of delay.  But you can manage that, right ?

[badcoder123]

Made this function and modified it

Func _ColourReturn($x, $y, $colour)
    If PixelGetColor($x, $y, $hWnd) = $colour Then Return True
    If PixelGetColor($x, $y, $hWnd) <> $colour Then Return False
EndFunc

Try this out. Just set a sleep delay in the do loop

 

Const $pixel_check_x = 1030
Const $pixel_check_y = 409
Const $pixel_check_color = 13093063
Const $send_delay_ms = 520
Const $iStaticSleep = xxxxxxxxxxxxxxxx


Do
    Sleep($iStaticSleep)
Until _ColourReturn($pixel_check_x, pixel_check_y, $pixel_check_color)

Sleep($send_delay_ms)

Send("{SPACE}")

Exit

Func _ColourReturn($x, $y, $colour)
    If PixelGetColor($x, $y) = $colour Then Return True
    If PixelGetColor($x, $y) <> $colour Then Return False
EndFunc

 

[careca]

It'll just loop until the color is found, just like the code in the first post, they seem the same to me.
 

Quote

 

from this moment next steps have to be with a static time but seems like the time for pixel search is random,

but it has to be static since the pixel coordinates are always the same

 

Well, in theory yes, since you give the coordinates, it should take the same time, every time, but probably because of the cpu load or other factors, there may be a small random delay each time, you have to code counting with it.

[SkyDancer]

18 hours ago, careca said:

but probably because of the cpu load or other factors, there may be a small random delay each time, you have to code counting with it.

That is not a small rng time, its around 50-200ms, can it be cuz I run script from IDE? Cuz tbh I didn't try to compile and use it as exe script file...

[SkyDancer]

nwm. Isn't relevent anymore