Grab, Throw and Drop MUI

emil23 · Sep 21, 2020

nice! i always needed spell like that, i will dl it once its reviewed

Nike2000 · Sep 23, 2020

If you throw just as the grab hold duration ends it throw the enemy but also creates a copy at your location, dupping the unit.

emil23 · Sep 27, 2020

can you add the pathning checking pls? an option to not be able to throw someone throught tress or inside the tress. it can really lead to many bugs in my map if i use it

emil23 · Sep 27, 2020

emil23 · Oct 10, 2020

noticed 2 more issues, during the caster is throwing the unit, the bag effect is still visible on the caster, so basicaly there are x2 bags , you can see it even on my upside photo, i think it should get fixed because when someone replaces bag to model of unit it will look like the unit is at hands of the caster and its flying at same time,

second thing is there is some delay it takes 1-3 seconds to create the bag on the caster idk whats the reaosn of it its not very serious thought

MyPad · Oct 18, 2020

This looks like a simple, yet interesting ability to have.

Spell:

The spell is as simple as can be. In what it does, this surely doesn't disappoint.
Very useful ability to have for bandits (or Orc raiders).

The details described below are applicable to previous versions of the spell, and might not apply to current versions.

Spell Mechanics:

As previously reported by emil23, the spell appears to suffer from a visual discontinuity regarding the bag. While being thrown, it is still present on the caster.
An edge-case Critical bug occurs when the caster throws the sacked unit at the instant before the next tick drops the sacked unit where the thrown bag is not removed, due to the reference to the sack instance being removed. A visual representation of this bug is shown below:

What happened above is a race condition where the sack instance was modified and destroyed at one moment, and an attempt to read the destroyed sack instance in the next moment.
In Unit - A unit Starts the effect of an ability events, you can replace instances of (Casting Unit) with (Triggering Unit). (Casting Unit) will always return (Triggering Unit) in ability-based events.

Listed below are suggestions for the spell:

Add a buff indicator (based on the Object Editor) to help the player know when he/she can throw the sacked unit.

Spell Rating:

Criteria:
Configurability:	7/15
Importability:	13/15
Maintainability:	25/30
Game Mechanics:	40/40
Total:	85/100

Gimli007 · Jan 8, 2021

please fix want to use it!!

emil23 · Feb 8, 2021

MyPad said:
This looks like a simple, yet interesting ability to have.

Spell:

The spell is as simple as can be. In what it does, this surely doesn't disappoint.
Very useful ability to have for bandits.

Spell Mechanics:

As previously reported by emil23, the spell appears to suffer from a visual discontinuity regarding the bag. While being thrown, it is still present on the caster.

An edge-case Critical bug occurs when the caster throws the sacked unit at the instant before the next tick drops the sacked unit where the thrown bag is not removed, due to the reference to the sack instance being removed. A visual representation of this bug is shown below:

View attachment 366412

What happened above is a race condition where the sack instance was modified and destroyed at one moment, and an attempt to read the destroyed sack instance in the next moment.

In Unit - A unit Starts the effect of an ability events, you can replace instances of (Casting Unit) with (Triggering Unit). (Casting Unit) will always return (Triggering Unit) in ability-based events.

Listed below are suggestions for the spell:

Add a buff indicator (based on the Object Editor) to help the player know when he/she can throw the sacked unit.

Spell Rating: N/A

For now, the spell rating is unavailable, due to the presence of a critical game bug.

Placing this on Awaiting Update for now.

Hi is it required to request a review? or mod will do it in free time

emil23 · Feb 8, 2021

@Tier10Trash hey could you tell me how are you checking if the area is blocked or not? i couldnot figure it out. i saw the gui blizzard function to check if terrain walkability is of or on but it only applies to terrain and i think it allows you to go throught pathning blockers or dooads. are u using something more to check if area is blocked?

MyPad · Feb 8, 2021

MyPad said:
An edge-case Critical bug occurs when the caster throws the sacked unit at the instant before the next tick drops the sacked unit where the thrown bag is not removed, due to the reference to the sack instance being removed.

Nicely done fixing this glaring bug. Race condition issues are quite difficult to track sometimes, and might be even harder to fix.

MyPad said:
As previously reported by emil23, the spell appears to suffer from a visual discontinuity regarding the bag. While being thrown, it is still present on the caster.

This point no longer stands as of the current version of this spell.

Before approving the resource, I would like to address a minor issue: The dummy unit model does not appear to be invisible, due to the presence of shadows and an arrow sound playing upon death.

I'll approve this in good faith that the issue above will be resolved.

EDIT:

I suppose I approved this too soon. I forgot to mention that the spell should have a separate config trigger, which this doesn't have.

Tier10Trash · Feb 11, 2021

emil23 said:
@Tier10Trash hey could you tell me how are you checking if the area is blocked or not? i couldnot figure it out. i saw the gui blizzard function to check if terrain walkability is of or on but it only applies to terrain and i think it allows you to go throught pathning blockers or dooads. are u using something more to check if area is blocked?

The walkability condition checks for doodads and pathing blockers as well, I added the trees into the test map to show this.

Grab, Throw and Drop MUI

Grab, Throw and Drop MUI (Map)

Attachments

Spell:

Spell Mechanics:

Spell Rating:

Spell:

Spell Mechanics:

Spell Rating: N/A

Similar threads