Seeking Cluster Rockets v1.0

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Seeking Cluster Rockets v1.0 | Reviewed by Maker | 29th Sep 2013

NEEDS FIX

Great tooltip, good effects, I like the concept If the target is close, the missiles circle around the target and do not hit When the caster stops channeling, the already flying missiles could still fly to the target instead of being destroyed instantly The looping trigger should be initially off

[tr]

 

[Daffa]

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
• If - Conditions
• (Seeking Cluster Rockets Loop <gen> is on) Equal to True
• Then - Actions
• Trigger - Turn off Seeking Cluster Rockets Loop <gen>
• Else - Actions

Why don't turn it off from WE?

 

[Wrda]

• Custom script: set udg_SCR_HarvestOrder = String2OrderIdBJ("harvest")

why you create a variable for harvest order?

Custom script: set udg_SCR_Harvester = CreateUnit(Player(15), 'hpea', 0, 0, 0)

• Custom script: call UnitAddAbility(udg_SCR_Harvester, 'Aloc')

why you just don't add locust ability in OE?

 

[Rheiko]

he wants to make everything,

really

really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really really Configurable

 

[Ofel]

@Daffa
It's just the same...
WE will write call DisableTrigger(gg_trg_Seeking_Cluster_Rockets) on map init. But that's not really a problem right?
Doesn't cause lag... :/

@Wrda
It reduce the lag somewhat

what's OE?

@Rheiko
As i said :

• -------- --------------------------- --------
• -------- End of Configuration --------

 

[Daffa]

Yeah, good point, but I prefer that instead, but that's personal, no biggies.
OE = Object Editor

 

[Rheiko]

@Wrda
It reduce the lag somewhat

what's OE?

maybe it's because ofel's laptop is slow
OE = Object Editor

 

[Ofel]

maybe it's because ofel's laptop is slow

Yeah...

OE=Object Editor

You don't have to repeat it again...

 

[Rheiko]

just to make it clear

 

[Ofel]

@Rheiko
Daffa's post is more clear than yours...
This is why :

by Daffa the Mage

OE = Object Editor

by Rheiko

OE=Object Editor

 

[Rheiko]

no it's the same

 

[Ofel]

Anyway...back to the topic

Any more suggestion for the next update?
Or addition to make it cooler?

 

[Rheiko]

hmm, i don't know

 

[Daffa]

How about change the target type to Instant?

 

[Rheiko]

it reminds me to dota's :\

 

[FUJI]

But in DotA it is really will make the user balanced in the skill because if it is target type it will cause a Imbalance in the skill. Anyway I suggest the Instant Cast and change the target per one fired rocket then if there is no other unit next to it then make the rocket detect the first target.

 

[Ofel]

@Daffa
Then the rockets will seek random nearby units?

 

[Wrda]

No, that sounds stupid, just let the ability be like it is, because it is just fine.

 

[Daffa]

Ofel
nevermind, that's almost same as Tinker's Dota. and Yes, seek random enemy

Wrda
Nice point.

 

[Ofel]

@Wrda & Daffa
ok...
Any more suggestions?

 

[FUJI]

Found a Bug Ofel ( I think ) :///

I think there's a bug in this spell or it is only a mistake in the triggers. Because the bug is really annoying xD. When I cast the spell in the MELEE - UNITS, in the range of 300 -> 800 ( distance of the caster and the target melee ) and when the MELEE - UNIT runs to attack the caster ( ofc xD. ) in the distance of the caster and the target between I think it is 150 range of them... The rocket does not check the distance of them and the rocket is moving around in the targeted unit without doing something :/// ( It is really annoying ) You should fix this. You can tell me if this is not a bug and if this is really part of the spell. here is the proof:

Weird Bug

See the rockets?

 

[Ofel]

@jakeZinc
That's why i add this :

• -------- Determines the rocket expiration time. This is used to remove stucked rockets --------
• Set SCR_RocketDebugTime = 8.00

It's used to remove rocket by time...

 

[FUJI]

@jakeZinc
That's why i add this :

• -------- Determines the rocket expiration time. This is used to remove stucked rockets --------
• Set SCR_RocketDebugTime = 8.00

It's used to remove rocket by time...

Hmmmm..... thanks for pointing me out ( sorry I didn't see that xD. )

Anyway I'll rate it 4/5 ( See the rate at the spell ).

 

[Ofel]

Thanks...

 

[pOke]

I really enjoyed this spell, and the VERY detailed config file is nothing to joke about!

Instead of Saving the X and Y as you do, you could just
SetUnitX/Y(udg_Dummy), GetUnitX(udg_Dummy) + udg_SpeedInterval * Cos/Sin(bj_DEGTORAD * udg_RocketAngle))

 

[FUJI]

I really enjoyed this spell, and the VERY detailed config file is nothing to joke about!

Instead of Saving the X and Y as you do, you could just
SetUnitX/Y(udg_Dummy), GetUnitX(udg_Dummy) + udg_SpeedInterval * Cos/Sin(bj_DEGTORAD * udg_RocketAngle))

It is more readability and neater if he store the movement in the real variable.

 

[Ofel]

I really enjoyed this spell, and the VERY detailed config file is nothing to joke about!

Glad you feel that...

Instead of Saving the X and Y as you do, you could just
SetUnitX/Y(udg_Dummy), GetUnitX(udg_Dummy) + udg_SpeedInterval * Cos/Sin(bj_DEGTORAD * udg_RocketAngle))

As what jakeZinc said...

@jakeZinc
Yeah that's true!

 

[T. D. W.]

Review

Suggestions

Rating

Judgement

My Words

If (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions (Seeking Cluster Rockets Loop <gen> is on) Equal to True Then - Actions Trigger - Turn off Seeking Cluster Rockets Loop <gen> Else - Actions

Directly Turn Off the trigger, there is no need for this

Trigger - Add to Seeking Cluster Rockets Cast <gen> the event (Unit - A unit Starts the effect of an ability)

Why don't you add the event in the Trigger Editor, i understand that the loop event needs this function but this one does not need it at all

Set SCR_ShotAngleWidth = (SCR_ShotAngleWidth / 2.00)

Change this ^ to this:

Set SCR_ShotAngleWidth = (SCR_ShotAngleWidth / 2.50)

It will prevent the missiles to do circles around the target
And maybe add an AoE/Unit Pointer but it's not that important

If You want me to review an Update of This Spell, Please Notifiy me this by VM or PM

 

[T. D. W.]

Bump!
I think someone forgot about his spell ^^

 

[FUJI]

^ No because he is just busy at school homework's stuffs ^^.

 

[Atideva]

simple and wonderfull