Heroic Leap v0.01

[Moderator]

18:59, 12th Jul 2010
TriggerHappy:

Nice effect and the coding was fine.

 

[G00dG4m3]

I hope you will enjoy this, and I believe that Cast sound fits the spell's theme

I'd like to get any comments from you to see your thoughts about this my work

 

[Tank-Commander]

Firstly, use a parabola instead of increasing/decreasing a variable to make it look alot smoother.

secondly use 'at 0.00' instead of 'at 10000000.0' pr w/e it is, at 0.00 makes it instant.

 

[Etzer]

You should pick every unit every 0.03 second, to make it look more smooth.
No leaks though

 

[G00dG4m3]

parabola = what?
I will be only nine-grader, so I dunno what is that

 

[MortAr]

parabola = what?
I will be only nine-grader, so I dunno what is that

http://en.wikipedia.org/wiki/Parabola

• If ((Leapers is empty) Equal to True) then do (Trigger - Turn on Heroic Leap Loop <gen>) else do (Do nothing)

Is bad, use normal ITE handle.

• Unit - Add Storm Crow Form to Caster
  • Unit - Remove Storm Crow Form from Caster

Should be in the cast trigger, not in the periodic. You need to add/remove it only 1 time.

• Animation - Change Caster flying height to ((Current flying height of Caster) + Real[1]) at 1000000000.00

0.00 is instant too.

- Use coordinates, locations are bad.
- You can leave the boundary of the map. Please add a condition to check the pathability.
- Theres not tree-destruction functions, better add some, otherwise caster might stuck between trees sometimes.

Neat idea, but not really original (imo), would vote if it was something special, but...

 

[G00dG4m3]

http://en.wikipedia.org/wiki/Parabola

Thank you, but it's quite hard for me

• If ((Leapers is empty) Equal to True) then do (Trigger - Turn on Heroic Leap Loop <gen>) else do (Do nothing)

Is bad, use normal ITE handle.

Why is this so bad???

• Unit - Add Storm Crow Form to Caster
  • Unit - Remove Storm Crow Form from Caster

Should be in the cast trigger, not in the periodic. You need to add/remove it only 1 time.

gonna fix that

• Animation - Change Caster flying height to ((Current flying height of Caster) + Real[1]) at 1000000000.00

0.00 is instant too.

I understood this before, didn't have time to change.

- Use coordinates, locations are bad.
- You can leave the boundary of the map. Please add a condition to check the pathability.
- Theres not tree-destruction functions, better add some, otherwise caster might stuck between trees sometimes.

I don't use JASS that much that I simply deny my sweet locations
It's a target-unit spell, and it is based off on Storm Bolt (stuns before jump), so it shouldn't get stuck.

Neat idea, but not really original (imo), would vote if it was something special, but...

Thank you

 

[Etzer]

It is so bad because you use "Do nothing" it uses up RAM for absoloutly no good. If you use normal ITE there you don't have to add an "else".

 

[MortAr]

As stated above, the (Do Nothing) function takes up a part of the RAM while being called, thus, for example, if you have 15 running instances of the spell (as far as I can see, you use the ITE everytime when the group-call ends), it will critically increase the lag for the players.

It's a target-unit spell, and it is based off on Storm Bolt (stuns before jump), so it shouldn't get stuck.

Yes it can always get stuck, thats why most of DotA's (yes I'm referring to DotA because thats the map with the most spells I've played) spells that uses a similar to this function, has the tree-destroying function, just in case.

I don't use JASS that much that I simply deny my sweet locations

Nobody has ever said you need to use JASS for them, custom-scripts will always do the job.

 

[tjordell]

This should be pretty much to calc the parabola =)

How to calc Parabola:

Set Var_Parabola[Int] = (((4.00 x Var_Height[Int]) / Var_MaxDistance[Int]) x ((Var_MaxDistance[Int] - Var_Distance[Int]) x (Var_Distance[Int] / Var_MaxDistance[Int])))

 

[Rodrigo96]

OMG I am the only one who noticed this ??? :
Because he used the Storm bolt to start the casting the until is stuned BEFORE the caster reaches him. That must be fixed. Set the storm bolt speed = to the caster's jump speed to fix it insted of puting a 10000 misile speed.
ALSO increase the caster's height as it travels to make it more as a jump.

 

[G00dG4m3]

OMG I am the only one who noticed this ??? :
Because he used the Storm bolt to start the casting the until is stuned BEFORE the caster reaches him. That must be fixed. Set the storm bolt speed = to the caster's jump speed to fix it insted of puting a 10000 misile speed.

OMG, dude, have you ever played WoW as warrior? Have you ever seen, how does Charge/Intercept abilities work? Both of them stun target before it starts running towards it (!) and stun usually finishes after Warrior reaches it's target.

 

[Rodrigo96]

I am just saying it will be 354 times more logical that the target is stuned when you strike him and not magicaly as soon as you start jumping. You can decide:
1)Do it like WOW warrior
2)Do it better than a WOW warrior.
Your call man.
I know you are a great spell maker but everyone got mistakes!!

 

[Aradar]

Hmm Rodrigo96 isent it up to the user to edit it for their own needs?

 

[Maximum_Evil]

Nice gonna test it

 

[Hades17]

Although I am not the creator of this Spell-Map, I added the functions mentioned in the comments.
I did adjust the height & smoothness and cleared those mistakes.


Edit: I implemented the Spell in a bigger Map and it's codeset is bugging around like hell, I twisted everything possible and kinda had to recode the whole Spell (used Paladon's Jump System after 1 hour without success).
The Unit always Jumps to the Center of the Map instead to the target, eventhough the expected trigger-action should make him do it correctly.

 

[Gabrael]

I'm having the same problem as Hades17: Whenever I use the spell, it sends my unit to the center of the map. I'm no expert at triggering, and understand almost nothing about the triggers involved in this spell, I've tried fiddling with those on this spell... to no avail.

Someone help!

 

[Ajijang3434]

Hello my friend! I would like to get your help on. I was making a Time Leap spell with almost the same trigger programmed, the only thing I am struggling on right now is I cannot summon my dummy even though I did the same process as yours. Take a look at my Trigger. Hope you can help me with this or even others. Just a begginer programmer of Warcraft, motivating my skills in this kind of programming because I wanted to become like others here.

• Loop
  • Events
    • Time - Every 0.03 seconds of game time
  • Conditions
  • Actions
    • For each (Integer D_Integer[2]) from 1 to D_Integer[1], do (Actions)
      • Loop - Actions
        • Set D_DB[D_Integer[2]] = (D_DB[D_Integer[2]] - D_Reached[D_Integer[2]])
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • D_Boolean[D_Integer[2]] Equal to True
            • D_DB[D_Integer[2]] Greater than 0.00
          • Then - Actions
            • Set D_Loc[0] = (Position of D_Caster[D_Integer[2]])
            • Set D_Loc[1] = (Target point of ability being cast)
            • Set D_Loc[2] = (D_Loc[2] offset by D_Speed[D_Integer[2]] towards D_Angle[D_Integer[2]] degrees)
            • Unit - Move D_Caster[D_Integer[2]] instantly to D_Loc[2]
          • Else - Actions
            • Unit - Create 1 Dummy for D_Owner[D_Integer[2]] at D_Loc[2] facing Default building facing degrees
            • Unit - Add Slow Dummy to (Last created unit)
            • Unit - Order (Last created unit) to Human Mountain King - Thunder Clap
            • Unit - Add a 3.00 second Generic expiration timer to (Last created unit)
        • Custom script: call RemoveLocation (udg_D_Loc[0])
        • Custom script: call RemoveLocation (udg_D_Loc[1])
        • Custom script: call RemoveLocation (udg_D_Loc[2])

 

[MacMardigan]

Hello, same problem here. Units go to the center of the map. Checked the variables/hashtables and triggers, everything seems to be in order (As far as I can understand).

This is a headscratcher