[Solved] Removing unit group

[PassDaSauce]

Just wanna ask what's wrong with my trigger. I can't seem to figure it out, I scoured the forums point out to do it like this.

 

[Pyrogasm]

You did not clean the unit group leak correctly. What is the line at the end suppose to accomplish? You are also leaking a location at the start.

Things That Leak

For a structured tutorial on the matter, read this. Hello guys! Credit to: Wolverabid, PurplePoot, Silvenon, Paskovich I want to make a list of things that leak and ways to fix them. Dynamic Unit Groups This trigger will instantly destroy dynamically generated unit groups. Trigger...

www.hiveworkshop.com

Your trigger will create a unit and then sequentially order that unit to attack every hero on the map, regardless of player force settings. Because the unit cannot attack instantly, only the last hero unit found in the group will actually be targeted and attacked (though the message will display for all heroes found). What are you trying to do?

 

[PassDaSauce]

You did not clean the unit group leak correctly. What is the line at the end suppose to accomplish? You are also leaking a location at the start.

Things That Leak

For a structured tutorial on the matter, read this. Hello guys! Credit to: Wolverabid, PurplePoot, Silvenon, Paskovich I want to make a list of things that leak and ways to fix them. Dynamic Unit Groups This trigger will instantly destroy dynamically generated unit groups. Trigger...

www.hiveworkshop.com

Your trigger will create a unit and then sequentially order that unit to attack every hero on the map, regardless of player force settings. Because the unit cannot attack instantly, only the last hero unit found in the group will actually be targeted and attacked (though the message will display for all heroes found). What are you trying to do?

I'm trying to make it attack or locate a random player when it arrives

 

[Pyrogasm]

There is a "Random unit from <group>" function.

• Set EmpressGroup = (Units in (Playable map area) matching (((Matching Unit) is a Hero) equal to True)) //If there are heroes on the map not owned by players this line will need an additional matching filter to only get units hostile to neutral hostile
• Set EmpressTarget = (Random Unit from EmpressGroup)

Do Nothing is useless and should never be used; you can leave blank Else blocks without issue. Also this for future reference: How To Post Your Trigger

 

[PassDaSauce]

There is a "Random unit from <group>" function.

• Set EmpressGroup = (Units in (Playable map area) matching (((Matching Unit) is a Hero) equal to True)) //If there are heroes on the map not owned by players this line will need an additional matching filter to only get units hostile to neutral hostile
• Set EmpressTarget = (Random Unit from EmpressGroup)

Do Nothing is useless and should never be used; you can leave blank Else blocks without issue. Also this for future reference: How To Post Your Trigger

thank you so much. will post triggers in that format nex time