Casting-Channeling Bar v1.6.0

[Moderator]

Approved since it works nicely.

-Change the version number to 1.x.x, obviously it is not 0.0.x
-Change the Temp real conditions as suggested in my reply to this thread
-Rethink the way you show/hide the floating text for player, now you do too many actions
-Add "unit is in cast group" condition to stops casting trigger, since now it triggers for all abilities

 

[maddeem]

Triggers

• Int Casting Stuff
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Hashtable - Create a hashtable
    • Set BarHash = (Last created hashtable)
    • -------- The number of the little lllllllll on the casting bar. . . --------
    • Set NumberofTicks = 60
    • -------- Size of the Bar --------
    • Set CastingBarSize = 6.00
    • -------- The Z offset of the casting bar. --------
    • Set BarOffset = 50.00
    • -------- Color of the casting bar for each player. . . --------
    • Set PlayerColors[1] = |cffff0303
    • Set PlayerColors[2] = |cff0042ff
    • Set PlayerColors[3] = |cff1ce6b9
    • Set PlayerColors[4] = |cff540081
    • Set PlayerColors[5] = |cfffffc01
    • Set PlayerColors[6] = |cfffe8a0e
    • Set PlayerColors[7] = |cff20c000
    • Set PlayerColors[8] = |cffe55bb0
    • Set PlayerColors[9] = |cff959697
    • Set PlayerColors[10] = |cff7ebff1
    • Set PlayerColors[11] = |cff106246
    • Set PlayerColors[12] = |cff4e2a04
    • Set PlayerColors[13] = |c002F2F2F
    • Set PlayerColors[14] = |c002F2F2F
    • -------- Spell Configuration --------
    • -------- This is the ability for the spell you want to set the casting time of --------
    • Set TempAbil = Flame Strike
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 1, 3.0)
    • -------- ^ --------
    • -------- That is the casting Time --------
    • Set TempAbil = Banish
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 1, 2.0)
    • Set TempAbil = Phoenix
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 1, 5.0)
    • Set TempAbil = Siphon Mana
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 1, 1.5)
    • -------- ^ --------
    • -------- Some Abilities require less/more casting time per level so change this --------
    • -------- 1 being the level of the ability --------
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 2, 1.0)
    • Custom script: call SaveReal(udg_BarHash,udg_TempAbil, 3, 0.5)
    • For each (Integer TempInteger) from 1 to NumberofTicks, do (Actions)
      • Loop - Actions
        • Set LoadingBarText = (LoadingBarText + l)

• BeginCast
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
  • Actions
    • Set TempUnit = (Triggering unit)
    • Set TempInteger = (Custom value of TempUnit)
    • Set BeginCastTime[TempInteger] = 0.00
    • Custom script: set udg_TempReal = LoadReal(udg_BarHash, GetSpellAbilityId(), GetUnitAbilityLevelSwapped(GetSpellAbilityId(), udg_TempUnit))
    • Custom script: set udg_Temp_Real = LoadReal(udg_BarHash, GetSpellAbilityId(),1)
    • -------- Some Abilities wont have a changing casting time, this makes it so that you --------
    • -------- Dont have to copy and paste the casting time per each level --------
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • TempReal Greater than 0.00
            • Temp_Real Greater than 0.00
      • Then - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • TempReal Equal to 0.00
          • Then - Actions
            • Set EndCastTime[TempInteger] = Temp_Real
          • Else - Actions
            • Set EndCastTime[TempInteger] = TempReal
        • Set TempString = ([ + (LoadingBarText + ]))
        • Floating Text - Create floating text that reads TempString above TempUnit with Z offset BarOffset, using font size CastingBarSize, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
        • Set CastingBar[TempInteger] = (Last created floating text)
        • Unit Group - Add TempUnit to CastingGroup
        • Trigger - Turn on CastingPeiodic <gen>
        • Floating Text - Hide CastingBar[TempInteger] for (All players)
        • Set TempPGroup = (All players matching (((Matching player) controller) Equal to User))
        • Player Group - Pick every player in TempPGroup and do (Actions)
          • Loop - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (TempUnit is visible to (Picked player)) Equal to True
              • Then - Actions
                • Set TempPGroup2 = (Player group((Picked player)))
                • Floating Text - Show CastingBar[TempInteger] for TempPGroup2
                • Custom script: call DestroyForce(udg_TempPGroup2)
              • Else - Actions
        • Custom script: call DestroyForce(udg_TempPGroup)
      • Else - Actions

• EndCast
  • Events
    • Unit - A unit Stops casting an ability
  • Conditions
  • Actions
    • Set TempUnit = (Triggering unit)
    • Set TempInteger = (Custom value of TempUnit)
    • Unit Group - Remove TempUnit from CastingGroup
    • Floating Text - Destroy CastingBar[TempInteger]
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Number of units in CastingGroup) Equal to 0
      • Then - Actions
        • Trigger - Turn off CastingPeiodic <gen>
      • Else - Actions

• CastingPeiodic
  • Events
    • Time - Every 0.05 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in CastingGroup and do (Actions)
      • Loop - Actions
        • Set TempUnit = (Picked unit)
        • Set TempInteger = (Custom value of TempUnit)
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • BeginCastTime[TempInteger] Greater than or equal to EndCastTime[TempInteger]
          • Then - Actions
            • Floating Text - Destroy CastingBar[TempInteger]
            • Unit Group - Remove TempUnit from CastingGroup
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Number of units in CastingGroup) Equal to 0
              • Then - Actions
                • Trigger - Turn off (This trigger)
              • Else - Actions
          • Else - Actions
            • Set BeginCastTime[TempInteger] = (BeginCastTime[TempInteger] + 0.05)
            • Set TempReal = ((BeginCastTime[TempInteger] / EndCastTime[TempInteger]) x (Real(NumberofTicks)))
            • Set TempString = ([ + (PlayerColors[(Player number of (Owner of TempUnit))] + ((Substring(LoadingBarText, 1, (Integer(TempReal)))) + (|r + ((Substring(LoadingBarText, (Integer(TempReal)), NumberofTicks)) + ])))))
            • Floating Text - Change text of CastingBar[TempInteger] to TempString using font size CastingBarSize
            • Floating Text - Hide CastingBar[TempInteger] for (All players)
            • Set TempPGroup = (All players matching (((Matching player) controller) Equal to User))
            • Player Group - Pick every player in TempPGroup and do (Actions)
              • Loop - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (TempUnit is visible to (Picked player)) Equal to True
                  • Then - Actions
                    • Set TempPGroup2 = (Player group((Picked player)))
                    • Floating Text - Show CastingBar[TempInteger] for TempPGroup2
                    • Custom script: call DestroyForce(udg_TempPGroup2)
                  • Else - Actions
            • Custom script: call DestroyForce(udg_TempPGroup)

 

[yurneric]

Suggestion for you:

- The use dynamic indexing is better (and cleaner).
- Unit group should be destroyed when number of units equals to 0. (It doesn't hurt to recreate it.)
- Note that you used "Begin CHANNELING". I would say, use "Begin CASTING". (I hope you know the difference.)


Comments:

- Fairly good idea.
- Troublesome casting time set-up for different spells.
- Just don't get confused with "casting time" and "channeling time".


Example:
- Click Storm Bolt icon, click on a target. (Issued order)
- Hero faces the target and getting ready to throw after 2 seconds of casting time. (Casting starts)
- 2 seconds passed, Storm Bolt thrown, cooldown starts. (Effect of ability starts)
- OR alternatively, Starfall starts. (Channeling starts)

 

[RakEnRoL]

Wow. Great Idea I was like thinking of this system but unfortunately, I don't know how to code or make these kind of stuff. You made my dreams come true :3

 

[Palooo3]

Nice what i see from triggers, will check at home and probablly using

 

[RakEnRoL]

I think it bugs. When I cast a spell, specifically Phoenix. I tried to hold the hotkey, it casts however the floating text is not visible.

 

[Maker]

Player Group(picked player) leaks.
There is IsLocationVisibleToPlayer function, but I don't think it is in GUI. You could use that to decide whether to show the text or not.
Maybe create two strings at init, different colours for empty and full. Replace the loops that create the bars with using substring.
I'd use hashtable to load the EndCastTime.

Init values like this:

• Custom script: call SaveInteger(udg_hash, 'AHfs', 0, 2)

AHfs is the raw code of Flame Strike, 2 is the duration. Use 0 as the child key for every ability.

Load values like this:

• Untitled Trigger 009
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
  • Actions
    • Custom script: if HaveSavedInteger(udg_hash, GetSpellAbilityId(), 0) then
    • Set TempUnit = (Triggering unit)
    • Set TempInteger1 = (Custom value of TempUnit)
    • Custom script: Set udg_EndCastTime[udg_TempInteger1] = LoadInteger(udg_hash, GetSpellAbilityId(), 0)
    • -------- *rest of the actions* --------
    • Custom script: endif

This way you can get rid og the potentially very long if/then/else.

- The use dynamic indexing is better (and cleaner).
- Unit group should be destroyed when number of units equals to 0. (It doesn't hurt to recreate it.)
- Note that you used "Begin CHANNELING". I would say, use "Begin CASTING". (I hope you know the difference.)

-Bribe's unit indexer is perfect for this
-The group shouldn't be destroyed
-Begins Channeling is the correct event

This system shoudn't be used for abilities that have no real channeling time, like Storm Bolt.

 

[RakEnRoL]

And for the effects. I think it's rather ugly seeing a large bar while playing. Imagine 12 players with their own units spamming spells with it. That is uhm. Hell'a ugly.

 

[Bribe]

Well the size of the text tag should be configurable, and I kind of agree
it is a bit large but I think that's fine for a demo map so it can be shown
in more detail.

Things like the colors of the text tag, as well, should be configurable.

 

[maddeem]

Thanks for the comments guys, I'll work on this when I have the time. But I have some quick questions.

IsLocationVisibleToPlayer, could someone post a demo or something of this? I know what the Boolean is I just don't know how to do it.

And as for saving abilities into hashtables, I had no idea you could do that =D
Totally great to know, and I'll definitely get rid of all those if/then/else

 

[Vlad727]

I really like this for the following reasons:
1. I always wanted to add something like this to my maps.
2. Useful in boss-fights, because the players can now see if a boss is starting to cast a spell and see how much time there is left to get the heck out of there.
3. there was a VJASS version, but I don't use VJASS.
4. It is also useful for players, and it looks good (if made smaller)
5. It is just AWESOME!!!

You will get +REP From me for sure,
and thanks for creating this.

This system shoudn't be used for abilities that have no real channeling time, like Storm Bolt.

So instant spells won't get a casting bar?
(which I understand since there instant)
But what if the ability does not have to be channelled to remain in play?
(when the caster is interupted the spell stops, like death and decay.
More like "fire bolt" were you aim, cast and fire!)

So is using "begins casting an ability" also right?
or am I wrong?

Just curious...

 

[Palooo3]

WOW I really like it!
Also supports interrupting cast with stun!
Anyone can tell me if there are leaks wich cause lagg? If not I am importing it
And if i have 50 spells do I have to make them all in one trigger (like you have banish, fire strike etc), or i can make them in separated triggers?

 

[maddeem]

I'm going to update it in a sec guys =p

 

[Palooo3]

Great waiting for it, maybe we are going to see more maps with nice systems as this one is

 

[maddeem]

Upgrade complete =p

 

[Bribe]

Looking really good so far.

I recommend letting there be a GUI variable in the configuration, which is set before each custom script hashtable save, that way you can select the variable ID from GUI if you don't want/don't know how to look up the rawcode every time.

• Set TempAbil = Blizzard //TempAbil is of type "ability"
• Custom script: call SaveReal(udg_BarHash, udg_TempAbil, 0, 3.0)

Also, I recommend taking advantage of the first key to represent each individual level in case there are different channeling times. Like this:

• Set TempAbil = Blizzard
• Custom script: call SaveReal(udg_BarHash, udg_TempAbil, 1, 3.0)
• Custom script: call SaveReal(udg_BarHash, udg_TempAbil, 2, 4.0)
• Custom script: call SaveReal(udg_BarHash, udg_TempAbil, 3, 5.0)

 

[maddeem]

Great idea, I plan on adding it as soon as I get the time =p

 

[Magtheridon96]

This looks pretty good

 

[maddeem]

v0.0.2 is out guys!

 

[maddeem]

v0.0.3 is out! It is now only visible to players who can see the casting unit, also added offset.

This will probably be the final version.

 

[Vlad727]

epic and realistic!

I like your style!
since I use floods camera system, this is a great improvement

My rating 5/5
reasons:
-It is GUI
-Really useful
-I always wanted to have a system like this for my maps

 

[Maker]

GetUnitAbilityLevelSwapped(abilID, unit) -> GetUnitAbilityLevel(unit, abilID)

The if/then/else:

Trig

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • Temp_Real Greater than 0.00
  • Then - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • TempReal Equal to 0.00
      • Then - Actions
        • Set EndCastTime[TempInteger] = Temp_Real
      • Else - Actions
        • Set EndCastTime[TempInteger] = TempReal

Trig

• Floating Text - Hide CastingBar[TempInteger] for (All players)
• Set TempPGroup = (All players matching (((Matching player) controller) Equal to User))
• Player Group - Pick every player in TempPGroup and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (TempUnit is visible to (Picked player)) Equal to True
      • Then - Actions
        • Set TempPGroup2 = (Player group((Picked player)))
        • Floating Text - Show CastingBar[TempInteger] for TempPGroup2
        • Custom script: call DestroyForce(udg_TempPGroup2)
      • Else - Actions
• Custom script: call DestroyForce(udg_TempPGroup)
• Else - Actions

There you hide it for all players, even though you create a group containing all human players.
You might also end up hiding it for a player and the showing it right away. Same thing in the loop.

• Group = human players
• Pick players
  • If unit is not visible for player
  • Then hide text
  • Endif

In the loop, show if visible, hide if not visible.

 

[maddeem]

Glad you like it Vlad =D

Maker I'm not exactly sure about what you are saying, could you try to clarify it a bit?

 

[baassee]

Cool stuff although I would like to have the option to only display the casting bar to myself / or my allies only as well. But that would be a turn around now would it?

 

[maddeem]

Hmm that would kinda ruin the whole point of this xD

 

[.OmG.]

Please remove so much units, just create few. I dont understand who is casting/

 

[RakEnRoL]

Is the casting bar configurable? you know, if you can change its size,colors or whatever.

 

[Adiktuz]

I'll look at it later... gotta compare it with the casting systems here...

 

[Magtheridon96]

Maker helped me notice this:

• Floating Text - Hide CastingBar[TempInteger] for (All players)
• Set TempPGroup = (All players matching (((Matching player) controller) Equal to User))
• Player Group - Pick every player in TempPGroup and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (TempUnit is visible to (Picked player)) Equal to True
      • Then - Actions
        • Set TempPGroup2 = (Player group((Picked player)))
        • Floating Text - Show CastingBar[TempInteger] for TempPGroup2
        • Custom script: call DestroyForce(udg_TempPGroup2)
      • Else - Actions
• Custom script: call DestroyForce(udg_TempPGroup)

>_> Fail

Do this:

• Custom script: call SetTextTagVisibility(udg_CastingBar[udg_TempInteger], IsUnitVisible(udg_TempUnit, GetLocalPlayer()))

Now tell me, wasn't that code at first an uber fail compared to this?

edit
Another thing:

• Custom script: set udg_TempReal = LoadReal(udg_BarHash, GetSpellAbilityId(), GetUnitAbilityLevelSwapped(GetSpellAbilityId(), udg_TempUnit))
• Custom script: set udg_Temp_Real = LoadReal(udg_BarHash, GetSpellAbilityId(),1)

->

• // top of the trigger
• Custom script: local integer id = GetSpellAbilityId()
• // trigger code
• Custom script: set udg_TempReal = LoadReal(udg_BarHash, id, GetUnitAbilityLevel(udg_TempUnit, id))
• Custom script: set udg_Temp_Real = LoadReal(udg_BarHash, id,1)

 

[Cihparg]

HINT: Use ' with size 10.

 

[maddeem]

I'll be updating this soon =p

 

[gorillabull]

very easy to implement

 

[maddeem]

Updated casting bar! It is now much more efficient!

 

[Matarael]

As soon as I read the title, I literally said out loud "that's a brilliant idea".

 

[Le Razer]

Sooo...

How do I Put this on my Map? IT IS VERY EPIC

 

[maddeem]

You simply copy all the triggers and follow the instructions (which aren't too detailed I know... If you have any questions on how to add casting bar to spells just vm me)

 

[Matarael]

Wait, I've been wondering, since I haven't implemented this and am just looking through the triggers: is this for casting time only? Or does it support CHANNELING time (which would be awesomer)?

 

[maddeem]

Casting time only... though I could create a channel addon.... Hmmp.

 

[Matarael]

Casting time only... though I could create a channel addon.... Hmmp.

Please do this... for the betterment of mankind...!

 

[Palooo3]

Hello guys, I got some baddy problem
I can easilly modify original map wich I download, like casting time, number of ticks and siúch things, but once I import triggers into my map i got this big bad error. Anyone got advice for me what should I do?

This system is too great to not be in my project

set udg_TempInteger2 = GetSpellAbilityId()

This line is bad

 

[Maker]

Create a variable called TempInteger2 in variable editor.

 

[Palooo3]

Thank you for advice, is it Array or normal?
Edit:

Got it! works lovely, thank you again Maker

 

[Orcnet]

Highly recommended!

 

[maddeem]

Please do this... for the betterment of mankind...!

It shall be done. Though, after tea time

 

[maddeem]

Ok so I made the channeling addon but my computer I saved it to is currently under repair :s
I hope to post new ver by the end of the day.

BOOYAH! It's finished, I believe this will be the last update for this system.

Oh, could I get a new review for this? The old review is quite outdated.

 

[Matarael]

Yesssssssssssssssssssss...!!!

Edit: You must spread some Reputation around before giving it to maddeem again.
Awww...

 

[maddeem]

Glad you like it

 

[Vengeancekael]

Oh my god this is really neat, I like it!

 

[maddeem]

Did some minor updates to the casting bar.