Hero Contest #4 - Results

[Pharaoh_]

Animalistic Nature

Contestants were supposed to create a hero with a concept linked with the world of animals. The hero should possess the basics of an animal and combine the key of human nature with the one of animal instincts.

Matheridon96's review

InfinateAnswers:


Concept: 8/10

It was sort of interesting playing with your hero.
His abilities can be exploited (in a good way) to create
amazing 'escape' scenarios

Coding: 4/10

Well, your code could've been better...
You repeated stuff like Triggering Unit instead of setting them inside variables
You used indexing when it wasn't required (An instant spell)
You used (Owner of Triggering Unit) when you could've used (Triggering Player)
You used (Ability Being Cast).... No comment..
You're indexing isn't dynamic at all.. You'd be looping through dead instances all the time..
Also, that would cause your map to lag like hell after a few hundred casts, and it would crash after 8191 casts... (Bad)
You had tons of leaks...

Synergy: 8/10

Well, the abilities were ok.. The only problem is that they could'nt be combined together to create
some sort of tactic.. Well, except for the Movement speed bonus and the Soar ability. You could use
the soar ability to escape, and the more movement speed you have, the better ^^
I had to give you points for that :/

Balance: 6/10

Well, the thing that made him too imbalanced was the cooldown of his Soar ability.. Same for the manacost.
If you could time it right, you could be invulnerable for a very long time.

Aesthetics: 4/5

Your tooltips were self-explanatory, except for the Eagle Warrior's ultimate ability.
You never mentioned that the effects occur when the Eagle Warrior dies.
Also, your hero looks pretty good and the icons are quite good :3

Total: 30/45




Champara Bros:



Concept: 9/10

Playing with a hero based on 'Damage Per Second' is pretty fun :>
I loved the concept and I loved some of the spell effects ^^

Coding: 3/10

lol at the 1-liner
Use (Triggering Unit) instead of (Casting Unit)
Don't repeat stuff like (Last Created Unit), use variables instead.
You have lots of leaks (Way too many to count)
Don't use Custom Values....
You're using Custom scripts, so why don't you use the natives directly instead of the BJs??
Also, Snake Reflexes has the potential to bug.. What if you suddenly teleported to the top of a hill and
you're suddenly stuck..

Synergy: 9/10

Well, the abilities could be combined to create some awesome tactics ^^
Activate the Poison blades, attack the enemy hero, trap him with Snake Spit,
Then use Serpent Dance and Poison Accumulation and you're good to go

Balance: 8/10

Snake Reflexes shouldn't give you the ability to teleport behind your enemy because that would
cause you to lose control and if there are too many units in range,
it could allow your enemy to escape by placing you behind a huge bunch of units.
Other than that, your hero seems balanced..

Aesthetics: 5/5

Your hero looks good, the tooltips were self-explanatory, and the icons were quite relevant to the abilities.

Total: 34/45




Jazztastic:



Concept: 8/10

The concept of a hero that gets strongger as the battle goes on is an excellent concept, although, this really affected
your score in the "Balance" field.

Coding: 6.5/10

I'm slightly disappointed at the fact that you used TriggerSleepAction.. That's a big No-No..
Also, in your "Feral Strike Cast" trigger, why are you using a local for one of your configurables >.<
And since you're using custom scripts, why are you using BJs instead of natives -.-
Finally, you should stop repeating stuff like (Last Created Unit) and just use a variable instead..

Synergy: 8.5/10

Drunken Haze and Tauren Smash could be used together to wound and disable an enemy hero.
Panther Instincts and Animal Rage go good together too ^^

Balance: 5/10

Unfortunately, your hero imbalanced.. He gains strength when he's attacked, so you'd actually require a huge amount of units to affect him...

Aesthetics: 4/5

Everything was perfect except for your the Drunken Haze icon..

Total: 32/45


emjlr3:



Concept: 9/10

Excellent Concept. It was pretty interesting to play with your hero.

Coding: 9/10

Your coding is excellent, but it wasn't perfect
- You used TriggerSleepAction
- You repeated a non-Handle-Var-returning function instead of declaring a local variable and using that.
- You could've used "exitwhen not UnitAlive(...) and not UnitAlive(...) instead of having that 2-liner.
Still, your coding was quite impressive, so I had to give you a high score over here

Synergy: 10/10

The abilities could be combined together to create some nice playing tactics, for example, use Mahalleinir on
your current position when you're attacked by melee units, then use feral and attack those melee units, or you could
assemble a pack of 3 wolves and let them 'tank' you while you deal damage to the enemy ^^

Balance: 9/10

Your hero is quite balanced.. He's very similar to the Melee heroes (In theory)..

Aesthetics: 4/5
Icons -> Excellent
Tooltips -> Excellent
Model -> Could be improved

Total: 41/45



Ahimtar:



Concept: 7/10

Well, the concept was ok.. I neither loved nor hated it :/

Coding: 3/10

- Just enumerate a group with those conditions instead of using those huge if/then/else blocks.
- You have lots and lots of leaks.
- Deal damage based on a formula, you don't need those if/then/else blocks at all...
- You're using waits...
- You're indexing is bad..
etc..

Synergy: 8/10

Well, his first ability is the kind of ability that no one would want to use due to the fact that
you could easily die sometimes.. His other abilities work well together though ^^

Balance: 7/10

He was only slightly imbalanced because you can spawn a huge amount of spiders and trap an enemy hero,
but since that doesn't occur too frequently, you're ok
Oh and because his first ability can literally kill him if he casts it thrice at level 1, I'm going to have to
take out a point

Aesthetics: 4/5

Well, the model could've been approved and the icon of his second ability should be changed to show that it can
be automatically cast.

Total: 29/45




Orcnet:


Concept: 7/10

The concept wasn't soooo interesting, but it was ok.. *sigh*

Coding: 2/10

3 of the spells were simple object editor work.
The triggered spell wasn't MUI and it used waits.
But, on the bright side, you had no leaks ^^
Also, one of your triggers is refferencing a null object editor object :/

Synergy: 8/10

Here's a possible tactic:
Use your first ability to teleport to an enemy unit, stun him with your second ability and
use your ultimate ability.. He's dead

Balance: 9/10

Your hero is quite balanced.

Aesthetics: 3/5

What forced me to give you a 3 instead of a 5 was the fact that all the icons were pink when your
hero was based on "Brute Force" and "Endurance" o.o

Total: 29/45



Xiliger:


Concept: 9/10

Kinda familiar
I still loved it though ^^

Coding: 5/10

A great contradiction arose here..
How could you know vJass when you used waits in your Cleave spell
Also, when you want to stun a unit, you shouldn't pause them..
It bugged when I charged a unit and killed it immediately.
You had no leaks in your GUI triggers.
Your vJass code needs an improvement though.
- The Actions function doesn't need those locals,
- You dont need to register an action for the trigger, you could simply register a condition
and use an if/then/endif block and return false at the end.
- You could omit the super slow SquareRoot function by comparing with 14400 instead of 120
- You need to check if the unit's life is less than 0.405 to see if it's dead.. not 0
- Don't use "WEAPON_TYPE_WHOKNOWS", just use "null".
- You aren't nulling Handle Vars.

Synergy: 8.5/10

The abilities could be used together to totally isolate and kill an enemy hero easily.
None of them were random and they were quite relevant to each other, so I'm giving you a high score

Balance: 7.5/10

He's really overpowered. You could deal huge amounts of damage and disable enemy heroes easily.
Still, he is pretty easy to beat with some teamwork.. I'd give you a 7.5

Aesthetics: 5/5

The icons were quite relevant, the model was good, and the tooltips were decent :3

Total: 35/45



Spinnaker:


Concept: 9.5/10

Such an amazing hero.. I'm speechless.

Coding: 9/10

The only negatives I saw in your code were occasions were you used useless locals and the fact that you
actually used TriggerRegisterTimerEvent when you could've used timers.

Synergy: 9/10

Try using Shadow Bite then Howl of Fear to force enemies to run away and you'll know why I'm giving you a good score
You could also use Shadow Bite and force your enemies to stay around you just so they don't move, giving you the chance to kill them easily!
Your Wolfs Adaptation ability helps here too

Balance: 9.5/10

He may have powerful abilities, but he's still vulnerable, making him very balanced

Aesthetics: 4/5

Everything was perfect.. Except for those errors in the tooltips..

Total: 41.5/45



BrambleClaw:


Concept: 8.5/10

Pretty good concept

Coding: 3.5/10

1- Way too simple
2- Using waits
3- Some things could be shortened by using loops

Synergy: 8/10

The abilities were very related to eachother which is quite good

Balance: 9.5/10

Your hero is quite balanced when it comes to his stats.
His abilities were also quite balanced too

Aesthetics: 5/5

The tooltips were good, the icons fit, and the model was great.

Total: 34.5/45



Blood Raven:


Concept: 8.5/10

Your concept was quite good

Coding: 0/10

No code... Only simple object editor work...

Synergy: 8.5/10

The abilities go great together
You could web an enemy unit and keep it in range of the Swarm. The poison arrows could help keep that unit closer to Swarm by
decreasing the movement speed of enemy units, etc..

Balance: 9/10

Your hero's vulnerability is balanced by her powerful abilities.

Aesthetics: 4.5/5

The icon of "Poison Arrows" needs to be changed to show that it could be cast automatically.

Total: 30.5/45




Zeatherann:


Concept: 7.5/10

The concept was ok..

Coding: 9/10

If you:
- Omitted those 2 redundant lines in function FN_BeastsEnum (set x=GetUnitX(..), set y=GetUnitY(..))
- Registered conditions instead of actions
- Used GetSpellTargetX and GetSpellTargetY instead of the locations
- Used a global dummy for tree detection

You would've gotten a 10 :3

Synergy: 9/10

Together the abilities could be used to:
- Trap enemy units (Wrath of Nature)
- Give your team an advantage (Red Moss)
- Give the enemies a disadvantage (Green Moss)

Balance: 9/10

Your hero is almost perfectly balanced

Aesthetics: 5/5

Model -> Good
Icons -> Good
Tooltips -> Good

Total: 39.5/45



Mckill2009:


Concept: 7.5/10

It was ok.. Though, it's not so clear that the hero is animalistic, the theme was presented ... well..

Coding: 9.5/10

The code was excellent, but in one of the functions, you actually repeated GetTriggerUnit() about 5 times.
Also, I don't see why you don't insist on using T32

Synergy: 5/10

The abilities didnt really go together well :/
(Other than the passive ability that strengthened your first ability)

Balance: 8/10

Your hero needs to be a 'bit' stronger, but she was still ok :S

Aesthetics: 5/5

Perfect icons, model, and tooltips

Total: 35/45



Gunslinger21:

Side Note: Next time, make sure you give a good testmap.. I had to write my own -.-

Concept: 8/10

The concept was good..

Coding: 4/10

- You're using waits
- You're repeating functions instead of storing them in variables
- You used (Casting Unit) once instead of (Triggering Unit)
- You used (Ability Being Cast) in the code...

Synergy: 9/10

The hero's ultimate ability strengthens his other abilities and his "Blue Howl" can help the wolves deal
more damage to enemy units.

Balance: 8.5/10

Your hero is quite balanced according to my tests

Aesthetics: 3.5/5

Your tooltips have errors, and one of your icons isn't correctly implemented

Total: 33/45




Bugz:


Concept: 9/10

The concept of a shaman who transforms to a dragon is quite interesting if you ask me.

Coding: 0/10

What is this I don't even..

Synergy: 3/10

The abilities couldn't be used together strategically, but they fit the theme.. well, 3 of them did..

Balance: 6.5/10

He needs a bit of balancing :/
A 60% Attack rate bonus is way too much >.<

Aesthetics: 3/5

The tooltips need some work..
The icons are ok, and the models are excellent.

Total: 21.5/45

Adiktuz review

Ahimtar - THE BROODFATHER

Judging

8/10
-The idea is nice, though not that original.

1/10
-Uses Waits (LOTS OF THEM)
-You could have compressed the AoE damage into just one action, and used a formula 125 + 75*Level of blahblah
-You leak locations on the VenomBolt AoE trigger
-The region was useless, you should have just used, Units within range
-I don't see the point of doing the VenomBolt AoE damage inside a player group loop (it will actually result to the damage being dealt more than once (it will hit once per player that is an enemy of the caster), and you didn't even used the player)
-Not sure if its MUI but at least, the trigger will stop once the variables reach the max value possible
-The other spells could bug if there is more than one Broodfather
-The create corpse trigger also leaks a lot of locations

9.5/10
-The abilities work good with each other and fit the broodfather theme
-Though why does spiders always have to be combined with poison?

7/10
-I suggest increasing the mana cost of Summon Spiders a bit.
-He was not as OP as I initially thought as I realized the spiders were an easy money source if you have
someone like the dreadlord.
-Though early game, he would be a good creep killer.
-Maybe you could nerf the bonuses of True Form a bit.

3/5
-The hotkey of Summon Spider is S which collides with the hotkey of Stop thus rendering the hotkey of Summon Spider unusable.
-True Form could have used another icon as it looks the same as the SS skill.
-And maybe try to find a better model, I don't kinda like the Archimonde thing.

28.5/45
-I liked the idea but this needs a LOT of OPTIMIZATION AND CLEANING... ^_^

BloodRaven - Kashaya

Judging

7/10
-The idea was just fine, though not that original and fun

3/10
-Lack of proper setting of data on the OE (evident on the tooltip problems)
-Lack of a bit of imagination
-You did not even bothered to make custom buffs for Spawn and Web when it just takes a few clicks to do that

6/10
-The abilities fit the theme, but it does not work good with each other

6/10
-Basing Spawn on Doom I think is a bit OP for a normal skill

.5/5
-The tooltips of the 2nd and third skills doesn't show level
-You don't play an animation when using the ultimate
-Change the icon of the ultimate
-Web says that it binds a unit for 0 seconds
-You should mention that PA reduces speed
-The tooltips give wrong data, like for the ulti it says 60 damage while on the OE the damage was set to only 50 etc.
-Something tells me that you didn't even bothered to check the tooltips in-game

22.5/45
-Needs to be reworked, and put more effort into it... ^_^

Brambleclaw - The Fallen

Judging

5/10
-The idea really was not good

1.5/10
-Spells are not MUI
-Uses useless (and highly unoptimized) WEU-only add-ons (like Set Hero Str which can be done via Hero - Modify Attributes)
-Uses lots of Waits
-Uses arrays when they aren't needed (Several variables are better than using an array in which you only use about 2 indexes)
-Uses the event Begins Casting which should be replaced by Starts the Effect

4/10
-I dont see synergy on his abilities and to the hero, and they make the theme vague

4/10
-The set-up was towards an OP one, especially that instant kill on low level creeps
-And you actually have 5 abilities by default, while you were only supposed to have only 4 default abilities by the rules

.25/5
-Icons have no DISBTN
-Tooltips need to reworked as they are hard to read right now
-Use the normal hotkey color for the hotkeys
-The icons were a bit too random
-The positioning of the 2nd and 3rd skill is swapped

14.75/45
-Of all the entries, this one seems to have the least effort spent (or if effort is not the right word, seriousness)... ^_^

Bugz - Zuluhed the Whacked

Judging

6/10
-Well, he was just a dragon warrior

4/10
-Poor OE work, simple edits and still has errors

5/10
-An AS aura doesn't seem to fit an intelligence hero
-His abilities doesn't really have synergy (well maybe aside from the 3rd and ulti) though its not that good too, considering that he was an INT hero and has a low HP

5/10
-Pretty weak
-I wasn't able to check ray as it doesn't allow self target and the only enemy was non undead

1/5
-The icons don't have DISBTN
-The tooltips could be improved a bit
-The tooltips for the first skill don't match, learn is Dragons breath, but skill is Healing Ray
-The hotkeys also don't work

21/45
-Yeah, its a WIP... ^_^

Champara Bros - Nagini-The Serpent Dancer

Judging

9.5/10
-This hero concept is cool and its also fun to use.
-Though it might have been better to make it an agility hero.

2/10
-Use Starts the effect of an ability rather than begins casting.
-I suggest using a generic dummy caster rather than a creating and hiding a peasant, and lower the timed-life.
-You leak unit group and locations on the PA trigger (and absolutely no effort done to remove leaks).
-I will always prefer a DDS than using the event Unit - A unit is attacked for spell triggers.
-Also, the spells are not MUI (I believe they should be MUI).
-And you also leak a lot on the Wait alternative trigger (again no efforts to clear leaks).

9.5/10
-Nagini's abilities all enhance his capability to kill units while also giving him the capability to escape dire situations.
-Though maybe a bit too close ranged for an intelligence hero.

8/10
-Well, the hero was a bit over-powered

4.9/5
-The model used was good and so are the icons and are fitting for the concept.
-The tooltips are also good.
-You could have changed the 1st skill's hotkey to E, and the second one's to T to make them a bit closer on most keyboards.

33.9/45
I really liked using the hero, but it will need a bit of balancing and lots of cleaning on the triggers. ^_^

emjlr3 - Perrin Goldeneyes, the Wolfbrother

Judging

8/10
-I liked the idea of a hammer wielding wolfmaster which can turn into a wolf for a bit, and it kinda reminded me of the pack master of Battle Realms.

6.5/10
-you could have used module initializers for a bit of optimization
-you could have added some configuration vars like for the raw codes
-I believe its better if you detect it too if he attacks (for the battlelust), since on the current set-up he won't be considered on battle if he for some reason gained invulnerablity
-I think you could have merged the actions for Feral (the separate actions if he has thrown the hammer or not etc) for a shorter code or at least modified the if-then-elses a bit for optimization (and put them inside the first if-then block, which will enable you to remove some of the boolean checks).
-Since you have imported TimerUtils, why not utilize it like replacing ALL CreateTimer with NewTimer?
-and its a bit of searching thru the whole codes if someone wants to modify and use your ideas... ^_^
-And read Maggy's judging as I believe its more detailed... ^_^

8/10
-no matter how I look at it, this hero is focused on mashing his enemies.

8/10
-The hero for me has a fine sense of balance. Its not strong and not weak either.

3/5
-the icons and the unit is fine.
-the tooltips could have been improved, especially for the third one and the ultimate (you could have written down the amount of bonuses that they give).

33.5/45
-Overall, I liked the concept and the execution of the idea... ^_^

Gunslinger21 - Nightmare

Judging

8/10
-The concept was nice

4.5/10
-Don't use waits
-Lower the life timer of dummy casters
-An error message for a failed SW will be nice
-The ultimate is a bit non-MUI, if at least one Nightmare learns the ultimate, they will all gain some of its bonuses like the removal of the HP price of SW
-Instead of creating a dummy unit to deal the price of SW, just reduce the triggering unit's hit point.
-TriggeringUnit is faster than casting unit, and you might want to save TU first into a variable if ur using it a lot

6/10
-His abilities seem to synergize a bit but not that good

8/10
-He seems to be balanced just fine

3/5
-The icons are fitting
-Blue howl doesn't have a DISBTN
-The additional ability given by your 2nd skill interferes with the positioning of the other abilities

29.5/45
-Its nice but needs a bit of work for fixing and stuff. ^_^

Infinite Answers - Eagle Warrior

Judging

7/10
Well, I kinda liked the concept of the hero

2/10
-The recycling for Soar Index when someone casts Dive is wrong, and will result to a lot of bugs. I believed you used indexing to try to make it MUI, but on your current set-up, it won't be MUI (I won't elaborate more on this as it won't matter anymore because of the next comment).
-For the Dive skill, it would have been better if you reduce his height gradually until he hits the ground (which means a total redo of the Dive Trigger).
-You could also have made him play an animation upon his impact on the ground.

-For the FW MS buff, I would prefer the use of a hidden spellbook rather than a dummy which is moved every .1 second as using the hidden spellbook method will reduce the number of actions run by the engine (since you will only need to run triggers when you learn and level the skill)... It will also remove the need for those array variables and remove the need for a dummy per hero... Therefore will result to a better performance.

-I don't see the point of using Arrays for the Demoralizing Screech ability as it was an instant effect...
-Also, you leak locations for the Demoralizing SScreech ability as you have wrongly placed the RemoveLocation call (the removal of the FearPoint should be inside the group loop, and you also forgot to Remove the FearedUnit[] location.. I would also be better if you removed the DS_Point right after you used it)...
-It would also have been better if they run off based on the angle between you and them so that they will always run away from you, because right now, if the unit is facing away from you, he will actually walk towards you rather than away from you...
-You could have also increased the fear distance a bit

-For swift assault, the location will leak, why?
For example we use SA_Index = 1
You set SA_Index to SA_Index + 1 ==> SA_Index is now 2
then you set SA_Point[SA_Index] ==> SA_Point[2]
then you set SA_Index to SA_Index - 1 ==> SA_Index is now 1
then you call RemoveLocation(SA_Point[SA_Index]) ==> which removes SA_Point[1], but the location you used was saved on SA_Point[2]
-And Again I don't see the point of using arrays as the effect was instant

-For Terrifying Screech, the same comments as that on Demoralizing Screech...

4/10
-I don't kinda get a good synergy from his skills
-Well, FW and Screech has a bit of it especially if the run issue is fixed
-Also, seeing that he is a strength hero, the skills are kinda not that fitting (the evasion for example has a very low chance, and could have been replaced by a damage reducing ability instead as its more fitting for a Strength hero)
-And the only useful thing about his ultimate was the screech ability of the bird which can disarm for 2 seconds (though I cannot consider this much for synergy as the original hero is already dead)

8/10
-The hero I think was a bit moderate, though some of his abilities can be easily countered (the fear part of DS for example, can be countered easily if the enemy always issues orders on their units)

4.75/5
-You should have indicated on the tooltip of his ultimate that it is activated upon death.

25.75/45
I liked the idea when I saw it, but after testing and checking the editor, this needs to be reworked (especially on the triggering side). ^_^

Jazztastic - Beast Kin Berzerker

Judging

5/10
-Well the concept is not that new, and its just like those same old barbarians

4/10
-local units/locations etc should be nulled at the end.
-you should avoid usage of WAITS as much as possible.
-You never used the locaiton saved on CASTERLOC and also never destroyed it = leak.
-Also, you can merge the local part and the set part (like local unit LS_CASTER = GetTriggerUnit()).
-And since you used Custom script, might as well change that SetUnitVertexColorBJ to its native.
-You might want to set Last Created Unit into a variable
-On your hashtable saving, You can actually remove the variables and just put the formula/value directly on the saving part.
-For his ulti, use the native functions rather than that SetUnitAbilityLevelSwapped and GetUnitAbilityLevelSwapped which are BJs.

9.5/10
-The synergy of the abilities are kinda good, the third one and the ulti for fighting, while a combo of the first and second is a good initializer

6/10
-The third skill is overpowered, 30% recovery upon creep kill and perma strength per hero kill is too much for a melee hero specially combined with his ultimate which gives him +2 str per hit.

1/5
-The tooltips of the first and second skill says Drunken Haze and War Stomp (which pretty much shows lack of effort to check a bit).
-The tooltip of PI learn and the actual PI does not match, the learn says he gains 1 strength per hero kill while the actual tooltip scales it by the level of PI
-You could also organize them a bit.

25.5/45
-I did not liked the hero that much, maybe because the idea was too common (as its just like those barbarians from a lot of games)

mckill2009 - Jastra, the dark elf

Judging

7/10
-The concept was fine though not that much fun

5/10
-use module initializer for LR
-you might need to null the local timer on the create method of LR
-since DATA is already of the type thistype, there isn't a need for a local thistype anymore (though not much diff I guess, and I know you used thistype this = DATA since its faster to type .blahblah than DATA.blahblah)
-Locustcount should be a struct member as right now, it would get affected by all Jastra's on the map causing a possible bug
-I think you could settle for just 1 hashtable for all spells since you just use the handleid of timers (which are specific per instance) for the parent key and 1 for childkey, so you could just use the childkeys 1 for LR, 2 for TW and 3 for FW.
-You might want to use TimerUtils and Bribe's Table for this
-The ulti is cancelled even if only one of the beasts die
-The can control part of the 1st skill doesn't seem to work

4/10
-I don't see much synergy with them (or none at all)
-I can't seem to decipher the relationship of them to the hero theme too

5/10
-Seems a bit weak
-Its not a good idea to have an ability which only affects 1 of the others (talking about the 3rd skill)

3/5
-The DE theme wasn't that much established aside from the unit model itself
-I suggest a change of the ulti's icon too

24/45
-This hero needs work especially in establishing its chosen theme... ^_^

Orcnet - Tikbalang

Judging

9/10
-Really nice concept taken from folklore. ^_^

4/10
-Abyssal is not MUI and uses waits (and on this case is much useless as you already have a periodic trigger, you could have placed the actions after the wait into the part where the periodic loop is turned off)
-The OE work was just basic and not much manipulation has been done.

9/10
-1st, then 2nd then Ultimate would be a nice combo unless they have a stunner
-I suggest a bit of replacement for the third skill, maybe slow rather than armor reduction, as it would work better for his abilities

4/10
-Lower the range of the 1st skill
-I cannot test it out very well as you did not place enemies on the test map so for me, that would be equal to low score (by reading the criteria you should have at least sensed a need for putting enemies as the balance part says: "How balanced the hero is? Is it extremely weak to own or extremely hard to face?")

2.5/5
-The icons look nice and fits with each other
-The tooltips can use a bit of simplification, like merging the damage data into the main text for the actual tooltips
-And some errors in the wordings like "Deals 250 of damage" for your first skill
-The model is also fine
-The hotkeys could have been fit into the ability names just like the default blizz ones (suggestion only)
-The buff tooltips could have a bit of work (for crumble, I think, hit by crumble is better as its not the unit that is crumbled, and for the gloom, in the presence of the Tikbalang)

28.5/45
-The hero was fine on first look, but it needs a bit of work

Spinnaker - Narght the Night Stalker

Judging

9/10
-I really liked the idea of this hero

5/10
-Instead of if not (GetWidgetLife < .405), just use if GetWidgetLife >= .405
-Combine your init functions for the LOTP skill (and the other skills too, coz your current set-up is already a waste of triggs)
-Use ModuleInits instead of your trigger inits
-Conditions are better to use than Actions as they are run before actions
-Combine the ability conds and actions into the conds
-You could have combined LOTP and RemoveWolf into one
-Instead of functions for attack type and damage type, just use globals

7.5/10
-His abilities seems to work fine with each other and with the theme of the hero

7/10
-Balance seems fine

4.5/5
-Almost everything is fine and fitting
-I suggest a change of the ulti's icon (its an owl right?)
-And the shadow cloud around you when you use Howl stays a bit longer than the duration of the spell

33/45
-Idea and execution was fine, just needs a bit more work... ^_^

Xiliger - Garruk

Judging

7/10
-The concept was just like same old barbarians but with a bit of twists

7/10
-I suggest usage of module initializers
-You should combine the condition and actions (for the JASS scripts)
-local units should be nulled, and also the usage of the local units on the initial actions part of the charge is useless as you only use them once
-I don't suggest using pause unit for stuns
-And don't use waits

9/10
-His abilities seem to fit the concept and to help each other out

8/10
-Overall, he seems to be somehow balanced though a bit leaning towards OP; I suggest a bit of increase in the mana cost of his abilities

4.5/5
-The model, icons and tooltip are good
-Though some of the tooltips can be simplified a bit
-I was just thinking that maybe you could replace the third skills icon as the current one was a troll

35.5/45
-Overall, the idea is good, and the execution was also good... ^_^

Zeatherann - Father Nature

Judging

7/10
-The idea was fine

2/10
-The variable setting for the ability rawcodes for FN_Spells_Main should only be done once and not for every run of that skill
-Use Conditions rather than Actions as they are run first
-It would be better to add some spaces or comments around your code, as they are a bit too clamped together
-The summons of your 2nd skill are created on the lower left part of the map and not on the target area
-WoN lvl3 doesn't have a cooldown
-Its also a bad idea to combine all of them into one trigger (on the TE)

7/10
-His 2nd,3rd and ulti has a fine synergy as far as I'm concerned

5/10
-Since some of the spells did not work correctly, the balance was hard to figure out
-But on the point were the spells are all usable (on the lower left of the map), he seems to be a bit OP

3/5
-You could have chosen better icons especially for green and red moss
-He shows death anim when transforming

24/45
-This needs work... ^_^

---

((Votes / Total votes) * 30) + ((Judge 1 + Judge 2) * (70 / 90))

Ahimtar: 45,111832
Blood Raven: 46,287157
Brambleclaw: 39,084775
Bugz: 35,0036
Champara Bros: 54,7591
emjlr3: 59,8924
GunSlinger21: 50,16955
InfinateAnswers: 47,6468
Jazztastic: 45,1118
mckill2009: 46,2784
Orcnet: 48,6182
Spinnaker: 63,78859
Xiliger: 55,6125
Zeatherann: 50,1681

Ranking

1) Spinnaker
2) emjlr3
3) Xiliger
4) Champara Bros
5) GunSlinger21
6) Zeatherann
7) Orcnet
8) InfinateAnswers
9) Blood Raven
10) mckill2009
11) Ahimtar & Jazztastic
12) Brambleclaw
13) Bugz

---

Contest | Poll

 

[Pharaoh_]

Congratulations to the winners!

Thank you everyone for participating in this contest.

 

[Bannar]

ME!!!!!!!!!!!!!!!!!!!!?????????? Thanks everyone!

Ton of thanks (and <3) to Pharaoh_ for hosting the contest - it's a pleasure to work with you.

Okey so now, my word to judges:

Adiktuz - I don't know vJass yet so ModuleInit.. what is module? . You are right with combining, and attack/damage globals (however, I changed those from globals into functions after looking on hive's jass section - there is ton of function usage.. ^^)
I'm used to not GetWidgetLife(), I see nothing wrong with that - but I think it's all about habits.

Maggy - you haven't found anything!?

To both judges: You both missed few issues (I missed just 1 from Adiktuz in my fixing):
- LOTP should ignore the sleeping targets
- LOTP should auto vanish wolfs if target if dead (it does, but after 8 sec, and 'waiting wolf' is still dangerous)
- My ultimate uses dummies of type: Worker (so worker icon appears when hero moves)
- GUYS? YOU DIDN't FIND ANY LEAKKKKKKK? My HoF leaks Effect handle a lot.

 

[Blood Raven]

Congratulations, Spinnaker and the other winners
What a loser i am

 

[Champara Bros]

Adiktuz I demand that you post me an explanation on how my spells are not MUI. I personally tested them and they work when two units cast them at the same time. Usually I would not object but I am but a point away from the third place.

 

[InfinateAnswers]

Hahaha im not joining contests anymore.

 

[Bannar]

Yeah (judges) you were right, poeple will hate you after result (=

@Champara Bros I'm with you, but on the other hand it's just Adiktuz opinion and you have to respect it. I can say the same: why my hero has lower synergy/balance points than (for eg) emjlr3 since his hero is overpowered, and synergies aren't something special either? I won't say anything about coding (because Adiktuz smashed everyone ^^) - I see that I did some mistakes, although every coder got his habits too.

 

[Champara Bros]

Yeah (judges) you were right, poeple will hate you after result (=

@Champara Bros I'm with you, but on the other hand it's just Adiktuz opinion and you have to respect it. I can say the same: why my hero has lower synergy/balance points than (for eg) emjlr3 since his hero is overpowered, and synergies aren't something special either). I won't say anything about coding (because Adiktuz smashed everyone ^^) - I see that I did some mistakes, although every coder got it's habits too.

thanks for support but spells not being mui is HUGE and I am point away from third place. I have every right to demand my third place if I deserved it. I only scored 2/10 in coding. Imagine how much would my spells actually being MUI contribute to my score. So yes I respect his opinion, in fact I am asking him to present to me on what plane my spells are not MUI. It is a pretty huge claim. BTW Congratulations on first place you really deserved it.

 

[Skycraft]

Congratulations to the winners!
Good effort on those who attempted the contest.

And many thanks to Pharaoh_ for hosting the contest.

 

[Garfield1337]

I sense an euphoria in Spinnaker ;D
Gz

 

[CeDiL]

Congratulations Spinnaker, i think you've deserved it. Hope to challenge with you in the next contest.

 

[Orcnet]

Well this is what happens when I break the fourth wall, anyways congrats to everyone who were part of this contest, to pharaoh, maggy, and adiktuz thanks for the results

I will now lurk in the techtree contest

 

[Ahimtar]

well, fml. i knew i am not very good at coding, but 4 points together??? thats just awful.

 

[Magtheridon96]

1 comment:
You don't need Module Initializers for spells, only systems

LOTP should auto vanish wolfs if target if dead (it does, but after 8 sec, and 'waiting wolf' is still dangerous)

I was aware of that bug
But since the wolf vanished after 8 seconds, it wasn't sooo bad

Oh and:

Bugz:


Concept: 9/10

The concept of a shaman who transforms to a dragon is quite interesting if you ask me.

Coding: 0/10

What is this I don't even..

Synergy: 3/10

The abilities couldn't be used together strategically, but they fit the theme.. well, 3 of them did..

Balance: 6.5/10

He needs a bit of balancing :/
A 60% Attack rate bonus is way too much >.<

Aesthetics: 3/5

The tooltips need some work..
The icons are ok, and the models are excellent.

Total: 21.5/45

xD

 

[Bannar]

LOL MAGGY!?
I'm gonna spread more and give you adidtional points for being a freak!

 

[Magtheridon96]

LOL MAGGY!?

I don't really get what you're loling at in my above statement ^^

edit

Adiktuz, Champara Bros spells were MUI, but the indexing was horrible.
I think it's fair to at least mention that :/

 

[Adiktuz]

I sent Champara Bros details about the query... I believe it explained everything... haha... and the majority of his deduction was from the fact that no effort was done for clearing leaks (and it has a hell lot of leaks)...

anyway, Congratz to the winners... ^_^

 

[Champara Bros]

As I explained to adiktuz there is no custom value bug, because it is used on dummies for saving purposes only. Dummies are not suspectable to change. Dummies were removed. So practicaly I lost 8 points on leaks.

 

[Adiktuz]

And impratical use of dummies (which on this case was a peasant unit)

Why?
-you needed to hide the peasant etc
-peasants created will add points to the owner

so you should have just used a dummy caster that is made in a way that it is already invisible and does not give any point to its owner when created...

and about your leaks, it was a hell lot of leaks and I'm sure I already said from the start that leaks will result on large deduction, and on this case the fact that you actually did not put effort into removing any leak just worsened the case...

Spoiler

Come on man, its a friendly competition, at least show some sportsmanship... you're just sulking since you lost the third place by 1.xx points... I think you wouldn't even have bothered to ask if this was not the case (as on your PM you actually said that you're asking since you lost only by 1.1 points)...

My sincerest apologies if the message sounds bad, maybe its just because I don't have any sleep yet for the past 24 hours... So I'm gonna sleep in a little while...

 

[Champara Bros]

And impratical use of dummies (which on this case was a peasant unit)

Why?
-you needed to hide the peasant etc
-peasants created will add points to the owner

so you should have just used a dummy caster that is made in a way that it is already invisible and does not give any point to its owner when created...

and about your leaks, it was a hell lot of leaks and I'm sure I already said from the start that leaks will result on large deduction, and on this case the fact that you actually did not put effort into removing any leak just worsened the case...

Come on man, its a friendly competition, at least show some sportsmanship... you're just sulking since you lost the third place by 1.xx points... I think you wouldn't even have bothered to ask if this was not the case (as on your PM you actually said that you're asking since you lost only by 1.1 points)...

I am not sulking. I just want to know judging pattern. I clearly asked you some questions regarding my review. I just asked why did you write my spells were not MUI. I am surprised that people actually used waits in their submissions. I heard that waits are really bad. You told me non clear and non understandable answer. So I asked you to clarify MUI part. I am not retarded to not understand the meaning of deducting the points based of leaks. I asked how can my MUI system bug. Nothing else, I do not care about. I do not ask you to justify yourself I am just asking you to point out desyncs, crashes, bugs, lags etc. in my mui system. NOTHING ELSE.

 

[Adiktuz]

The case would be different if you haven't said on your first PM the thing about you losing by just 1.1 points... Somehow that line gave me the impression that ur only after some corrections since that might enable you to get the third place...

Anyway, for the judging of the code,

-lots of points were deducted from leaks (more leaks = more points lost), and effort on removing leaks also play a part...

-the next thing that get a reduction is usage of waits...

-then its other simpler things like bad indexing, repeated usage of function calls which can be avoided via variables, unnecessary use of arrays, and the others...

Note: If I don't reply, it means I'm already sleeping...

 

[GunSlinger]

5th place on my first try... Again, like in the short story contest. So happy!


Anyways, congrats to thy winners!

 

[Champara Bros]

Don't blame me. You are the judge who failed to recognize MUI system. I have every right to demand an explanation. Besides my mui system works fine disregarding the leaks I didn't remove. You are not allowed to deduct points because you would do something differently. If it works properly for a mui system and you can not name apparent bugs leave it be. Next time if you may, you join the contest and do anything the way you want, but as a judge you decide whose work will be rewarded. You are not allowed to easily dismiss something without proper arguments. You are not allowed to trash contestants work based on your thoughts. So you failed to explain to me why my mui system is so wrong. That means that you are rejecting responsibility for you actions. If I judge any contest, I will not rate it by something i would do. Often I see a model on the site that I would do so much better, but I do not automatically rate it with 1 and trash it online. that is not sportsmanship. I am not saying that I did best and spinnaker is an idiot and xiliger is cheater and emj3 is sucking up to the important people. I am asking you what you are responsible to tell me. So do not post again how you hate the leaks. I cant imagine judge who can deduct more than half points on leaks. Leaks should have taken certain amount out of 10. You are not allowed to give points as you please, based of your opinion.

 

[Skycraft]

@Champara Bros,

There is no need to be a sore loser, it's just a competition.
Get over it and do better next time.

 

[Adiktuz]

I believe I already said on my PM that I did not deduct points for the MUI thing on your entry as I was unsure of it (which Maggy already cleared out that it was in fact MUI)... and leaks are still leaks, and on your entry there was a hell lot of leaks... I believe its just right to deduct a lot of points from it...

Also, I believed that I emphasized that your reduction due to leaks got higher because of two factors: 1) you have a lot of leak, 2)You did not put effort into cleaning even one leak which based from your PMs where just due to you being lazy...

leak counting

and I do not count leaks via the number of actions leaking I see in the editor but based on how many times that action would be run... So let's say two triggs both have 1 leaking action

Trigger 1 only runs that action once per cast

Trigger 2 runs that action for every unit picked each cast

On this case, Trigg 1 will only leak once per cast while Trigg 2 will leak an indefinite amount of times per cast depending on the number of units inside the group...

And for me, putting the leak removal on a wrong place is far better than not trying to remove a leak at all...

Also, you cannot say that the deduction of huge amount of points from you due to leaks is not right because I use the same point system for all of you meaning I judged all of your entries from the same point of views, the same scale etc...

Also, judges do give points as they please based on their standards... How can you judge something if you don't have a standard to base it from eh?

Its not just you who got lots of deductions due to leaks (which will be apparent if you take time to read the scoring for your competitors) so don't whine about me giving a lot of minus for the leaks...

 

[Champara Bros]

I am not asking you about leaks, I am just shocked that your review totally disregarded MUI. Which is weird. MUI is being highly valued. So I expected that if you realize my spells were mui, you will increase my points by 1.5 at least. But you did not, and when I asked you why, you said my system was bug prone. I asked you why and you had no clue. So if that means being a sore loser then I am sore loser. As I stated over pm's if there is something wrong with system( NOT LEAKS !!!) that you post it. You did not. So my system is wrong, you don't know why, but still you would not increase my points, because you are not a man enough to accept you were wrong. Also you totally disregarded my OE work. So lay of leaks.
END OF DISCUSSION FROM ME.

 

[Magtheridon96]

Actually Champara Bros, a judge can either judge you based on what you presented, or based on what you lacked.
For Aesthetics, I just gave everyone a 5/5 if they gave me a decent model, decent icons and decent tooltips.
For coding, if you had leaks, I wouldn't give you more than an 7.5/10 (Leaks have a huge impact on your code)
If you repeated functions, I took off a point.
If your spells werent MUI, I took off a point or 2 depending on how many spells werent MUI
Anything else would lead me to take off 0.25 to 0.5 points

I didnt care about algorithms, I cared about efficiency

Also you totally disregarded my OE work

>_>

you said my system was bug prone

Not totally bug prone, just really bad
Here's a useful link for future refference: free porn Dynamic Indexing

 

[Adiktuz]

Why will I add 1.5 points when I did not reduce the points in the first place? I believe I said it a matter of times... And I also said that I was unsure when I wrote down it wasn't MUI (which is why I did not reduce the points), isn't that already accepting that I made a mistake?

And I said right from the start that I'm judging via reduction meaning all of the entries were assumed to be "perfect" at the start (hence a perfect score for each part), then things which make it deviate from the "perfect" form will result to deductions...

Meaning, on your case,I found your OE work to be good enough so as not to cause any further deductions from your score...

 

[Champara Bros]

Actually Champara Bros, a judge can either judge you based on what you presented, or based on what you lacked.
For Aesthetics, I just gave everyone a 5/5 if they gave me a decent model, decent icons and decent tooltips.
For coding, if you had leaks, I wouldn't give you more than an 7.5/10 (Leaks have a huge impact on your code)
If you repeated functions, I took off a point.
If your spells werent MUI, I took off a point or 2 depending on how many spells werent MUI
Anything else would lead me to take off 0.25 to 0.5 points

I didnt care about algorithms, I cared about efficiency



>_>



Not totally bug prone, just really bad
Here's a useful link for future refference: free porn Dynamic Indexing

I never said anything about you. You are great judge as i mentioned before. + rep for doing adiktuz's work.
And adiktuz how can you not add points for spell being mui. in you review you wrote that you believed my spells were not mui so I assumed that you deducted points there so I objected.
P.S. Maggy is a better judge. He gave me answer to my question.

 

[Adiktuz]

Its your assumption, not everything that was written was pertaining to point deduction as I also include some suggestions... And you also have a bit of error in continuously saying that I should give you back the points when I already said in the earlier parts that I did not reduce the points... Its not my problem anymore if you don't believe it...

And I know that Maggy is better right from the start as his reviews are always much much deeper, but this is never a contest between who is a better judge anyway... haha... though from my point of view, explaining isn't really a part of being a judge in the real world as most contests only require the judge to pass the points without any explanation... its actually a good thing that contests here require that part... ^_^

anyway, its better for you if we just stop this now, its adverse effect is only upon you...

 

[Pyritie]

spinnaker, what do you want on your award icon?

 

[Bannar]

I've never thought that you choose the icon by yourself ;P Give me a second okey? Meaby Pharaoh_'s avatar ^^

EDIT: Okey, I decided. Let me the icon of hero who made me win this contest. I was thinking about the Far Seer's wolves too, but it would be shame to not choose the hero's icon as an award.

Nargth the Night Stalker (BTN one ofcourse).

 

[Magtheridon96]

Maggy is a better judge

But Adiktuz is like one of my friends on the hive
That just offends him :'(


Oh and Spinnaker, Can I take your hero and implement him into my map?
I'll call him "Spinnaker" - Wolfbane / Netherwolf

 

[Bannar]

^Someone loves me ^^

Sure, you can. You gonna convert him into vJass? I can paste the fixed version (but after techtree contests, that is 4th Sept). I'm still not sure about that one - but if I participate I need a lot of free time.

 

[baassee]

Gosh why do you even question a judge about their judgement? I can certify that both judges have enough knowledge to judge a hero contest (maybe not a zephyr contest though but this isn't one of those) and I just went thru most of the resources and looked at the judgements and yeah there are minor flaws that I would have reconsidered and maybe judged different but still nothing major.

Thumbs up to the judges! (And yes I might be a friend of both the judges and yes I might take their part a bit but as in soccer, it's always the judge that will have the last word.)

 

[Magtheridon96]

Wouldn't it be funny if someone called a judge judgemental?

Sure, you can. You gonna convert him into vJass?

Ofcourse ^^
But, only minor changes would be made

 

[Bannar]

Maggy, hope you know where effect handle leak lies ^^ Because you haven't found it so far :F

 

[Adiktuz]

Waiting to see Spinnaker's award icon on his posts.... ^_^

Maggy is better, it doesn't offend me in anyway as I always respected Magtheridon96 (especially since I used to work on SoE too)... or anyone here which has a fine attitude...

@baassee - I was just thinking why you didn't join this contest... haha...

 

[baassee]

I was away on work for a month.

 

[Bannar]

Omg, I have found my first post, and the guys who gave me first rep. ADIKTUZ YOU WERE ONE OF THEM <3

The end of the begining. Sadly, I were active only for one mount in 2010 :S

 

[Magtheridon96]

Maggy is better, it doesn't offend me in anyway as I always respected Magtheridon96 (especially since I used to work on SoE too)... or anyone here which has a fine attitude...

I remember back in the day when I uploaded my Atomic Bomb spell
Boy that was retarded of me >.<
Anyways, I respect you too ^_^

Oh and btw, you might wanna update your sig cause we both resigned from SoE like 2 months lol
Wolfe hates me now >.<

@Spunky (Your new nickname Spinnaker )
I just implemented your spells and I'm in the process of converting them
I'll send the code as soon as I'm done ^^

 

[emjlr3]

emj3 is sucking up to the important people

you know nothing John Snow.....if you had, you'd realize just how dumb you really sound

in any case, fun contest - I was happy to have the opportunity to put down on paper my favorite character from one of my favorite epic series'.

to be honest - I was a bit disappointed with the ultimate, but was out of time and inspiration, just at a lack for any other more appropriate ideas.

I have quite a bit of glaring differences in how I approach a hero and coding than most, and my viewpoints differs greatly from some of the views the judges seemed to share - I am not going to bitch

if one or both of the judges would like to discuss their comments, feel free to PM me. if not, no harm done.

btw, when do I get my medal of honor?

 

[Bannar]

btw, when do I get my medal of honor?

As soon as Pyrite comes here, but it won't be an award icon if you wan't to know.

Change your language on forum, you are arrogant, immature child. If I were a judge, even that there is no such field you would get -10 for being little kid. There are people who deserve 2nd place much more than you (in my opinion).

 

[Champara Bros]

you know nothing John Snow.....if you had, you'd realize just how dumb you really sound

in any case, fun contest - I was happy to have the opportunity to put down on paper my favorite character from one of my favorite epic series'.

to be honest - I was a bit disappointed with the ultimate, but was out of time and inspiration, just at a lack for any other more appropriate ideas.

I have quite a bit of glaring differences in how I approach a hero and coding than most, and my viewpoints differs greatly from some of the views the judges seemed to share - I am not going to bitch

if one or both of the judges would like to discuss their comments, feel free to PM me. if not, no harm done.

btw, when do I get my medal of honor?

you got my post wrong. When adiktuz said I was sore loser I said that I would be a sore loser if I started insulting people who are better than me and if I discredited their work. So I said that your place was WELL DESERVED. Anyways congratulations on the win. And nexr time read post more carefully. Because I said that I COULD say that spinnaker is an idiot, xiliger is cheater and you are sucking up if I really was a sore loser. But you guys earned your places fair and I was just asking adiktuz why my MUI system was so wrong.

 

[Bannar]

And I'm sure Champara Bros, that no one here negates you for that. There is nothing wrong with asking, since both judges are responsible, futhermore there are here to answer your questions unless they are inaccurate.

 

[Champara Bros]

And I'm sure, Champara Bros that no one here negates you for that. There is nothing wrong with asking, since both judges are responsible, futhermore there are here to answer your questions unless they are inaccurate.

agreed. I am just dissapointed that it was the maggy who in the end answered my question. I thought: "Why was I bugging Adiktuz then ?".

 

[baassee]

Close the thread please.

 

[Deaod]

Spinnaker

- not making use of libraries/scopes at all. Consequently, names are prone to collisions
- using a hashtable instead of a struct
- not capable of handling unlearning
- not recycling groups (thus leaking)
- why not use a damage detection library like DamageEvent?
- why not use an indexing library like AutoIndex? A decent one that doesnt abuse GUI.
- hardcoded, magic values (851986, "origin")

I wonder how you can overlook basic stuff like inadequate data structures?

Seriously, stuff like this doesnt deserve a 9/10, it maybe deserves the 5/10 it got from Adiktuz, although not only for the reasons he listed.
In my opinion, Magtheridon96 is an incompetent judge.

Heres a few things just to make the point:

Don't repeat stuff like (Last Created Unit), use variables instead.

Last Created Unit is a variable, although one defined in blizzard.j. As long as youre only calling that function in the same frame, theres no problem.

Don't use "WEAPON_TYPE_WHOKNOWS", just use "null".

Is there any difference? No? Good. That makes this a useless comment.

You aren't nulling Handle Vars.

Not all handle subtypes need to be nulled.

You used (Casting Unit) once instead of (Triggering Unit)

I forget which one of those is slower, but i do know that the difference is irrelevant and that theres absolutely no reason not to use Casting Unit.

You need to check if the unit's life is less than 0.405 to see if it's dead.. not 0

Actually, when a unit dies, its HP are set to 0. The only case checking against 0.406 [sic, because thats the actual limit] protects you from is when another piece of code increases the units HP to somewhere below 0.406.

How could you know vJass when you used waits in your Cleave spell

Knowing vJass and using Waits has no connection whatsoever.

 

[Garfield1337]

You used (Casting Unit) once instead of (Triggering Unit)

Actually Casting unit calls a native function so it might even be faster.
So it's misleading to say it's better to use Triggering unit.

 

[Bannar]

I wonder how you can overlook basic stuff like inadequate data structures?

Seriously, stuff like this doesnt deserve a 9/10, it maybe deserves the 5/10 it got from Adiktuz, although not only for the reasons he listed.
In my opinion, Magtheridon96 is an incompetent judge.

Heres a few things just to make the point:

Don't repeat stuff like (Last Created Unit), use variables instead.

Last Created Unit is a variable, although one defined in blizzard.j. As long as youre only calling that function in the same frame, theres no problem.

Don't use "WEAPON_TYPE_WHOKNOWS", just use "null".

Is there any difference? No? Good. That makes this a useless comment.

You aren't nulling Handle Vars.

Not all handle subtypes need to be nulled.

You used (Casting Unit) once instead of (Triggering Unit)

I forget which one of those is slower, but i do know that the difference is irrelevant and that theres absolutely no reason not to use Casting Unit.

You need to check if the unit's life is less than 0.405 to see if it's dead.. not 0

Actually, when a unit dies, its HP are set to 0. The only case checking against 0.406 [sic, because thats the actual limit] protects you from is when another piece of code increases the units HP to somewhere below 0.406.

How could you know vJass when you used waits in your Cleave spell

Knowing vJass and using Waits has no connection whatsoever.

^Is that to me too? Hero doesn't have Cleave, handles are nulled, (Last created unit)?? My script checks if hp if greater than 0.405.
(Casting unit)?? (Triggering unit)?? WEAPON_TYPE_WHOKNOWS??

I don't get you Deaod. You must be blind or you just don't pay enought attention to what you are reading.

EDIT: >> I see it was about other participant.

Spinnaker

- not making use of libraries/scopes at all. Consequently, names are prone to collisions
- using a hashtable instead of a struct
- not capable of handling unlearning
- not recycling groups (thus leaking)
- why not use a damage detection library like DamageEvent?
- why not use an indexing library like AutoIndex? A decent one that doesnt abuse GUI.
- hardcoded, magic values (851986, "origin")

Spells aren't written in vJass dude. I'm still learning it. It shuts down 99% of things you have mentioned, furthermore more than 100% counting your blindness. I'm jassing since 1,5 month, so the script was done when it wasn't even a month. Furthermore it was a motivation for me, because whole hero was at first GUI and I've decided: Jass now or never - it was like a quick school.

The contest is over, respect the participants and judges, end of story.