Spellpack Chaos Spells

[Gtam]

Bumpidie Bumpidie Bump!!!!!!!!!!!

 

[Dr.Jack]

At fire blast make unit face within 0 seconds.
Your chaos line & meteor spell are a bit unsafe, too much things can go wrong. Also you could improve the coding. You are recycling at 1000 instead of 8190. You could also turn on/off triggers and add conditions to make the coding more optimized.

Spells complex as this should really be made with Jass... Not only that reading this code is a pain it could be really improved.

As it is I don't find this resource approveable. Unless other staff member decides to approve this, this will remain unapproved. Sorry, thanks for contributing though!!

 

[Gtam]

At fire blast make unit face within 0 seconds.
Your chaos line & meteor spell are a bit unsafe, too much things can go wrong. Also you could improve the coding. You are recycling at 1000 instead of 8190. You could also turn on/off triggers and add conditions to make the coding more optimized.

Spells complex as this should really be made with Jass... Not only that reading this code is a pain it could be really improved.

As it is I don't find this resource approveable. Unless other staff member decides to approve this, this will remain unapproved. Sorry, thanks for contributing though!!

What do you mean with the fire should i do it or did id o it and its wrong and what can go wrong with the chaos line and meteor and what can be optimized

 

[Gtam]

bump with the same Qs as above

 

[Gtam]

bump

 

[GetTriggerUnit-]

the fire should i do it or did id o it and its wrong and what can go wrong

lol... How could he answer a so confusing question?

 

[Gtam]

nvm that part i fixed it

 

[Gtam]

Bump. why cant the questions be anwsered

 

[Avaleirra]

The spells look great! +rep (I might use them )

 

[Gtam]

Thx some one finds them useful

 

[Tyman2007]

Add a little more variety to the spells.
Seems to me like you're making the same spell over and over again, but with different models and a different way or triggering it.
Chaos line is the most unique, but it goes in a line (like all the rest)

Periodic event should be 0.03 instead of 0.01. 0.01 is known to cause bugs.

Other than that, They are good and some people can find them useful, however importing is a bit longer than other spells (which doesn't really matter to some people)

It's GUI, so people can definately learn on how you made those spells. For GUI spells, these are pretty good.

Just to help you out and encourage you, +rep.

 

[Gtam]

Thx for appreciating some GUI spells. I will try to make my next spell pack so that each spell is diffrent.

 

[Gtam]

bump.

 

[Compute(<>)]

nice spell pack its awsome and all those models that looks like you imported but actulay just normal models. +rep from my side for the great spells and making me learn a thing or two in the object editor.

 

[Komaqtion]

I'm not sure why you're using an array here (Fire Blast spell):

• Set FB_Offset[(Integer A)] = (FB_Caster_Loc offset by FB_TempLine towards FB_Angle degrees)

You can just use a single temppoint variable (Will save a tiny bit of memory )

Also, maybe make the 150.00 here a configurable, so you can easily change the distance between the effects

• Set FB_TempLine = (FB_TempLine + 150.00)

And again, why array here (Chaos Line spell):

• Set RL_C_P[RLI] = (Position of RL_C[RLI])
  • Set RL_Selected_P[RLI] = (Target point of ability being cast)
  • Set RL_Temp[RLI] = (RL_C_P[RLI] offset by 200.00 towards (Angle from RL_C_P[RLI] to RL_Selected_P[RLI]) degrees)

Just use normal temp stuff there (I know you're using that in the other trigger, but it's not the same )

I'd also suggest you change the event from 0.01 to 0.03 (And then also change the damage from 1.00 to 3.00 )

I'd really like if you added a lot more configurables in the triggers, like the AoE of all hte group selections, and stuff like that
You can (Almost) never have too many configurables...

In "Meteor", what is this line doing here:

• If (AIM2 Less than 100) then do (Set AIM2 = (AIM2 + 1)) else do (Set AIM2 = 1)

I can't see you using that variable anywhere...

Also, same here... Change the event to 0.03 or something...

And, yet again... Why array here ?!

• Set M_DamageP[0] = (Position of (Picked unit))

It's just unecessary to use that many arrays when you don't need them...

Anyways, hope some mod can review this soon, as I think it's in good shape to handle it

 

[Gtam]

bump Updated

 

[Ayanami]

For Flame Strike, may I suggest not using Shadow Strike? 2 Shadow Strikes will not stack. Instead, damage using trigger and add an aura to reduce the attack speed instead. Then remove it after the duration using dummy and expiration timer.

 

[yami-joey]

Oh...really cool

I'll use these spells for my Panda Wars map, that fine with you?

 

[Gtam]

sure go ahead.

 

[Gtam]

Xmas Bump