Gaze of Medusa Spell v1.2

[Devalut]

Welcome to the hive, first and foremost I'd recommend removing the default melee initialization and create a trigger to reveal the map:

Spoiler

• Map Reveal
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Visibility - Disable fog of war
    • Visibility - Disable black mask

I'd suggest you thumb through these:
Memory Leaks
Resource Submission Rules (Models, skins, icons, spells)

 

[OnGe]

Thanks for nice welcome. There is custom initialization in fact and fog of war is there to demonstrate statues acctualy dont see. I tried to place figurants close enough to be seen reasonably.

 

[JC Helas]

Welcome to hive.

Well all i can say is you have to use dynamic index instead of using wait which is unnecessary way, you have to imagine that ability could be in other unit and may use it at the same time.

I highly recommend to use dynamic index
Spend time from reading this link
Visualize: Dynamic Indexing

 

[OnGe]

Thanks for comment. Dynamic indexing seems like overengineering to me, since this is targeted spell with (almost) instant effect. Wait is there to prevent original Charm spell effect to mess with trigger and Dynamic indexing would not help that.

Anyway, I have changed it to be staticaly indexed (instanced in array indexed by casting unit owning player) so it should be pretty much safe for multiplayer now.

Also, I have tweaked some effects and made statues to act more like statues.

 

[Pyrogasm]

The spell can still fail if you group-cast the ability with multiple units or are extremely fast casting it an additional time during the wait. Waits have a minimum duration of ~0.27s so that will not be a 0.01. For that matter you could put a 0 wait and it would be the same duration. There are a few options I would recommend:

• Don’t base the spell on Charm. Base it on Channel or something else, then you don’t have to worry about this and use the wait.
• Use a locally shadowed global to store the unit during the wait: local udg_
• Use a simple but clever GUI wait indexing method with 2 counter variables:

• Set C1 = C1 + 1
• Set UnitArray[C1] = the unit
• Wait 0.00 seconds
• Set C2 = C2 + 1
• Unit - Kill UnitArray[C2]
• If (All conditions are true) then do (Actions) else (Actions)
  • If - Conditions
    • C1 equal to C2
  • Then - Actions
    • Set C1 = 0
    • Set C2 = 0
  • Else - Actions

 

[OnGe]

The spell can still fail if you group-cast the ability with multiple units or are extremely fast casting it an additional time during the wait. Waits have a minimum duration of ~0.27s so that will not be a 0.01. For that matter you could put a 0 wait and it would be the same duration. There are a few options I would recommend:

• Don’t base the spell on Charm. Base it on Channel or something else, then you don’t have to worry about this and use the wait.
• Use a locally shadowed global to store the unit during the wait: local udg_
• Use a simple but clever GUI wait indexing method with 2 counter variables:

• Set C1 = C1 + 1
• Set UnitArray[C1] = the unit
• Wait 0.00 seconds
• Set C2 = C2 + 1
• Unit - Kill UnitArray[C2]
• If (All conditions are true) then do (Actions) else (Actions)
  • If - Conditions
    • C1 equal to C2
  • Then - Actions
    • Set C1 = 0
    • Set C2 = 0
  • Else - Actions

Local variables seems like best solution to me right now. Thanks for that idea!

About different base spell - is there something else than Charm and Hex what limits casting by level of target? Because this is what I see as key feature.

 

[OnGe]

In the end, I have realized I do not need variable at all and referencing to target of ability being cast will do the trick.

I have updated spell with few more tweaks. I hope this is final one

 

[Pyrogasm]

Not that I am aware of, though you could simulate an error message and deny casting by checking the level yourself.