Floating test problem!

[CoLd Bon3]

Hi! I want to create a floating text that show after X 0.10 seconds the spell will cast. I have created the following :

• Time till cast
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Casting unit) Equal to Lich 0000 <gen>
  • Actions
    • Floating Text - Create floating text that reads 3.0 above Lich 0000 <gen> with Z offset 0.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
    • Wait 0.10 game-time seconds
    • Floating Text - Destroy (Last created floating text)
    • Floating Text - Create floating text that reads 2.9 above Lich 0000 <gen> with Z offset 0.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
    • Wait 0.10 game-time seconds
    • Floating Text - Destroy (Last created floating text)
    • Floating Text - Create floating text that reads 2.8 above Lich 0000 <gen> with Z offset 0.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
    • Floating Text - Destroy (Last created floating text)

And that repeating 30 times at all till reach zero. The problem is that when the floating text goes somewhere around 1.7 sec my hero has already casted the spell. (i've maked sure that the castung time of my spell is 3 sec) Why?!

 

[defskull]

Please avoid the usage of Waits
It can cause your trigger to malfunction
Please use Integer/Real type variable for the use of per second/counter-based trigger
I'll create a test map for you, wait okay =D

 

[priest170234]

Okay. Few comments. Like what defskull said, don't use waits.
Next, having a Destroy(Last Created Floating Text) after 0.1 second could interfere.
Because if there was another floating text created, this would be destroyed instead of the one you wanted to remove. I suggest using a variable.

By the way, use Unit - A unit Finishes casting an ability
instead. That should solve the problem.

 

[CoLd Bon3]

If i use A unit Finishes casting an ability there will be no need of that text. I want the time TILL my hero casts the spell. Got it?

 

[defskull]

Test map attached

 

[CoLd Bon3]

It is perfect, but can it be created like this - "Frostbolt - 3.0" after 0.10 sec - "Frostbolt - 2.9" and can u make it without that blinking? (it 'blinks' every 0.10 sec)

 

[defskull]

Change the Lifespan to 0.05 seconds
For Frostbolt, Use Concatenate string
String 1 is Frost bolt (text) (space here)
String 2 is Timer (Real variable) (Convert Real to String action)

I can't trigger it right now, in DotA game >.<

 

[priest170234]

Here, try this. Does not seem to blink much.

http://www.hiveworkshop.com/forums/pastebin_data/wqb0vl/_files/Timer.w3x

 

[CoLd Bon3]

It works PERFECTLY! But there is one last thing - how i can destroy that text if my unit moves somewhere (i don't want the text if the hero isn't casting )

 

[priest170234]

Kay, let me work on it for awhile kay?

Hey wait, if you move your hero during the period of casting time, the spell would be cancelled.

EDIT: Sorry, did'nt read your post correctly. I understand now. Give me few minutes.

 

[CoLd Bon3]

Yes! And if it is possible can it be maded when the timer reaches 0.0 to blink in green and when is interrupted to blink in red and stop the countdown.

 

[Maker]

EDIT: New version at post #33.

 

[priest170234]

Alright, this is it. Made with the green/red and stop floating text when stop casting.

http://www.hiveworkshop.com/forums/pastebin_data/wqb0vl/_files/Timer.w3x

Take note: This is not MUI.

 

[CoLd Bon3]

Thanks! I give you all +rep! (defskull, i need to spread some rep to give you )

 

[CoLd Bon3]

I hate to put in my maps triggers that i don't understand, so i've been watching all that triggers and after 10 min i have tried to make them and i actually made it :

• first trig
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
  • Actions
    • Set Caster = (Triggering unit)
    • Set Timer = 3.00

• second trig
  • Events
    • Time - Every 0.10 seconds of game time
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Timer Greater than 0.00
      • Then - Actions
        • Floating Text - Create floating text that reads (Flame Strike in + ((String(Timer)) + sec.)) above Caster with Z offset 30.00, using font size 8.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
        • Floating Text - Change (Last created floating text): Disable permanence
        • Floating Text - Change the lifespan of (Last created floating text) to 0.15 seconds
        • Floating Text - Change the fading age of (Last created floating text) to 0.10 seconds
      • Else - Actions
        • Floating Text - Destroy (Last created floating text)
    • Set Timer = (Timer - 0.10)

• third trig
  • Events
    • Unit - A unit Stops casting an ability
  • Conditions
    • Timer Greater than 0.01
  • Actions
    • Game - Display to (All players) the text: Player 1's Flame St...
    • Set Timer = 0.00

but i have a tiny little problem - the floating text is a little bit in the right and i want it right above the caster. Please help And i want to ask a thing - what means

• Floating Text - Change (Last created floating text): Disable permanence

???

 

[Maker]

Floating texts always start from the center of the unit. I don't think you chan offset them.

Disable permanence means it's not permanent, it has timed duration.

 

[priest170234]

The floating text is already on top of the caster. Because it is directly on top, and that whatever text is from left to right reading order. So when the text is created, it continues on in the right direction.

E.g. The below: [The 3.00 is the text, the . is the point of caster]
----------------------
3.00
.
----------------------
See now?

• Floating Text - Change (Last created floating text): Disable permanence

Means that the floating text is not permanent; allowing you to do changes to it.
A permanent floating text cannot be removed.

 

[CoLd Bon3]

The floating text is already on top of the caster. Because it is directly on top, and that whatever text is from left to right reading order. So when the text is created, it continues on in the right direction.

E.g. The below: [The 3.00 is the text, the . is the point of caster]
----------------------
3.00
.
----------------------
See now?

• Floating Text - Change (Last created floating text): Disable permanence

Means that the floating text is not permanent; allowing you to do changes to it.
A permanent floating text cannot be removed.

Ok. But how i can make it be from the left of the caster, so when it is writed from the left to right it will just gonna be what i want?

 

[defskull]

Create dummy unit offset by Position of unit + 90 degree... ?
And create the floating text ABOVE the dummy unit.. ?

 

[CoLd Bon3]

Hmm... how? Can you show me?

 

[priest170234]

Instead of creating the floating text above the unit, I create the floating text above the unit, with the offset towards left, so when the left to right reading order applies, the floating text looks centralized.

Here's a test map.

Btw, if you need it in MUI, I can do it, if you want.

 

[CoLd Bon3]

Ah... i've finally created that what i want. Thank you guys, thank you so much! You can see what i maded below

 

[priest170234]

Checked. Yes, it will work only if the spell cooldown is longer than 3 seconds. And you're the only one who uses it in your map.
Meaning: There's only one unit in the whole map using this skill anytime.

If not, it have to be MUI. Or else, it will bug...

 

[CoLd Bon3]

What means MUI?

 

[Maker]

What means MUI?

Multi-instanceable (or multi-unit instanceable).

If something is MUI, it works for many units at the same time. If it is not MUI, it only works correctly if there's only one unit using the system/spell.

 

[priest170234]

Multi-instanceable (or multi-unit instanceable).

If something is MUI, it works for many units at the same time. If it is not MUI, it only works correctly if there's only one unit using the system/spell.

Like what he said. MUI means that the spell will bug if there are more than one unit at a time using it.

 

[CoLd Bon3]

Yeah, i understand! I have that problem in my thread '2 very big problems!' (i want it MUI, but it isn't) check it, please

 

[priest170234]

Kay, I would make it MUI for you. Hold on..

 

[priest170234]

Sorry for the really delayed map, was busy. Anyway, here's it!

 

[Maker]

Sorry for the really delayed map, was busy. Anyway, here's it!

Code review here, read if you're interested

Review

There's no need for this. You never have to store/load the unit like this. You need the unit's handle to load something from a hashtable, and if you have the handle, you already have another reference to the unit.

• Hashtable - Save Handle Of(Triggering unit) as 2 of (Key (Triggering unit)) in Timer_Hash

You don't need to load the value, just set it to a real value:

• Floating Text - Create floating text that reads (String((Load 1 of (Key (Triggering unit)) from Timer_Hash))) at TempPoint with Z offset 150.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency

Update trigger is always on. turn it off when the group is empty, and on when the unit count goes from 0 to 1. Same with stop trigger, it shouldn't be always on.

You're doing both these checks during every loop, which is not needed.

• (Load 1 of (Key (Picked unit)) from Timer_Hash) Equal to 0.00
• (Load 1 of (Key (Picked unit)) from Timer_Hash) Less than or equal to 0.10

You don't flush child hashtables.

You are leaking. Position of a unit is one point, the offset point is another. You are not removeing them.

• Set TempPoint = ((Position of (Triggering unit)) offset by 110.00 towards 180.00 degrees)

 

[priest170234]

Code review here, read if you're interested

Review

There's no need for this. You never have to store/load the unit like this. You need the unit's handle to load something from a hashtable, and if you have the handle, you already have another reference to the unit.

• Hashtable - Save Handle Of(Triggering unit) as 2 of (Key (Triggering unit)) in Timer_Hash

You don't need to load the value, just set it to a real value:

• Floating Text - Create floating text that reads (String((Load 1 of (Key (Triggering unit)) from Timer_Hash))) at TempPoint with Z offset 150.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency

Update trigger is always on. turn it off when the group is empty, and on when the unit count goes from 0 to 1. Same with stop trigger, it shouldn't be always on.

You're doing both these checks during every loop, which is not needed.

• (Load 1 of (Key (Picked unit)) from Timer_Hash) Equal to 0.00
• (Load 1 of (Key (Picked unit)) from Timer_Hash) Less than or equal to 0.10

You don't flush child hashtables.

You are leaking. Position of a unit is one point, the offset point is another. You are not removeing them.

• Set TempPoint = ((Position of (Triggering unit)) offset by 110.00 towards 180.00 degrees)

Haha, thanks for reviewing! Anyways, this is the first few times I am sing hashtables.. So.. Maybe about what flushing thing I don't get it

Can you edit that map and show me the better way of doing it?

 

[CoLd Bon3]

Code review here, read if you're interested

Review

There's no need for this. You never have to store/load the unit like this. You need the unit's handle to load something from a hashtable, and if you have the handle, you already have another reference to the unit.

• Hashtable - Save Handle Of(Triggering unit) as 2 of (Key (Triggering unit)) in Timer_Hash

You don't need to load the value, just set it to a real value:

• Floating Text - Create floating text that reads (String((Load 1 of (Key (Triggering unit)) from Timer_Hash))) at TempPoint with Z offset 150.00, using font size 10.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency

Update trigger is always on. turn it off when the group is empty, and on when the unit count goes from 0 to 1. Same with stop trigger, it shouldn't be always on.

You're doing both these checks during every loop, which is not needed.

• (Load 1 of (Key (Picked unit)) from Timer_Hash) Equal to 0.00
• (Load 1 of (Key (Picked unit)) from Timer_Hash) Less than or equal to 0.10

You don't flush child hashtables.

You are leaking. Position of a unit is one point, the offset point is another. You are not removeing them.

• Set TempPoint = ((Position of (Triggering unit)) offset by 110.00 towards 180.00 degrees)

I've finded hashtable tutorial and i read it. Now i will practice! Thanks!!!

 

[Maker]

Flushing child hashtable clears all stored data for the given key. It frees memory.

I posted a MUI floating text version in this thread yeasterday. Here it is again.

 

[priest170234]

Hey wait, you mean the real value of the offset also leaks?