SRL Archive - MSI Bugs

Please get the files in the 2nd post.

Redownloaded MSI and I only added the walking, did not change one thing. Tested, same error

I really don't feel like picking out the code I added, so I just uploaded each file :) The walking_reflection is not actually needed, but to test, you have to use that (as far as I know :))

the first think i notice is that in your global.simba you didnt change the highest walk number after adding your

WALK_MINE_DB_WEST_1 = 12;
  WALK_MINE_DB_WEST_2 = 13;
  WALK_MINE_DB_WEST_3 = 14;
  WALK_MINE_DB_WEST_4 = 15;
  WALK_MINE_DB_WEST_5 = 16;
  WALK_MINE_DB_WEST_6 = 17;
  WALK_MINE_DB_WEST_7 = 18;
  WALK_MINE_DB_WEST_8 = 19;

  WALK_MINE_DB_WEST_BANK_1 = 20;
  WALK_MINE_DB_WEST_BANK_2 = 21;
  WALK_MINE_DB_WEST_BANK_3 = 22;
  WALK_MINE_DB_WEST_BANK_4 = 23;
  WALK_MINE_DB_WEST_BANK_5 = 24;
  WALK_MINE_DB_WEST_BANK_6 = 25;
  WALK_MINE_DB_WEST_BANK_7 = 26;

(**
 * All the different indexes for MSI_WalkInfo global variable; each index is
 * used in MSI_WalkTPA by 3Garrett3 in Walking_Color.simba
 *)
const
  WALK_VE_MINE_1 = 10;
  WALK_VE_MINE_2 = 11;

  WALK_ARRAY_LENGTH = 11; // Change to the highest walk constant [COLOR="Red"]HERE[/COLOR] need to change to 26 as thats the highest number now

also think about adding fail-safe walking for each of your walks.

From: bobzilla69
the first think i notice is that in your global.simba you didnt change the highest walk number after adding your

WALK_MINE_DB_WEST_1 = 12;
  WALK_MINE_DB_WEST_2 = 13;
  WALK_MINE_DB_WEST_3 = 14;
  WALK_MINE_DB_WEST_4 = 15;
  WALK_MINE_DB_WEST_5 = 16;
  WALK_MINE_DB_WEST_6 = 17;
  WALK_MINE_DB_WEST_7 = 18;
  WALK_MINE_DB_WEST_8 = 19;

  WALK_MINE_DB_WEST_BANK_1 = 20;
  WALK_MINE_DB_WEST_BANK_2 = 21;
  WALK_MINE_DB_WEST_BANK_3 = 22;
  WALK_MINE_DB_WEST_BANK_4 = 23;
  WALK_MINE_DB_WEST_BANK_5 = 24;
  WALK_MINE_DB_WEST_BANK_6 = 25;
  WALK_MINE_DB_WEST_BANK_7 = 26;

(**
 * All the different indexes for MSI_WalkInfo global variable; each index is
 * used in MSI_WalkTPA by 3Garrett3 in Walking_Color.simba
 *)
const
  WALK_VE_MINE_1 = 10;
  WALK_VE_MINE_2 = 11;

  WALK_ARRAY_LENGTH = 11; // Change to the highest walk constant [COLOR="Red"]HERE[/COLOR] need to change to 26 as thats the highest number now

also think about adding fail-safe walking for each of your walks.

Oh yea, my bad. Also, please download the files in my second post.

After you redownloaded MSI, did you still comment out the stuff in MSI_PerformWalk in Walking_Reflection.simba? If you did, don't. Comment out the line that includes reflection in Setup.simba. That way it won't use reflection, and nothing needs to be changed in MSI_PerformWalk.

From: Coh3n
After you redownloaded MSI, did you still comment out the stuff in MSI_PerformWalk in Walking_Reflection.simba? If you did, don't. Comment out the line that includes reflection in Setup.simba. That way it won't use reflection, and nothing needs to be changed in MSI_PerformWalk.

When I started fresh on the new MSI, I had added everything and then just commented out the reflection include line, which gave me an unknown Identifer error, but I think my problem may be the fact that I didn't do what bobzilla said.

the easier way to run it with no reflection is at the bottom of the runme.simba there is a section that checks reflection

{$IFDEF REFLECTION}
    // So if the hooks are outdated, the script will attempt colour walking if the points exist
    if (not MSI_SetupReflection(False)) then
      ReflectionBroken := True;
  {$ELSE}
    ReflectionBroken := True; // Set to true because it's not even being used
  {$ENDIF}

change this to

{$IFDEF REFLECTION}
    // So if the hooks are outdated, the script will attempt colour walking if the points exist
    if ( MSI_SetupReflection(False)) then
      ReflectionBroken := True;
  {$ELSE}
    ReflectionBroken := True; // Set to true because it's not even being used
  {$ENDIF}

and it will report reflection as broken and use color :)

Oh wow, thanks :)

What was I thinking, I found the problem, it was the fact that I didn't know there was a RS_onMinimap function :)

Walking to the mine is flawless, with even updated methods with no stupid s**t like I had before. On the other hand I tried to update walking to the bank and <->ed up, so now I have to rewrite that. Despite that, I am pretty happy that I fixed the bug. :)

Awesome, glad to hear it. :) When you think it's ready to commit, can you post it on this thread, please?

From: Coh3n
Awesome, glad to hear it. :) When you think it's ready to commit, can you post it on this thread, please?

You got it!