Water Blast v1.2.1

[Moderator]

Water Blast v1.1.0 | Reviewed by Maker | 10th November 2013 [COLOR="gray" [COLOR="gray" [COLOR="gray" [COLOR="gray" [COLOR="gray" Concept[/COLOR]] I can see the spell being useful Triggers[/COLOR]] Configurable, leakless, MUI The harvester does not need Invulnerability ability, it has Locust Filter out magic immune units in the cast trigger to prevent KB3D config to run Magic immunes are prevented in KB3D config The knockback system knocks units over pathing blockers Objects[/COLOR]] No unneeded objects You could change the learned icon position There is no follow through animation, therefore cast animation looks bad The learn tooltip does not show the hotkey The ability could have AoE indicator The ability name is coloured in learned tooltip for level 1 but not for levels 2 and 3 Effects[/COLOR]] There is no effect spam and the effects fit the theme Rating[/COLOR]] CONCEPT TRIGGERS OBJECTS EFFECTS RATING STATUS APPROVED

 

[T. D. W.]

1.0 Review

Personal Words

Review

Suggestions

Rating

Judgement

Note

Well, my impression was a little high :/

First of All, this is the FPS i got when using the spell!
okay, the FPS was 40 due to the amount of units on the screen, but ! the KB was enough to crush the 40 FPS to 2! My impression on this: the KB is bad

If there's too many knocked unit at once the knock might get a bit messy and also will lead to unwanted lags.

I don't understand, if you put this by yourself, then why the hell did you add so much units in this so small area

Move units flowing SetUnitX and SetUnitY instead of Moving them to a location via GUI action

• Destructible - Pick every destructible within 256.00 of WB_Loc2 and do (Actions)

Why did you use 256? is it because the default was 256 hen? i suggest a 150 instead for realism

Okay, no more bad points, but there is more stuff:

Creating Effects only on trees isn't a great Fx thing

The KB is really really poor, not even it looks like lagging when units hit an obstacle, it lags, and it has no Fx!

Okay i'll resume the spell: AoE Knock-Back and damage + tree destroy...

Don't take this rudely: https://www.hiveworkshop.com/forums/spells-569/knock-back-3d-v-1-7-1-a-242031/

I know that this spell was a long time in the Lab, and am disappointed of the result :/

After an edit to the test map (reducing peasants massively) The KB lags less, but the spell is still lacking

Improve the spell, HARDLY

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

 

[Daffa]

First of All, this is the FPS i got when using the spell!
okay, the FPS was 40 due to the amount of units on the screen, but ! the KB was enough to crush the 40 FPS to 2! My impression on this: the KB is bad

Yeah, the KB is pretty sucks I guess.
I'm pretty sure that I need to switch to KB2D by Bribe.

I don't understand, if you put this by yourself, then why the hell did you add so much units in this so small area

It was by accident, I guess this can be solved with fixing the KB.

Move units flowing SetUnitX and SetUnitY instead of Moving them to a location via GUI action

Yeah, I guess it can boost the performance.
Consider the FX on KBed Unit as well on the next update.

Creating Effects only on trees isn't a great Fx thing

The KB is really really poor, not even it looks like lagging when units hit an obstacle, it lags, and it has no Fx!

I consider adding them to units.
Consider it done.

Don't take this rudely: Knock Back 3D v. 1.7.1

I know that this spell was a long time in the Lab, and am disappointed of the result :/

And I'm even more disappointed with my works, so no worries. It's not "that long", just about 10 days or so.

And I was about to add KB3D of yours as well.

Thanks a ton for the review Jad.
+4 Rep
+Credit in Future Versions

------------------------------------------------------------------------------------------------------
Major Planned Updates (morph into V1.1) :
1. Using KB2D & KB3D instead of my lagging KB
2. Adding FX to KBed Units
3. Adding Damage during Knockback
4. Adding Option to Slow units after KB
This will be done in couple of days (based on my school activities).

 

[FUJI]

Ill review this later because I'm in mobile right now xD

The spell made by daffa the Mage, seems cool!

 

[T. D. W.]

Olay daffa , the spell has a potential but it is lacking at the moment

 

[chobibo]

How does the knockback work without a periodic event?

 

[T. D. W.]

The second trigger at hive gains his periodic event from the init trigger (add trigger event)

 

[chobibo]

Oh, that's cool, never thought of that.

 

[FUJI]

Water Blast Version v1.0 This spell had been reviewed by jakeZinchttps://www.hiveworkshop.com/forums/members/jakezinc/#visitor_messaging Unit Group - Add (Picked unit) to WB_Knocked Use your own stored variable: Set WB_TempUnit = (Picked unit) This: Set WB_GroupAmount = (Number of units in WB_Knocked) Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions WB_GroupAmount Greater than (>) 0 Then - Actions Trigger - Turn on Water Blast Knock <gen> Else - Actions You can just use integer counter variable instead of creating a looping through it. Merge this: Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions (Current order of WB_DestKiller) Equal to (==) (Order(harvest)) Then - Actions Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions (WB_TempDest is alive) Equal to (==) True WB_DestroyTreesEffect Equal to (==) True into one ITE Block. Put this inside the Trees Alive Checker ITE Block: Destructible - Kill WB_TempDest so that it will check the tree if it is alive then kill it. Set WB_KAngle = (Load 0 of WB_KeyID from WB_Hash) Set WB_KRange = (Load 1 of WB_KeyID from WB_Hash) put this loaded datas inside this ITE Block: Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions WB_KTime Greater than (>) 0.00 Then - Actions Merge this Condition: WB_KTime Greater than (>) 0.00 and (WB_TempUnit is alive) Equal to (==) True Into one ITE so that the clearing attached datas and removing units is one. Other than that none =). The documentation is well clean and nice. The attacktype and the damagetype must be in the configuration: Unit - Cause WB_CastUnit to damage WB_TempUnit, dealing WB_Damage[WB_Level] damage of attack type Spells and damage type Normal , nice configuration. It is MUI. It can drop my FPS below ( like Jad said =) ) "Manual : 1. Tick "Create Unknown Variables when pasting trigger data" 2. Copy the Ability from Object Editor 3. Copy the Water Blast Folder" this instruction is lacking maybe you can add more details or notes. The special effects are lacking, don't add highly particle effect on destroying trees may cause that lag also, maybe flak cannon or ancient protector missile. There are no knockback effects. Useful in the target radius that can radius easily the enemies and knockback them, it does also destroying trees. Custom script: if ( udg_WB_DestroyTrees == true ) then Custom script: set udg_WB_DestKiller = CreateUnit(Player(15), 'hpea', 0, 0, 0) Custom script: call UnitAddAbility( udg_WB_DestKiller, 'Aloc' ) Custom script: call UnitAddAbility( udg_WB_DestKiller, 'Avul' ) Custom script: call ShowUnit( udg_WB_DestKiller, false ) Custom script: endif You can just use a GUI Friendly than this. Other than that none. The learn extended description of the tooltip is none, lacking of information but it does support the values of the level. Rejected: 1-10 Unacceptable : 11-30 Lacking : 31-49 Useful : 50-69 Recommended : 70-90 Highly Recommended : 91-100 Director Cut : 101-105 1) Triggering - 11/20 2) Documentation - 8/10 3) Importing Instruction - 4/10 4) Special Effects - 6/15 5) Usefulness - 13/20 6) Tooltip - 6/10 7) Configuration - 7/10 Total Score - 55/105 jakeZinc Rating - 2.5/5 Status - Useful and Needs Fix

 

[Daffa]

Ill review this later because I'm in mobile right now xD

The spell made by daffa the Mage, seems cool!

Don't say that yet, it's pretty lacking currently.

Olay daffa , the spell has a potential but it is lacking at the moment

I know that, and you're review give nice pointers to me.

How does the knockback work without a periodic event?

This :

The second trigger at hive gains his periodic event from the init trigger (add trigger event)

 

[chobibo]

Yeah, I've already seen Jad's reply earlier. I even mentioned how cool that was...

 

[Daffa]

Updated to Version 1.1.0
I hope I can get it well without problem this time.

 

[T. D. W.]

1.1.0 Review

Personal Words

Review

Suggestions

Rating

Judgement

Note

Well, all i can say is that it is a lot better than last time

2 things:

1.

Trigger - Run KB3D Config <gen> (ignoring conditions)

This is Extremely Risky, what if the user already has KB3D v. 1.7.1 and doesn't want to import the one from your test map?
An easy solution CnP the following trigger, rename it, and place it in the Folder of water blast, or you can just CnP the values and run KB3D normally, more efficient than running a trigger

2. the KB should be a little more configurable, and faster, it isn't realistic right now, the units move slowly, go up slowly, go down slowly.... And the Effects can be enhanced by wether changing the model you are using for the KB, or making it create all along the KB, (KB3D_Fx)
Along with the KB problem, change KB3D_Fx_Attach to "origin", KB3D_Speed to 350 and KB3D_Z_Offset to 400

None

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

 

[T. D. W.]

1.1.0 Review

Personal Words

Review

Suggestions

Rating

Judgement

Note

Better, yet there is still lacks of improvements
Please Change the following ASAP

I already said that in the last review, KB3D Config is not a trigger to initialize KBs you HAVE to initialize them in the cast trigger, if you have some questions, ask me
KB3D_Fx_Attach should be origin, OR delete the line where you set it since it is origin by default
Update the KB3D
Increase the Speed, decrease the Z offset (in the KB3D config)

The knockback system knocks units over pathing blockers

Since KB3D_JumpOverCliff is TRUE

Filter out magic immune units in the cast trigger to prevent KB3D config to run
Magic immunes are prevented in KB3D config

This should be configurable any way
Other stuff has been said by me and maker
I am sorry but Maker should not APPROVE this YET, please fix all the stuff i mentioned

None

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

 

[chobibo]

What a harsh review...

 

[T. D. W.]

Not my intention, but it shouldn't be approved but needs fix, as i said in the last review, not the one of today, the 'voting for rejection' was to make him MOVE a little... It's been time since i told him to fix the stuff, and for final, ALL the needed changes are from KB3D... And not minor worries...

 

[JoHnyW]

spell seems to be cool but sorry for noob question but i havent been to w3 modding since long time where can I " Tick "Create Unknown Variables when pasting trigger data" "

 

[T. D. W.]

File> Preferences> Tick the option

 

[T. D. W.]

Okay for the configuration now, some little points:

• You should update the KB3D System, remove the folder/section in the test map, then paste the KB3D folder in the KB3D test map, easy as that
• If you do not want to update... then you should set KB3D_Fx_Attach to origin or chest and not the path of the effect
• Remove KB3D_Fx or set it to empty string
• increase the speed a little

EDIT: this will be the result:

 

[Daffa]

will do, I have a sickness at the moment (and that's laziness )

 

[Rheiko]

Rheiko : Minor Mistake Pointer

did i? 0.0
i don't remember i did it =P

 

[T. D. W.]

I am lazy... well, really lazy, but just chaging a variable to null and another to origin and a final to 300, is not so hard ^^'

 

[Daffa]

did i? 0.0
i don't remember i did it =P

I don't either

I am lazy... well, really lazy, but just chaging a variable to null and another to origin and a final to 300, is not so hard ^^'

Can I just do it tomorrow, pretty please :kitty beg:

 

[Daffa]

Just a little note : update some bunch of crap stuff (check log) in Version 1.2.0
I intend to move to the next spell in mind.