Nature Drain v1.1 [GUI][MUI]

[Moderator]

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

Needs better tree detection.
The indexes aren't resetting.
Set the dummy to can't raise, doesn't decay.

 

[Jay the Editor]

1- You don't need to store it into a unit group variable and destroy it if you're using bj_wantDestroyGroup function.
2-

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
• If - Conditions
  • ND_Index Equal to 0
• Then - Actions
  • Set ND_Index2 = 0
  • Trigger - Turn off (This trigger)

Check only this after you decrease the index which is right below this trigger

• Set ND_Index = (ND_Index - 1)

look like this

• Set ND_Index = (ND_Index - 1)
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ND_Index Equal to 0
  • Then - Actions
    • Set ND_Index2 = 0
    • Trigger - Turn off (This trigger)

3- Change dummy death type to "Can't raise, Does not decay"
4- Turn off the collision for the dummy

Overall, I love the ideas and I quite like the effects (i've been searching for this! )

EDIT: I just realised the spell is not that efficient as it runs once every 2 seconds. The second caster need to wait for a short duration before the effect comes out after the first caster cast.

 

[Jazztastic]

When writing in English, you should put a space after commas. It makes reading less strenuous.

My thought exactly.

 

[Bannar]

Many things mentioned already, although I'll add few things:
It's an old inefficient indexing system. Recycles indexes only when all instances have ended. Great here would be Hanky's dynamic indexing or Bride's Unit Indexer (woah Bribe DD I still can't stop having fun with it) - although if we stick to this one spell is done pretty nice.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((Picked destructible) is dead) Equal to True
  • Then - Actions
    • Animation - Play (Picked destructible)'s death animation

Isn't that silly? If destructible dies, death animation if played automaticaly.

Posters above told you that after using 'bj_wantDestroyGroup = true' you do not need to set/destroy group manualy all long as group you are manipulating is needed only temporary. How it should look:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every int in (Units within 400.00 of ND_TreePoint matching (((Matching unit) belongs to an enemy of (Owner of ND_Caster[ND_IndexLoop])) Equal to True))
  • Loop - Actions

For smooth looping you should change periodic, 2.00 seconds creates delay at cast-loop action what makes spell looks sometimes not naturaly. Let it be a period between 0.03 - 0.1.

You shouldn't do such functions:

• Custom script: set bj_wantDestroyGroup=true
• Unit Group - Pick every unit in ND_UnitGroup and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Picked unit) is alive) Equal to True
        • ((Picked unit) is A structure) Not equal to True
        • ((Picked unit) is Magic Immune) Not equal to True

I recommend setting the requirenets for matching unit in group creation and adding just one but very important condition - ((Picked unit) Not equal to No unit. Try:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every int in (Units within 400.00 of ND_TreePoint matching ((((Matching unit) belongs to an enemy of (Owner of ND_Caster[ND_IndexLoop])) and (((Picked unit) is alive) Equal to True) and (((Picked unit) is A structure) Not equal to True) and (((Picked unit) is Magic Immune) Not equal to TrueEqual to True))))
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Picked unit) Not equal to No unit
      • Then - Actions

Last thing that I want you to notice is that spell features almost no configurables meaning user has to manipulate the whole trigger himself to customize it, rather than leaving core alone and changing just few variables at the begining of the code.

 

[Maker]

Use this tree detection: https://www.hiveworkshop.com/threads/destroy-only-trees.194925/post-1910913