Multi-Wave

[Moderator]

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

10:10 GMT, 1st Mar 2011
Bribe:

Instead of (Triggering unit) over and over, you have your variable MW_GetTriggerUnit ... use that instead as it boosts performance.

Still using "Custom value" of units, this needs to use a hashtable or else it will conflict with any other system using custom values.

Rejected until updated.

 

[Deuterium]

To start with, idea of the spell + SFX is sort of terrible.

Also, you use a Wait.

Well simply said, this spell meets the standards, but is lacking.

 

[Hiruty]

sadly yes i did use a wait q.q becuase for some reason thespell would fire befor it is hit >.> so i had to add that wait sadly.....

 

[Deuterium]

It's only about the wait, it also lacks idea and nice SFX :s no offense... i'm not trying to be rude.

 

[StaberFire]

None of the spell effects look any good with eachother... you managed to use 5 effects for the spell and have none of them coordinate with the others. That alone is impressive.

Also the waves don't shoot out at the proper angle. The light waves extend in a conical fashion from the facing angle of the caster, but the water waves go to his left, leaving the entire right area a blindspot.

In short, better effects, fix your angles.

And use a counter instead of a wait.

 

[Zack1996]

The effects looked horrible. Try alternating the Shockwaves and Crushing Wave effects.
Also, please remove the dummy's classification as a worker, and give its model as "none.mdl", set its selection scale to -1 (shift and click enter to insert negative for this value), this should make it as invisible as possible.

A fairly simple spell, but should be fine for approval.

 

[eleljrk]

Change the distance from 400 to 0 or it wont hit the enemies..
Else i like the idea, but bad triggered..

 

[baassee]

First of all, the idea is kind of off but here we go.

Before triggering I have some points.

The multiwave spell is based on warstomp. Does both damage and stun, just had to remind you that.

Triggering

- Very complex triggering, with that I dont mean that it's very good scripted but it's very very very compressed together use like a dummy comment line around each and every comment like this.

Tip

• -------- --------------------------------- --------
• -------- More readable now or not? :) --------
• -------- --------------------------------- --------

- I really dislike using facings usally isnt the best to coordinate with.

- Looks a bit slopy, first, a weird wave then KABOOM it all starts.

- Your way of MUIness is ... weird. Cause it seemable that the actions dont collide with each other, tested this with like 20 units at the same time :O btw threw my testing I noticed that you were missing a proper hotkey.

- This line

• Unit - Remove ShockWave from (Triggering unit)

you dont refer to your variable which you were so good and did before and also I see no point with removing the wave at all actually.

same goes with the second loop.

and also, this might not be your fault but doesnt it seem that the one wave travel more the nsecond?

 

[Hiruty]

• Unit - Remove ShockWave from (Triggering unit)

you dont refer to your variable which you were so good and did before and also I see no point with removing the wave at all actually.

same goes with the second loop.

and also, this might not be your fault but doesnt it seem that the one wave travel more the nsecond?

View attachment 65029

I have fixed those bugs now with the updated version
i fixed the waves being longer then each other with a dled modle from here
nothing was ever wrong with the hotkey
i didnt see that bug sry (

• Unit - Remove ShockWave from (Triggering unit)

)

and thankz for pointing those out!

 

[baassee]

the hotkey isnt W it's T ...

 

[Hiruty]

i checked it it is W

 

[-Berz-]

Why are you using <Triggering unit> if you set it at the start?

• Set MW_GetSpellLevel = (Level of MultiWave for (Triggering unit))

• Set MW_FaceAngle = (Facing of MW_GetTriggerUnit)

This is not accurate (what happens if you change unit's turn rate?)
Use "Angle between points (In your case caster-targeted point of ability being cast).

• Unit - A unit Is attacked

Bugs.

• Wait 0.30 game-time seconds

Bad.

• Set MW_Angle = (MW_Angle + 20.00)

Loop is instant so set it to 180.

And your triggering is really hard to read, make clearer variables names like Get_Triggering_Unit(Yeah welcome to the GUI )
And make it like:
----------------------
-Spell Settings-
---------------------

Meh, seems to be "ok" thou the ingame effect isn't anything spectacular.
Your triggering lacks little things, don't worry practice a bit and you should find your way around really fast

 

[Titania]

Hotkey now is C?? And the spell is not really that useful...I can't think of anything other then an archer using it...But archers are supposed to shoot arrows

 

[Hiruty]

Titania, i made this spell a year ago when i was still a noob. I like my noob heritage and i thought i would just make the spell easier to read, smaller and better looking. I know the spell is nothing special i don't even know if it will get approved ( most likely not ). So yah in conclusion the spell is crap and people only dl it cause i'm too lazy to change the screen shot to the new ingame one.

 

[Titania]

Ic...Can you make some nightelf spells...I don't see them much now

 

[Bribe]

I still don't see the updated screenshot :-/

Also, Custom value of a unit often conflicts with other systems. This instead needs to use a hashtable or, even better, a unit indexing library.