Suggestions on this map and rounds

[Venge22]

Hi everyone. wat r ur thoughts on this map? And wat triggers would i do to get it to do rounds?

 

[Septimus]

Ok, I going to be brutally honest with you by giving you a tough critique.

First of all, the terrain is completely bland with nothing at it. Bland, boring and 100% dull.

Second, this trigger not only laggy cause it spawn a mass amount of monster at same time. But, have lot's of memory leak at the same time as well.

Untitled Trigger 005
Events
Time - Every 20.00 seconds of game time
Conditions
Actions
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 015 <gen>) facing (Center of Region 015 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 014 <gen>) facing (Center of Region 014 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 013 <gen>) facing (Center of Region 013 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 012 <gen>) facing (Center of Region 012 <gen>)
Unit - Create 10 Abomination for Player 10 (Light Blue) at (Center of Lower Left <gen>) facing (Center of Lower Left <gen>)
Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at (Center of Lower Right <gen>) facing (Center of Lower Right <gen>)
Unit - Create 12 Doom Guard for Player 10 (Light Blue) at (Center of Upper Left <gen>) facing (Center of Upper Left <gen>)
Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at (Center of Upper Right <gen>) facing (Center of Upper Right <gen>)
Unit - Create 11 Zergling for Player 10 (Light Blue) at (Center of Mid Demon Gate <gen>) facing (Center of Mid Demon Gate <gen>)
Unit - Create 9 Infernal for Player 10 (Light Blue) at (Center of Upper Demon Gate <gen>) facing (Center of Upper Demon Gate <gen>)
Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at (Center of Lower Mid Demon Gate <gen>) facing (Center of Lower Mid Demon Gate <gen>)
Unit - Create 8 Infernal for Player 10 (Light Blue) at (Center of Lower Demon Gate <gen>) facing (Center of Lower Demon Gate <gen>)
Unit - Create 9 Doom Guard for Player 10 (Light Blue) at (Center of Left Demon gate <gen>) facing (Center of Left Demon gate <gen>)
Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at (Center of Left Demon Gate <gen>) facing (Center of Left Demon Gate <gen>)

This is the amount of leak found just by using leak checker.

(Line: 6) (Word: 12) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 015 <gen>) facing (Center of Region 015 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 015 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 6) (Word: 18) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 015 <gen>) facing (Center of Region 015 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 015 <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 015 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 7) (Word: 12) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 014 <gen>) facing (Center of Region 014 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 014 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 7) (Word: 18) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 014 <gen>) facing (Center of Region 014 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 014 <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 014 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 8) (Word: 12) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 013 <gen>) facing (Center of Region 013 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 013 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 8) (Word: 18) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 013 <gen>) facing (Center of Region 013 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 013 <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 013 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 9) (Word: 12) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 012 <gen>) facing (Center of Region 012 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 012 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 9) (Word: 18) Location Leak
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 012 <gen>) facing (Center of Region 012 <gen>)
Unit - Create 10 Doom Guard for Player 10 (Light Blue) at (Center of Region 012 <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Region 012 <gen>)
(Suggested Fix) Unit - Create 10 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 10) (Word: 11) Location Leak
Unit - Create 10 Abomination for Player 10 (Light Blue) at (Center of Lower Left <gen>) facing (Center of Lower Left <gen>)
Unit - Create 10 Abomination for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Left <gen>)
(Suggested Fix) Unit - Create 10 Abomination for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 10) (Word: 17) Location Leak
Unit - Create 10 Abomination for Player 10 (Light Blue) at (Center of Lower Left <gen>) facing (Center of Lower Left <gen>)
Unit - Create 10 Abomination for Player 10 (Light Blue) at (Center of Lower Left <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Left <gen>)
(Suggested Fix) Unit - Create 10 Abomination for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 11) (Word: 12) Location Leak
Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at (Center of Lower Right <gen>) facing (Center of Lower Right <gen>)
Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Right <gen>)
(Suggested Fix) Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 11) (Word: 18) Location Leak
Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at (Center of Lower Right <gen>) facing (Center of Lower Right <gen>)
Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at (Center of Lower Right <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Right <gen>)
(Suggested Fix) Unit - Create 10 Fel Stalker for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 12) (Word: 12) Location Leak
Unit - Create 12 Doom Guard for Player 10 (Light Blue) at (Center of Upper Left <gen>) facing (Center of Upper Left <gen>)
Unit - Create 12 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Left <gen>)
(Suggested Fix) Unit - Create 12 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 12) (Word: 18) Location Leak
Unit - Create 12 Doom Guard for Player 10 (Light Blue) at (Center of Upper Left <gen>) facing (Center of Upper Left <gen>)
Unit - Create 12 Doom Guard for Player 10 (Light Blue) at (Center of Upper Left <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Left <gen>)
(Suggested Fix) Unit - Create 12 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 13) (Word: 12) Location Leak
Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at (Center of Upper Right <gen>) facing (Center of Upper Right <gen>)
Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Right <gen>)
(Suggested Fix) Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 13) (Word: 18) Location Leak
Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at (Center of Upper Right <gen>) facing (Center of Upper Right <gen>)
Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at (Center of Upper Right <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Right <gen>)
(Suggested Fix) Unit - Create 10 Sludge Monstrosity for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 14) (Word: 11) Location Leak
Unit - Create 11 Zergling for Player 10 (Light Blue) at (Center of Mid Demon Gate <gen>) facing (Center of Mid Demon Gate <gen>)
Unit - Create 11 Zergling for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Mid Demon Gate <gen>)
(Suggested Fix) Unit - Create 11 Zergling for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 14) (Word: 18) Location Leak
Unit - Create 11 Zergling for Player 10 (Light Blue) at (Center of Mid Demon Gate <gen>) facing (Center of Mid Demon Gate <gen>)
Unit - Create 11 Zergling for Player 10 (Light Blue) at (Center of Mid Demon Gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Mid Demon Gate <gen>)
(Suggested Fix) Unit - Create 11 Zergling for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 15) (Word: 11) Location Leak
Unit - Create 9 Infernal for Player 10 (Light Blue) at (Center of Upper Demon Gate <gen>) facing (Center of Upper Demon Gate <gen>)
Unit - Create 9 Infernal for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Demon Gate <gen>)
(Suggested Fix) Unit - Create 9 Infernal for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 15) (Word: 18) Location Leak
Unit - Create 9 Infernal for Player 10 (Light Blue) at (Center of Upper Demon Gate <gen>) facing (Center of Upper Demon Gate <gen>)
Unit - Create 9 Infernal for Player 10 (Light Blue) at (Center of Upper Demon Gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Upper Demon Gate <gen>)
(Suggested Fix) Unit - Create 9 Infernal for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 16) (Word: 12) Location Leak
Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at (Center of Lower Mid Demon Gate <gen>) facing (Center of Lower Mid Demon Gate <gen>)
Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Mid Demon Gate <gen>)
(Suggested Fix) Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 16) (Word: 20) Location Leak
Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at (Center of Lower Mid Demon Gate <gen>) facing (Center of Lower Mid Demon Gate <gen>)
Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at (Center of Lower Mid Demon Gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Mid Demon Gate <gen>)
(Suggested Fix) Unit - Create 9 Satyr Soulstealer for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 17) (Word: 11) Location Leak
Unit - Create 8 Infernal for Player 10 (Light Blue) at (Center of Lower Demon Gate <gen>) facing (Center of Lower Demon Gate <gen>)
Unit - Create 8 Infernal for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Demon Gate <gen>)
(Suggested Fix) Unit - Create 8 Infernal for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 17) (Word: 18) Location Leak
Unit - Create 8 Infernal for Player 10 (Light Blue) at (Center of Lower Demon Gate <gen>) facing (Center of Lower Demon Gate <gen>)
Unit - Create 8 Infernal for Player 10 (Light Blue) at (Center of Lower Demon Gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Lower Demon Gate <gen>)
(Suggested Fix) Unit - Create 8 Infernal for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 18) (Word: 12) Location Leak
Unit - Create 9 Doom Guard for Player 10 (Light Blue) at (Center of Left Demon gate <gen>) facing (Center of Left Demon gate <gen>)
Unit - Create 9 Doom Guard for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Left Demon gate <gen>)
(Suggested Fix) Unit - Create 9 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 18) (Word: 19) Location Leak
Unit - Create 9 Doom Guard for Player 10 (Light Blue) at (Center of Left Demon gate <gen>) facing (Center of Left Demon gate <gen>)
Unit - Create 9 Doom Guard for Player 10 (Light Blue) at (Center of Left Demon gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Left Demon gate <gen>)
(Suggested Fix) Unit - Create 9 Doom Guard for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 19) (Word: 12) Location Leak
Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at (Center of Left Demon Gate <gen>) facing (Center of Left Demon Gate <gen>)
Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at ^Leak
(Suggested Fix) Set MyLocation = (Center of Left Demon Gate <gen>)
(Suggested Fix) Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

(Line: 19) (Word: 19) Location Leak
Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at (Center of Left Demon Gate <gen>) facing (Center of Left Demon Gate <gen>)
Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at (Center of Left Demon Gate <gen>) facing ^Leak
(Suggested Fix) Set MyLocation = (Center of Left Demon Gate <gen>)
(Suggested Fix) Unit - Create 11 Fel Stalker for Player 10 (Light Blue) at MyLocation facing MyLocation <gen>)
(Suggested Fix) Custom script: call RemoveLocation(udg_MyLocation)

Third, this trigger have been set improper way.

Untitled Trigger 004
Events
Time - TimeRemaining expires
Conditions
Actions
Player Group - Pick every player in (All players) and do (Game - Defeat (Picked player) with the message: U failed to get to ...)

Setting it that way would make all player defeated including the enemy computer. You should have just set it like this.

Untitled Trigger 004
Events
Time - TimeRemaining expires
Conditions
Actions
Player Group - Pick every player in (All allies of Player 1 (Red)) and do (Game - Defeat (Picked player) with the message: U failed to get to ...)

Fourth, you did not fix the starting location of the player. This enable player to use force 2 (Computer player) and the starting location of all player is completely random.

Fifth, instead of making this separate trigger.

Demon Victory settings
Events
Time - TimeRemaining expires
Conditions
Actions
Game - Victory Player 10 (Light Blue) (Show dialogs, Show scores)

Untitled Trigger 004
Events
Time - TimeRemaining expires
Conditions
Actions
Player Group - Pick every player in (All allies of Player 1 (Red)) and do (Game - Defeat (Picked player) with the message: U failed to get to ...)

It would be better if you just combine it into 1 trigger just like this.

Untitled Trigger 004
Events
Time - TimeRemaining expires
Conditions
Actions
Player Group - Pick every player in (All allies of Player 1 (Red)) and do (Game - Defeat (Picked player) with the message: U failed to get to ...)
Game - Victory Player 10 (Light Blue) (Show dialogs, Show scores)

In addition, this setting kinda time consuming although it is acceptable.

Victory Settings
Events
Unit - A unit enters Gate to Heaven <gen>
Conditions
Actions
Game - Victory Player 9 (Gray) (Show dialogs, Show scores)
Game - Victory Player 8 (Pink) (Show dialogs, Show scores)
Game - Victory Player 7 (Green) (Show dialogs, Show scores)
Game - Victory Player 6 (Orange) (Show dialogs, Show scores)
Game - Victory Player 5 (Yellow) (Show dialogs, Show scores)
Game - Victory Player 4 (Purple) (Show dialogs, Show scores)
Game - Victory Player 3 (Teal) (Show dialogs, Show scores)
Game - Victory Player 2 (Blue) (Show dialogs, Show scores)
Game - Victory Player 1 (Red) (Show dialogs, Show scores)

You can just shorten it by setting this way.

Victory Settings
Events
Unit - A unit enters Gate to Heaven <gen>
Conditions
Actions
Player Group - Pick every player in (All allies of Player 1 (Red)) and do (Game - Victory (Picked player) (Show dialogs, Show scores))

Finally, the game is a complete blah cause only have 1 type of hero. Try spice it up by adding several different hero for it.

Judge by this condition of your map, I can knew you are new to the World Editor. This map not ready to be play, there is still plenty of job you need to get it fix before it could even consider as beta stage map.

Score : 0/100
Rating : 1/5

If you think this review is harsh, then had it your way. It is for your own good and I do not want you to receive the heavy ho once you upload it to any website.

With the terrain preview. it would not going to impress me to even download it at all. Here is a example of 1 horrible map that just only have 3 download although it been upload for several hour.

http://www.hiveworkshop.com/forums/resource.php?t=94094&prev=r=20&status=g&page=2