Celerity movement speed increase VS description

Where bugs go to lie down and rest

Moderator: Moderator

Post Reply
Message
Author
marvalis
Uruivellas
Posts: 683
Joined: Sun Sep 05, 2010 5:11 am

Celerity movement speed increase VS description

#1 Post by marvalis »

/edit nevermind, this is not a bug I was wrong.

Celerity increases movement speed by 5% and does not take into account your talent level?
Description says it should be 5% * talent?

Code: Select all

newTalent{
	name = "Celerity",
	type = {"chronomancy/speed-control", 1},
	require = chrono_req1,
	points = 5,
	mode = "passive",
	on_learn = function(self, t)
		self.movement_speed = self.movement_speed + 0.05
	end,
	on_unlearn = function(self, t)
		self.movement_speed = self.movement_speed - 0.05
	end,
	info = function(self, t)
		local power = self:getTalentLevelRaw(t) * 5
		return ([[Increases your movement speed by %d%% and switching between already equipped weapon sets (default hotkey x) no longer takes a turn.]]):
		format(power)
	end,
}
Last edited by marvalis on Mon Jan 23, 2012 5:08 pm, edited 1 time in total.

phantomglider
Archmage
Posts: 372
Joined: Fri Jan 20, 2012 12:13 am

Re: Celerity movement speed increase VS description

#2 Post by phantomglider »

Pretty sure that it works as described. When you put a point in it, it boosts your movespeed by 0.05. Put in another point, boosts your movespeed by another 0.05. on_learn triggers every time you put a talent point in.

Compare Trap Mastery:

Code: Select all

on_learn = function(self, t)
		local lev = self:getTalentLevelRaw(t)
		if lev == 1 then
			self:learnTalent(self.T_EXPLOSION_TRAP, true, nil, {no_unlearn=true})
		elseif lev == 2 then
			self:learnTalent(self.T_BEAR_TRAP, true, nil, {no_unlearn=true})
		elseif lev == 3 then
			self:learnTalent(self.T_CATAPULT_TRAP, true, nil, {no_unlearn=true})
		elseif lev == 4 then
			self:learnTalent(self.T_DISARMING_TRAP, true, nil, {no_unlearn=true})
		elseif lev == 5 then
			self:learnTalent(self.T_NIGHTSHADE_TRAP, true, nil, {no_unlearn=true})
		end
	end,
	on_unlearn = function(self, t)
		local lev = self:getTalentLevelRaw(t)
		if lev == 0 then
			self:unlearnTalent(self.T_EXPLOSION_TRAP)
		elseif lev == 1 then
			self:unlearnTalent(self.T_BEAR_TRAP)
		elseif lev == 2 then
			self:unlearnTalent(self.T_CATAPULT_TRAP)
		elseif lev == 3 then
			self:unlearnTalent(self.T_DISARMING_TRAP)
		elseif lev == 4 then
			self:unlearnTalent(self.T_NIGHTSHADE_TRAP)
		end
	end,
<Ferret> The Spellblaze was like a nuclear disaster apparently: ammo became the "real" currency.

Varil
Halfling
Posts: 93
Joined: Mon Mar 01, 2004 6:58 pm

Re: Celerity movement speed increase VS description

#3 Post by Varil »

Without reading the code completely...that depends. Does "on learn" mean only when you put the first point in it, or every time you put a point into it?

bricks
Sher'Tul
Posts: 1262
Joined: Mon Jun 13, 2011 4:10 pm

Re: Celerity movement speed increase VS description

#4 Post by bricks »

It's every time. If you're unsure, you can learn the talent up to level 5 and check movement speed on the character sheet.
Sorry about all the parentheses (sometimes I like to clarify things).

edge2054
Retired Ninja
Posts: 3756
Joined: Fri May 28, 2010 4:38 pm

Re: Celerity movement speed increase VS description

#5 Post by edge2054 »

I think how movement speed works has been changed a lot though since that talent was written.

+100% movement speed is now two moves for each of the enemies one were it used to be 'moving for free'.

I'll have another look at the balance of this talent soon. Compared to the Cursed talent that does the same thing this one feels really underpowered.

As to the bug yeah, it's setting the actors movement speed to whatever it currently is + 0.05 every time you put a talent in. It's a permanent change and almost all passive talents work this way.

Varil
Halfling
Posts: 93
Joined: Mon Mar 01, 2004 6:58 pm

Re: Celerity movement speed increase VS description

#6 Post by Varil »

I can't say anything about Cursed...but Celerity feels fine to me, especially for a tier 1 talent, and this is coming from someone who feels that when it comes to combat, Mobility Is King.

Then again, Temporal Warden is mad OP. So that might be part of it. ;)

Post Reply