The loading screen won't apear.

[powerattack]

hey,

Im currently farming KS with YAKKS script (customized it a bit for my warden), it runs fine but lately it randomly stops at the revival point and it keeps trying to leave party.

Executing RoMScript "LeaveParty();".
Executing RoMScript "GetSkillCooldown(1,9);".
The loading screen didn't appear...

I will post a screen next time I get it. But since it keeps looping with leave party and loading screen wont apear, I can't see what happend before this.

Any known problem? couldn't find it on the forum.

My best gues is that it loads to fast and still thinks its in KS and needs to get out?

Thanks in advance

[powerattack]

Ive changed:

--don't reset if we are already in DDC
local zone = getZoneId();
if (zone ~= 6) then
player:clearTarget();
sendMacro("LeaveParty();");
waitForLoadingScreen(30);

to 20 and now trying if it will help, but it usually takes a few hrs to get stuck...

[rock5]

What the number in waitForLoadingScreen means is, how long it waits for the loading screen to appear before returning false. So for example, say the loading screen appears after 5 seconds, then it stops waiting for the loading screen to appear and instead waits for the it to disappear however long it takes. So I don't see changing it to 20 will make any difference.

[powerattack]

Any idea how I can fix the issue?
this is the exact error code

Code: Select all

Cleared bags: 0 spaces free
Resetting instance
Use MACRO: Executing RoMScript "LeaveParty();".
Use MACRO: Executing RoMScript "GetSkillCooldown(1,9);".
The loading screen didn't appear...
Use MACRO: Executing RoMScript "LeaveParty();".
Use MACRO: Executing RoMScript "GetSkillCooldown(1,9);".

then it just keeps going with leave party and get skill cooldown

I was thinking about removing this section from the script(checking if it worked in the morning):

Code: Select all

	else
		--we have already left KS and are in DC, something went wrong somewhere, drop party and recall to logar and re-run in
		--recall to logar and run back!!!!
		repeat
			sendMacro("LeaveParty();");
			local cooldown, remaining = sendMacro("GetSkillCooldown(1,9);"); --this is the index of Transport: Logar spell for me - wait for it to come off cooldown
			while (remaining > 1) do
				cooldown, remaining = sendMacro("GetSkillCooldown(1,9);")
				yrest(5000);
			end;
			RoMScript('CastSpellByName("Transport: Logar")');
			waitForLoadingScreen(30); 
			player:update();
			zone = getZoneId();
		until (zone == 1);
		
		loadPaths(KS_ReturnFromTransportWaypoint); -- this is a waypoint that will talk to my housekeepers, get teaching certs and run back to KS
		return;
	end;

[rock5]

Firstly I suggest you use

Code: Select all

cooldown, remaining = sendMacro("GetSkillCooldown(1,2);")

That's the cooldown for Recall which shares it's cooldown with the other transport skills. If Recall is not on cooldown then you are free to use "Transport: Logar". And Recall is always in the same place.

So if I understand correctly you are in DC and the transport function is failing. Did you try the

Code: Select all

CastSpellByName("Transport: Logar")

in game to make sure it worked? I remember there used to be an issue of different servers using different wording for the transport skills.

[Bill D Cat]

The strings are defined as follows in the current game version:

Code: Select all

Sys540001_name="Recall"
Sys540190_name="Transport: Logar"
Sys540191_name="Transport: Reifort"
Sys540192_name="Transport: Valley of Preparation"
Sys540193_name="Transport: Heffner Camp"
Sys540195_name="Morfantas City Teleportation"

So I'm thinking that using something like this would work fine regardless of language.

Code: Select all

RoMScript("CastSpellByName(\""..getTEXT("Sys540190_name").."\")")

But would doing something like this actually make it a whole lot easier?

Suggestion for skills.xml additions.

Code: Select all

	<skill name="RECALL"								id="540001"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />
	<skill name="TRANSPORT_LOGAR"						id="540190"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />
	<skill name="TRANSPORT_REIFORT"						id="540191"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />
	<skill name="TRANSPORT_VALLEY_OF_PREPARATION"			id="540192"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />
	<skill name="TRANSPORT_HEFFNER_CAMP"				id="540193"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />
	<skill name="MORFANTAS_CITY_TELEPORTATION"			id="540195"	range="0"	type="transport"	casttime="3"	cooldown="3600"	target="self" />

[rock5]

The strings are defined as follows in the current game version:

Code: Select all

Sys540001_name="Recall"
Sys540190_name="Transport: Logar"
Sys540191_name="Transport: Reifort"
Sys540192_name="Transport: Valley of Preparation"
Sys540193_name="Transport: Heffner Camp"
Sys540195_name="Morfantas City Teleportation"

Yes, but I seem to remember an issue where people would say they see, for instance, "Transport: Logar" but that didn't work. They had to use some other string, I don't remember what. So I still think it should be tested to make sure that it works.

But would doing something like this actually make it a whole lot easier?

Suggestion for skills.xml additions.

Code: Select all

   <skill name="RECALL"                        id="540001"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />
   <skill name="TRANSPORT_LOGAR"                  id="540190"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />
   <skill name="TRANSPORT_REIFORT"                  id="540191"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />
   <skill name="TRANSPORT_VALLEY_OF_PREPARATION"         id="540192"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />
   <skill name="TRANSPORT_HEFFNER_CAMP"            id="540193"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />
   <skill name="MORFANTAS_CITY_TELEPORTATION"         id="540195"   range="0"   type="transport"   casttime="3"   cooldown="3600"   target="self" />	

Not really. The purpose of the skills file is really for the skills the bot uses automatically. The transport skills would still have to be cast manually in the script by the user. So it will be either

Code: Select all

player:cast("TRANSPORT_LOGAR")

or

Code: Select all

RoMScript('CastSpellByName("Transport: Logar")')

You could just as easily create a custom function for tranporting if you really wanted to make it a little easier.

[BlubBlab]

I didn't wanted to spoil thinks but thats the reason I created that:
http://www.solarstrike.net/phpBB3/viewt ... =27&t=5163
(I had the same discussion back then with rock before I created it)

[Bill D Cat]

Custom functions are my specialty.

Code: Select all

function recall(_location)
	local _cooldown, _remaining = sendMacro("GetSkillCooldown(1,2)")
	if _remaining <=1 then
		if _location == nil then 
			_location = getTEXT("Sys540001_name")
		elseif string.lower(_location) == "logar" then
			_location = getTEXT("Sys540190_name")
		elseif string.lower(_location) == "reifort" then
			_location = getTEXT("Sys540191_name")
		elseif string.lower(_location) == "heffner" then
			_location = getTEXT("Sys540193_name")
		elseif string.lower(_location) == "morfantas" then
			_location = getTEXT("Sys540195_name")
		else
			_location = getTEXT("Sys540001_name")
		end
		RoMScript("CastSpellByName(\"".._location.."\")")
	else
		print("Transport skills are still on cooldown!")
	end
end

[rock5]

Yes, that's an existing solution.

Edit: Ah, 2 solutions.