Tab 21 & 27

Is it just me or are these broken?
Using this:

Simba Code:

program new;
{$i SRL/SRL.scar}

procedure Test;
var
  I : Integer;
begin
  for I := 21 to 37 do
  begin
    if (TabExists(I)) then
      GameTab(I);
  end;
end;

begin
  SetupSRL;
  ActivateClient;
  Wait(1500);
  Test;
end.

GameTab(21); works fine in my script

prayer tab right?

21 = Combat and 27 = prayer. They refuse to work for me for some reason.

Quote:

---

Originally Posted by grats

GameTab(21); works fine in my script

prayer tab right?

---

oh yea my bad, I use both those in my script (strange that the only two I use in my script are the ones that give you problems)

I don't have it check if the tab exists though, maybe tab exists is having problems with the combat & prayer tabs?


try without the if statement

Well try running this and see if it works for you, please:

Simba Code:

program new;
{$i SRL/SRL.scar}

procedure Test;
var
  I : Integer;
begin
  for I := 21 to 37 do
  begin
    if (GameTab(I)) then
    begin
      WriteLn('Tab ' + ToStr(I) + ' exists.');
    end else
      WriteLn('Tab ' + ToStr(I) + ' does not exist.');

    Wait(150);
  end;
end;

begin
  SetupSRL;
  ActivateClient;
  Wait(1500);
  Test;
end.

Set your client to Runescape, of course. Also see if you have the latest SRL, please.

yea worked fine for me, all the way through the tabs

Progress Report:

SRL Compiled in 62 msec
Tab 21 exists.
Tab 22 exists.
Tab 23 exists.
Tab 24 exists.
Tab 25 exists.
Tab 26 exists.
Tab 27 does not exist.
Tab 28 exists.
Tab 29 does not exist.
Tab 30 exists.
Tab 31 exists.
Tab 32 exists.
Tab 33 exists.
Tab 34 exists.
Tab 35 exists.
Tab 36 exists.
Tab 37 exists.
Successfully executed.

it clicked all the tabs though

I probably need to add a small wait at the end of the for loop.

Is your SRL updated?

Quote:

---

Originally Posted by RISK

I probably need to add a small wait at the end of the for loop.

Is your SRL updated?

---

yea all my stuff is fully updated

How strange... What's your TabExists like? If you don't mind. Here's mine:

Simba Code:

function TabExists(Tab: Integer): Boolean;
var
  tP: TPoint;
begin
  Result := false;
  if (not loggedIn) then
    exit;

  if (tab < 20) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not using the new constants! Please upgrade your script.', warn_AllVersions);
    Exit;
  end;

  if (not(InRange(Tab, Tab_Combat, Tab_LogOut))) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not a valid tab number.', warn_AllVersions);
    Exit;
  end;

  if (Tab = tab_LogOut) then
  begin
    Result := (GetColor(746, 0) = 65536);
    Exit;
  end;

  tP := Point(((Tab - 21) mod 8 * 30) + 537, ((Tab - 21) div 8 * 298) + 186);
  Result := CountColorTolerance(5597817, tp.x-12, tp.y-12, tp.x+12, tp.y+12, 10) < 81; // the count when no icons are overlapping the GameTab.
end;

Simba Code:

{*******************************************************************************
function TabExists(Tab: Integer): Boolean;
By: ZephyrsFury & IceFire908; fixed by Zyt3x
Description: Returns True if the Tab exists.
*******************************************************************************}
function TabExists(Tab: Integer): Boolean;
var
  tP: TPoint;
begin
  Result := false;
  if (not loggedIn) then
    exit;

  if (tab < 20) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not using the new constants! Please upgrade your script.', warn_AllVersions);
    Exit;
  end;

  if (not(InRange(Tab, Tab_Combat, Tab_LogOut))) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not a valid tab number.', warn_AllVersions);
    Exit;
  end;

  if (Tab = tab_LogOut) then
  begin
    Result := (GetColor(746, 0) = 65536);
    Exit;
  end;

  tP := Point(((Tab - 21) mod 8 * 30) + 537, ((Tab - 21) div 8 * 298) + 186);
  Result := CountColorTolerance(5597817, tp.x-12, tp.y-12, tp.x+12, tp.y+12, 10) < 81; // the count when no icons are overlapping the GameTab.
end;

both identical

Could you add this at the end of the function and test it again?
WriteLn(CountColorTolerance(5597817, tp.x-12, tp.y-12, tp.x+12, tp.y+12, 10));

Code:

---

Compiled successfully in 562 ms.
SRL Compiled in 15 msec
92
Tab 21 does not exist.
7
Tab 22 exists.
43
Tab 23 exists.
54
Tab 24 exists.
51
Tab 25 exists.
78
Tab 26 exists.
81
Tab 27 does not exist.
43
Tab 28 exists.
81
Tab 29 does not exist.
55
Tab 30 exists.
65
Tab 31 exists.
66
Tab 32 exists.
61
Tab 33 exists.
42
Tab 34 exists.
53
Tab 35 exists.
74
Tab 36 exists.
Tab 37 exists.
Successfully executed.

---

Added the WriteLn as you requested.

Quote:

---

Originally Posted by Zyt3x

Could you add this at the end of the function and test it again?
WriteLn(CountColorTolerance(5597817, tp.x-12, tp.y-12, tp.x+12, tp.y+12, 10));

---

Simba Code:

Compiled successfully in 1155 ms.
SRL Compiled in 47 msec
56
Tab 21 exists.
53
Tab 22 exists.
45
Tab 23 exists.
54
Tab 24 exists.
51
Tab 25 exists.
78
Tab 26 exists.
81
Tab 27 does not exist.
43
Tab 28 exists.
81
Tab 29 does not exist.
55
Tab 30 exists.
65
Tab 31 exists.
66
Tab 32 exists.
61
Tab 33 exists.
42
Tab 34 exists.
53
Tab 35 exists.
74
Tab 36 exists.
Tab 37 exists.
Successfully executed.

Easy fix, change "< 81" to "<= 81".

RISK, are you sure all your settings are correct? Highest brightness, fixed size and 32-bit color? If you are 100% sure about those things, then maybe JaGeX has started dynamically changing tab colors too, now....

Could you guys do some testing? reload RuneScape, reload worlds and so on, see if the color changes (or the number before "tab X exists.")
I'm at school atm so I can't test.. :/

Here's my settings:
http://i.imgur.com/rZ5xm.png

I'll gladly test it, hold on.

E: First world I go to, it decides to work:

Code:

---

Compiled successfully in 546 ms.
SRL Compiled in 16 msec
56
Tab 21 exists.
53
Tab 22 exists.
45
Tab 23 exists.
54
Tab 24 exists.
51
Tab 25 exists.
78
Tab 26 exists.
81
Tab 27 exists.
66
Tab 28 exists.
81
Tab 29 does not exist.
55
Tab 30 exists.
65
Tab 31 exists.
66
Tab 32 exists.
61
Tab 33 exists.
42
Tab 34 exists.
53
Tab 35 exists.
74
Tab 36 exists.
Tab 37 exists.
Successfully executed.

---

E2: I changed my max screen size to "Any" before I tested and it worked... Maybe it really was my settings? If so, I'm sorry. :duh:

E3: I tried it on a different character and it gave me this:

Code:

---

Compiled successfully in 546 ms.
SRL Compiled in 16 msec
92
Tab 21 does not exist.
7
Tab 22 exists.
43
Tab 23 exists.
54
Tab 24 exists.
51
Tab 25 exists.
78
Tab 26 exists.
81
Tab 27 exists.
66
Tab 28 exists.
81
Tab 29 does not exist.
55
Tab 30 exists.
65
Tab 31 exists.
66
Tab 32 exists.
61
Tab 33 exists.
42
Tab 34 exists.
53
Tab 35 exists.
74
Tab 36 exists.
Tab 37 exists.
Successfully executed.

---

Quote:

---

Originally Posted by Zyt3x

Easy fix, change "< 81" to "<= 81".

RISK, are you sure all your settings are correct? Highest brightness, fixed size and 32-bit color? If you are 100% sure about those things, then maybe JaGeX has started dynamically changing tab colors too, now....

Could you guys do some testing? reload RuneScape, reload worlds and so on, see if the color changes (or the number before "tab X exists.")
I'm at school atm so I can't test.. :/

---

identical settings except my max size is 800x600 though, only difference

works the same for me on all worlds, gotta go to work gl trying to figure this out, might be able to sneak in while at work

Interest piece of information... The character it works on has been made for a while and has skills trained. The character it does not work on has not been trained and was made under 2 hours ago for HLR.

Maybe that has something to do with it?

It doesn't really look dynamic to me, but I'm worried about that
"92
Tab 21 does not exist." ...

Was there any difference between the tabs? Maybe we only have to change the colorcount number...

Trained account's tabs:
http://i.imgur.com/YR4vH.png

Non-trained account's tabs:
http://i.imgur.com/Hu6N9.png

It looks like the glow from the task list tab is confusing it.

E: Actually the trained account's whole tab interface looks darker than the non-trained account's. Or maybe it's my half-asleep imagination.

Quote:

---

Originally Posted by RISK

Trained account's tabs:
[picture was here]

Non-trained account's tabs:
[picture was here]

It looks like the glow from the task list tab is confusing it.

E: Actually the trained account's whole tab interface looks darker than the non-trained account's. Or maybe it's my half-asleep imagination.

---

Compared the brightest shade of yellow of the clan icon with both pictures....No change.

Quote:

---

Originally Posted by RISK

E: Actually the trained account's whole tab interface looks darker than the non-trained account's. Or maybe it's my half-asleep imagination.

---

Yes, I see this too.

We'll have to re-do TabExists() (and perhaps some other functions)
Unfortunately I do not have any time to do this at all until ~Tuesday next week (5 days from now) :/

You could try to fix them yourself? :) You'd get your name in the include and most importantly you'll get more respect from the higher boards ;)

Sure, I'll have a go! Seems like a fun challenge! :)

TabExists fix:

Simba Code:

{*******************************************************************************
function TabExists(Tab: Integer): Boolean;
By: ZephyrsFury & IceFire908; fixed by Zyt3x; fixed again by RISK
Description: Returns True if the Tab exists.
*******************************************************************************}
function TabExists(Tab: Integer): Boolean;
var
  tP: TPoint;
begin
  Result := false;
  if (not loggedIn) then
    exit;

  if (tab < 20) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not using the new constants! Please upgrade your script.', warn_AllVersions);
    Exit;
  end;

  if (not(InRange(Tab, Tab_Combat, Tab_LogOut))) then
  begin
    SRL_Warn('TabExists', 'Tab ' + IntToStr(Tab) + ' is not a valid tab number.', warn_AllVersions);
    Exit;
  end;

  if (Tab = tab_LogOut) then
  begin
    Result := (GetColor(746, 0) = 65536);
    Exit;
  end;

  tP := Point(((Tab - 21) mod 8 * 30) + 537, ((Tab - 21) div 8 * 298) + 186);
  Result := CountColorTolerance(5597817, tp.x-12, tp.y-12, tp.x+12, tp.y+12, 10) <= 92; // the count when no icons are overlapping the GameTab.
end;

Code:

---

Compiled successfully in 531 ms.
SRL Compiled in 16 msec
Tab 21 exists.
Tab 22 exists.
Tab 23 exists.
Tab 24 exists.
Tab 25 exists.
Tab 26 exists.
Tab 27 exists.
Tab 28 exists.
Tab 29 exists.
Tab 30 exists.
Tab 31 exists.
Tab 32 exists.
Tab 33 exists.
Tab 34 exists.
Tab 35 exists.
Tab 36 exists.
Tab 37 exists.
Successfully executed.

---

Good work RISK :)

Thank you, sir. :) It means a lot.

Have you tested this on tabs where there is no icon?
I found the number 81 (The number that were there before) by counting the pixels of that color with that tolerance when no icons were on top of it. I.E that number will always be lower when an icon is on top of it so that why the < 81 is there.

So just increasing that number will often give false positives, which we do not want