For changelogs and discussion related to a specific revision.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#41
Post
by rock5 » Sat Feb 26, 2011 6:13 am
Alkaiser wrote:SCOUT_CONCENTRATE is missing from skills_local.xml
Is that the one that is instant cast, cooldown 5m and focus regeneration speed is increased by 100%?
And I guess it should only be used in battle.
Edit: Sorry, found it. It was misnamed. I'll add the correct name and add it to skills_local. Should I add 'inbattle=true' so it only uses it in battle or should I leave it to the user.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Alkaiser
- Posts: 222
- Joined: Sat Sep 25, 2010 2:03 pm
#42
Post
by Alkaiser » Sat Feb 26, 2011 12:28 pm
rock5 wrote:Alkaiser wrote:SCOUT_CONCENTRATE is missing from skills_local.xml
Is that the one that is instant cast, cooldown 5m and focus regeneration speed is increased by 100%?
And I guess it should only be used in battle.
Edit: Sorry, found it. It was misnamed. I'll add the correct name and add it to skills_local. Should I add 'inbattle=true' so it only uses it in battle or should I leave it to the user.
That sounds good. It would be wasted outside of battle anyway.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#43
Post
by rock5 » Sat Feb 26, 2011 7:42 pm
Actually, what would be good is if you could check to see if your focus was below a certain point before using. Unfortunately there is no check for that.
I'll just add the inbattle=false.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Alkaiser
- Posts: 222
- Joined: Sat Sep 25, 2010 2:03 pm
#44
Post
by Alkaiser » Mon Mar 07, 2011 6:25 pm
Is there some reason WARRIOR_BLASTING_CYCLONE has a casttime? It always shows "failed to cast" in the console... I assume because there is no cast bar for this skill.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#45
Post
by rock5 » Mon Mar 07, 2011 10:20 pm
Alkaiser wrote:Is there some reason WARRIOR_BLASTING_CYCLONE has a casttime? It always shows "failed to cast" in the console... I assume because there is no cast bar for this skill.
If you have the skill can you confirm the details? Does it use 35 or 50 rage? Does it have a cooldown of 12 or 30 seconds? Is it 2s cast time or instant?
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Alkaiser
- Posts: 222
- Joined: Sat Sep 25, 2010 2:03 pm
#46
Post
by Alkaiser » Wed Mar 09, 2011 12:46 am
It uses 35 rage and has a 12 sec cooldown. As far as I know, it is instant cast.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#47
Post
by rock5 » Thu Mar 10, 2011 2:11 am
It probably changed since it was first added. I'll change it in my next commit.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
lisa
- Posts: 8332
- Joined: Tue Nov 09, 2010 11:46 pm
- Location: Australia
#48
Post
by lisa » Thu Mar 10, 2011 3:40 am
functions.lua has
"function levelupSkill"
which levels up skills that are in database/skills.xml
There are however passive skills that still require lvling up and since they arn't in skills.xml anymore it gets an error.
Simple solution is to just add skills to the skills.xml such as
Code: Select all
<skill name="SCOUT_RANGED_WEAPON_MASTERY" />
It doesn't need anymore information then that for it to work.
A longer solution would be to rewrite the function to work even if the skill isn't in skills.xml
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#49
Post
by rock5 » Thu Mar 10, 2011 5:52 pm
That's a good point Lisa. I believe it would also need the aslevel, skilltab and skillnum values to work with the current code.
The 2 main problems I see are;
- it would be a big job to add them all to the skills database
- it would require the passive to be added to the skills list in the profile which is confusing because they are supposed to be the skills you 'use' but passives are not 'used'.
Maybe what we need is another function that bypasses the skills list and can be used to level up a skill directly eg.
Code: Select all
levelupSkillManually(_skillname, _times)
It could use the 'GetSkillDetail' function in-game to find the skill then level it. Users could add it in their 'onLevelup' section to level their favourite passives.
What do you think.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
lisa
- Posts: 8332
- Joined: Tue Nov 09, 2010 11:46 pm
- Location: Australia
#50
Post
by lisa » Fri Mar 11, 2011 10:03 pm
Yeah I agree adding them all to the skills DB would take a while and it's not really necessary. I would guess less then 25% of users may concider getting bot to lvl their skills, I just happen to be one of them lol
From what I can tell so far you would need the tab and number for each skill anyway. "GetSkillDetail" uses them to get name and such but I didn't come across any functions to get the numbers from the name.
I see 2 ways about it,
1. make up a passive skills DB and add a new lvlpassiveskill function to match
2. people use the in game function "SetSpellPoint" and add the tab and number themselves. sendMacro("SetSpellPoint(4, 2);");
Would also need to make sure the current function doesn't try to lvl the passive skills anymore.
The best option is probably 1 but would involve a bit of time doing it. I am stretched pretty thin at the moment for time =(
Option 3 is of course leave the functions as is and just add the passive skills to the bottom of skills.xml lol
-
lisa
- Posts: 8332
- Joined: Tue Nov 09, 2010 11:46 pm
- Location: Australia
#51
Post
by lisa » Fri Mar 11, 2011 11:09 pm
As it is the existing function only lvles up 1 passive skill, adding this to skills.xml would be fine for now as no other passives are lvled by the function.
Code: Select all
<skill name="SCOUT_RANGED_WEAPON_MASTERY" aslevel="6" skilltab="4" skillnum="4" />
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#52
Post
by rock5 » Sat Mar 12, 2011 12:48 am
lisa wrote:From what I can tell so far you would need the tab and number for each skill anyway. "GetSkillDetail" uses them to get name and such but I didn't come across any functions to get the numbers from the name.
Actually I envisioned it would do a search for the name eg.
Code: Select all
function levelupSkillManually(_skillname, _times)
for tab = 1, 4 do
slot = 0
repeat
local name, _, icon, _, rank, type, upgradeCost, isSkillable, isAvailable = RoMScript("GetSkillDetail("..tab..","..slot..")")
if name == _skillname then
-- take steps to level up the required number of times
return
end
until name == nil
end
end
Of course that's really rough but you get the idea. I'd get the skill details in batches to reduce the number of RoMScripts so it wouldn't take too long. I've gotten good at that.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
lisa
- Posts: 8332
- Joined: Tue Nov 09, 2010 11:46 pm
- Location: Australia
#53
Post
by lisa » Sat Mar 12, 2011 3:50 am
Trouble is you would need to use the ingame names for skills.
"Flame" and not "MAGE_FLAME"
"Electrostatic Charge" and not "MAGE_ELECTROSTATIC_CHARGE"
Which might confuse some people and not sure how it would work for different languages.
I can't help but feel if people can name the skill then they can also just use the tab, num.
Code: Select all
function levelupSkillManually(_skilltab, _skillnum, _times)
local times = 0
if _times == nil then _times = 1 end
repeat
sendMacro("SetSpellPoint(".._skilltab..", ".._skillnum..");");
times = times + 1
repeat until times = _times
end
I can't help but feel I am still overthinking things lol
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#54
Post
by rock5 » Sat Mar 12, 2011 8:08 am
I don't really think it's a problem that, for this function, the users would have to use the real name. It is, in effect, a 'bypass' function after all, so I'd think it's logical that it wouldn't use the bots skill names. Language shouldn't be an issue as long as they use the name as they see it in-game(not sure about special characters).
But you're probably right. It would be just about as easy to just use the tab and number. I just thought it might be that little bit more convenient to use the name and more reliablely level the right skill.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Giram
- Posts: 191
- Joined: Thu Aug 26, 2010 3:34 pm
- Location: Finland
#55
Post
by Giram » Sat Mar 12, 2011 3:07 pm
I think it was after this update when i started getting this error:
Sat Mar 12 21:59:02 2011 : ...Runes of Magic/micromacro/scripts/classes/player.lua:1260: Error in your profile: onLeaveCombat error: ...t/Runes of Magic/micromacro/scripts/classes/pawn.lua:536: bad argument #1 to 'gmatch' (string expected, got nil)
I think it's related to the housemaid buff userfunction in my case. This happened after killing mob and didn't have some buffs on. When i just took those potions it's working again and won't give that error next time it needs to take potion or food. And there are also times that i don't get this error.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#56
Post
by rock5 » Sat Mar 12, 2011 9:50 pm
Giram wrote:I think it was after this update when i started getting this error:
Sat Mar 12 21:59:02 2011 : ...Runes of Magic/micromacro/scripts/classes/player.lua:1260: Error in your profile: onLeaveCombat error: ...t/Runes of Magic/micromacro/scripts/classes/pawn.lua:536: bad argument #1 to 'gmatch' (string expected, got nil)
I think it's related to the housemaid buff userfunction in my case. This happened after killing mob and didn't have some buffs on. When i just took those potions it's working again and won't give that error next time it needs to take potion or food. And there are also times that i don't get this error.
You're probably right as I changed the buff functions to get their data from memory in that revision although I tried to make it backward compatible.
What version of the bot are you running at the moment? What is in your 'onLeaveCombat' section? Have you updated to the latest housemaid userfunction?
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Giram
- Posts: 191
- Joined: Thu Aug 26, 2010 3:34 pm
- Location: Finland
#57
Post
by Giram » Sun Mar 13, 2011 11:02 am
Rombot version 3.29 rev 580
Here is one onleave but it's happening with all that have useGoodie. Sometimes it works and sometimes it gives me error.
Code: Select all
<onLeaveCombat><![CDATA[
-- Additional Lua code to execute after killing an enemy
]]>
useGoodie("speed");
useGoodie("luck");
useGoodie("patt3");
catchCavy();
lootBodies();
player.free_flag1 = false;
</onLeaveCombat>
The userfunction i am using now is posted on here
http://www.solarstrike.net/phpBB3/viewt ... 949#p18949. It was updated to use id of the buff.
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#58
Post
by rock5 » Sun Mar 13, 2011 10:31 pm
I think the problem is with the useGoodies function. If I print the value for GetIdName(506679), I get nothing(that's the patt3 item name). It looks like none of the foods work. Maybe they are the wrong id numbers.
I got some food from my house maid and they weren't called 'Smoked Bacon with Herbs' and 'Salted Fish with Sauce' but 'Housekeeper Special Smoked Bacon with Herbs' and 'Housekeeper Special Salted Fish with Sauce' . The ids for those are 207210 and 207209 respectively. I suspect a review of the food ids is in order.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
-
Alkaiser
- Posts: 222
- Joined: Sat Sep 25, 2010 2:03 pm
#59
Post
by Alkaiser » Fri Mar 18, 2011 8:22 pm
Missing from skills_local.xml
Code: Select all
<ROGUE_LIONS_PROTECTION en="Lion\'s Protection" />
-
rock5
- Posts: 12173
- Joined: Tue Jan 05, 2010 3:30 am
- Location: Australia
#60
Post
by rock5 » Fri Mar 18, 2011 10:27 pm
Alkaiser wrote:Missing from skills_local.xml
Code: Select all
<ROGUE_LIONS_PROTECTION en="Lion\'s Protection" />
I'll add it to the next commit.
- Please consider making a small donation to me to support my continued contributions to the bot and this forum. Thank you. Donate
- I check all posts before reading PMs. So if you want a fast reply, don't PM me but post a topic instead. PM me for private or personal topics only.
- How to: copy and paste in micromacro
________________________
Quote:
- “They say hard work never hurt anybody, but I figure, why take the chance.”
Who is online
Users browsing this forum: No registered users and 0 guests