Download
(6 Kb)
Download
Updated: 09/12/23 01:05 AM
Pictures
File Info
Compatibility:
base-game patch (9.1.5)
Updated:09/12/23 01:05 AM
Created:02/27/18 04:49 PM
Monthly downloads:1,975
Total downloads:290,740
Favorites:152
MD5:
Categories:Buff, Debuff, Spell, DPS, Combat Mods, Graphic UI Mods, Info, Plug-in Bars, PvP
Greyskull : Weapon / Spell Damage Meter  Popular! (More than 5000 hits)
Version: 1.1.1
by: ghostbane [More]
NEW: - Hybrid mode
Select Hybrid in the settings menu and Greyskull will appear with whatever is higher, Weapon Damage or Spell Damage.

Greyskull - Customisable power meter

Set coloured rules for specific power levels for visual ques of when one or several damage procs are up to time your burst.



Customisable Options:
  • Hybrid mode (new)
  • UI sizing
  • Render limit
  • Colour options

Dependencies: LibAddonMenu
1.1.1
- Temp removing account-wide settings option due to issue

1.1.0
- Added option for Account-wide settings
- API bump for U39

1.0.1
- Upgrading API version for Necrom

1.0.0
- Fixed migration script
- Added new Hybrid feature
- Small optimizations

0.4.5
- Updating API version to Scribes of Fate
- Nil issue fixed from no number entry in the settings
- Re-factored some code to modern ESO addon standards ( thanks Baertram! )

0.4.3
Updating API version to Flames of Ambition

Fixed bug #3322 on the user defined background colour not loading correctly on start up, special thank you to Skinfaxe_DK for reporting it.

0.4.2
Fixing Scene error that was pushed up.

0.4.0
Special thank you to Sharlikran for informing me that LibStub is in the retirement home!

Fixed long standing issue of the initial render having a faux red border and white text. We all know how much a weapon swap taxes the CPU, need it not be in vain no longer

0.3.9
Updating API version so OCD pansies don't have to click the scary checkbox.

0.3.8
Using global libs instead of stone-age on-board references. Pip pip.

0.3.6
Fixed a LUA error issue prompted by not reading what you write, gj

0.3
Fixed an issue on where Spell Damage rules where not applying
Optional Files (0)


Archived Files (3)
File Name
Version
Size
Uploader
Date
1.1.0
6kB
ghostbane
09/06/23 04:27 PM
1.0.1
6kB
ghostbane
06/05/23 02:45 AM
1.0.0
6kB
ghostbane
05/24/23 09:19 AM


Post A Reply Comment Options
Unread 09/14/23, 07:59 PM  
jinncrazey

Forum posts: 1
File comments: 82
Uploads: 0
thanks for the response hope u fix it! For now I got bored and did the settings in-game on 1 character and logged out and copy pasted it over on the other chars on savedVars file in notepad.
edit: woops nvm.. after I did this. It reset the whole savedvars T_T all that data on 30 chars.
Last edited by jinncrazey : 09/14/23 at 08:38 PM.
Report comment to moderator  
Reply With Quote
Unread 09/12/23, 02:37 AM  
ghostbane
AddOn Author - Click to view AddOns

Forum posts: 4
File comments: 58
Uploads: 6
Originally Posted by Saenic
Imho you could rewrite the addon (and therefore discarding the previous user settings) with accountwide settings and hybrid mode as default. By doing this, you could also merge some settings for thresholds.
I guess this would be suitable for most users.
Yes, that may have to be the approach. Years ago when this was first made, it was built with the preference of having threshold settings per character, as builds/characters differed.

I think it would be wise now, instead of per character, allow the use of having threshold settings defined as a 'build/profile name' that would be universially available to all chracters. The other settings would remain as account-wide.

This would involve, as you say, a data reset, something I've always actively avoided. But for the sake of a better expierence going forward, it may have to happen.
Report comment to moderator  
Reply With Quote
Unread 09/12/23, 02:19 AM  
Saenic
 
Saenic's Avatar
AddOn Author - Click to view AddOns

Forum posts: 7
File comments: 125
Uploads: 2
Imho you could rewrite the addon (and therefore discarding the previous user settings) with accountwide settings and hybrid mode as default. By doing this, you could also merge some settings for thresholds.
I guess this would be suitable for most users.
Report comment to moderator  
Reply With Quote
Unread 09/12/23, 01:06 AM  
ghostbane
AddOn Author - Click to view AddOns

Forum posts: 4
File comments: 58
Uploads: 6
Originally Posted by jinncrazey
Trying to fix the settings to have account-wide on all my characters by manually editing the savedvars(reloadui always disconnects me from game so I cant set it in-game). Is changed ["global"] settings to true but then when I login the addon disappeared from the list of addon settings in-game though the addon works and shows up in my screen as 0 and red font. Resetting it to false brings back the char-settings and the settings reappear in the list. Pls help ty.
Yes, there is an issue that can affect some users settings. I've released a new update that disables the account-wide feature so this can be fixed. You should be able to get the addon running as before with the new version.
Report comment to moderator  
Reply With Quote
Unread 09/11/23, 09:34 PM  
jinncrazey

Forum posts: 1
File comments: 82
Uploads: 0
Trying to fix the settings to have account-wide on all my characters by manually editing the savedvars(reloadui always disconnects me from game so I cant set it in-game). Is changed ["global"] settings to true but then when I login the addon disappeared from the list of addon settings in-game though the addon works and shows up in my screen as 0 and red font. Resetting it to false brings back the char-settings and the settings reappear in the list. Pls help ty.
Report comment to moderator  
Reply With Quote
Unread 08/29/23, 01:03 AM  
Myrthian

Forum posts: 28
File comments: 164
Uploads: 0
I'd like to also have the option to display:
- Crit chance
- Crit damage
in the widget.
Report comment to moderator  
Reply With Quote
Unread 08/25/23, 03:13 AM  
ghostbane
AddOn Author - Click to view AddOns

Forum posts: 4
File comments: 58
Uploads: 6
Originally Posted by Cheshire
Hi, would really like an option in this add-on that would make its settings account wide
Hey there, I will look into this at its next version!
Report comment to moderator  
Reply With Quote
Unread 08/24/23, 03:42 AM  
Cheshire

Forum posts: 1
File comments: 16
Uploads: 0
Hi, would really like an option in this add-on that would make its settings account wide
Report comment to moderator  
Reply With Quote
Unread 10/01/22, 11:04 PM  
shhfiftyfive

Forum posts: 0
File comments: 1
Uploads: 0
4.4?

i don't see a version 4.4 hosted with below mentioned updates.
Report comment to moderator  
Reply With Quote
Unread 06/11/22, 11:29 AM  
silver1surfer69

Forum posts: 7
File comments: 38
Uploads: 0
Sadly its not working with U34. Its not showing itself on my screen. TY!
Report comment to moderator  
Reply With Quote
Unread 08/25/21, 08:10 AM  
ghostbane
AddOn Author - Click to view AddOns

Forum posts: 4
File comments: 58
Uploads: 6
For some reason, tweaks added to 0.4.3 was from a much older version. ( Note 0.4.2 in *other files* )

I'll put yours up as 0.4.4 anyway when I get a moment, thank you for the hard work as always.

Originally Posted by Baertram
Please remove the libraries LibStub and LiBAddonMenu from your libs subfolder!
They should not be included anymore to addons, especially not LibStub which is obsolete since Summerset!

Your addon is leaking the variable hasAPM to global!
Please add local up in front or add it to your already global variable Greyskull
Btw, GREYSKULL_HEIGHT and width should be moved to Greyskull table as well. It's easier to see they belong to each other than defining global variables more and more, if you already got 1 global table where all your addon data could be stored.

+ Please use either GetWorldName() in the SavedVariables (e.g. as 3rd parameter "namespace" or as last "profile", or use ZO_SavedVars:NewCharacterIdSettings so it uses the unique (per server at least) character IDs instead of the names, so we are able to differ the
settings properly per server.
Beware that changing the namespace or profile parameter will have the users loose their savedvariables if you do not migrate them to the new subtables!
Changing the ZO_SavedVars:New to NewCharacterIdSettings could automatically migrate to the charIds though (if :New will use charcter names).
Names are not rename safe! So ALWAYS use the characterIds now (since Summerset already).

And please unregister the EVENT_ADD_ON_LOADED once your addon was found as else it will rerun your code checks for every addon activated!
Add at if addOnName == Greyskull.name then
Code:
EVENT_MANAGER:UnegisterForEvent(Greyskull.name, EVENT_ADD_ON_LOADED)

Thank you.

Edit:
I'm working on an updated version with these changes, and will post my file here as a link. Also trying to fix the error message that was posted below.
Greyskull v0.4.4 beta by Baertram

Changelog:
-Updated API version for current live and next patch
-Removed embedded libraries LibSub (obsolote! Do not install anymore!) and LibAddonMenu-2.0 (please download and install the newest version manually from www.esoui.com)
-Fixed error message nil > number
-Added right click on UI to switch between weapon damage or spell damage view
-Recoded some passages to be more performant
-Removed self where not needed, and directly used Greyskull instead
-Fixed global leaking variables hasAPM
-Fixed non-used variables GREYSKULL_WIDTH and GREYSKULL_HEIGHT -> Moved to global Greyskull.width and Greyskull.height
-Fixed SavedVariables to use characterIds instead of names + the servername. Your SV will reset so you have to reconfigure them. But the old data is still kept for the future so you could still create a kind of migration rouine to copy old data to new server dependent ones
-Fixed optional dependency to addon APMeter
-Fixed EVENT_ADD_ON_LOADED to unregister (as the optional dependency to APMeter handles the call to APMeter addon before Greyskull)
Report comment to moderator  
Reply With Quote
Unread 08/11/21, 06:04 AM  
Baertram
Super Moderator
 
Baertram's Avatar
ESOUI Super Moderator
AddOn Author - Click to view AddOns

Forum posts: 4903
File comments: 5975
Uploads: 78
Please remove the libraries LibStub and LiBAddonMenu from your libs subfolder!
They should not be included anymore to addons, especially not LibStub which is obsolete since Summerset!

Your addon is leaking the variable hasAPM to global!
Please add local up in front or add it to your already global variable Greyskull
Btw, GREYSKULL_HEIGHT and width should be moved to Greyskull table as well. It's easier to see they belong to each other than defining global variables more and more, if you already got 1 global table where all your addon data could be stored.

+ Please use either GetWorldName() in the SavedVariables (e.g. as 3rd parameter "namespace" or as last "profile", or use ZO_SavedVars:NewCharacterIdSettings so it uses the unique (per server at least) character IDs instead of the names, so we are able to differ the
settings properly per server.
Beware that changing the namespace or profile parameter will have the users loose their savedvariables if you do not migrate them to the new subtables!
Changing the ZO_SavedVars:New to NewCharacterIdSettings could automatically migrate to the charIds though (if :New will use charcter names).
Names are not rename safe! So ALWAYS use the characterIds now (since Summerset already).

And please unregister the EVENT_ADD_ON_LOADED once your addon was found as else it will rerun your code checks for every addon activated!
Add at if addOnName == Greyskull.name then
Code:
EVENT_MANAGER:UnegisterForEvent(Greyskull.name, EVENT_ADD_ON_LOADED)

Thank you.

Edit:
I'm working on an updated version with these changes, and will post my file here as a link. Also trying to fix the error message that was posted below.
Greyskull v0.4.4 beta by Baertram

Changelog:
-Updated API version for current live and next patch
-Removed embedded libraries LibSub (obsolote! Do not install anymore!) and LibAddonMenu-2.0 (please download and install the newest version manually from www.esoui.com)
-Fixed error message nil > number
-Added right click on UI to switch between weapon damage or spell damage view
-Recoded some passages to be more performant
-Removed self where not needed, and directly used Greyskull instead
-Fixed global leaking variables hasAPM
-Fixed non-used variables GREYSKULL_WIDTH and GREYSKULL_HEIGHT -> Moved to global Greyskull.width and Greyskull.height
-Fixed SavedVariables to use characterIds instead of names + the servername. Your SV will reset so you have to reconfigure them. But the old data is still kept for the future so you could still create a kind of migration rouine to copy old data to new server dependent ones
-Fixed optional dependency to addon APMeter
-Fixed EVENT_ADD_ON_LOADED to unregister (as the optional dependency to APMeter handles the call to APMeter addon before Greyskull)
Last edited by Baertram : 08/11/21 at 03:54 PM.
Report comment to moderator  
Reply With Quote
Unread 08/07/21, 03:31 AM  
Camlax

Forum posts: 0
File comments: 1
Uploads: 0
ERROR! Please help.

Addon wont work without constant error message. SOS.

user:/AddOns/Greyskull/greyskull.lua:168: operator < is not supported for nil < number
stack traceback:
user:/AddOns/Greyskull/greyskull.lua:168: in function 'Greyskull.Render'
|caaaaaa<Locals> inital = T, self = [table:1]{isLoaded = T, name = "greyskull"}, color = [table:2]{1 = 1}, powerLevel = 3929, powerGroup = [table:3]{}, i = 1 </Locals>|r
user:/AddOns/Greyskull/greyskull.lua:115: in function 'Greyskull.Initialize'
|caaaaaa<Locals> self = [table:1] </Locals>|r
user:/AddOns/Greyskull/greyskull.lua:374: in function 'OnPlayerActivated'
|caaaaaa<Locals> eventCode = 589824 </Locals>|r
Report comment to moderator  
Reply With Quote
Unread 05/25/21, 10:47 AM  
TheYcUtS
 
TheYcUtS's Avatar
AddOn Author - Click to view AddOns

Forum posts: 9
File comments: 87
Uploads: 3
@ghostbane

First, thx for the addon. Great deal, great work mate!

Second, any option to make borders/edges/bg like default esoui ?

For example
Last edited by TheYcUtS : 05/31/21 at 02:54 PM.
Report comment to moderator  
Reply With Quote
Unread 04/09/21, 01:39 PM  
YesImBattman
 
YesImBattman's Avatar

Forum posts: 4
File comments: 34
Uploads: 0
Re: Re: Damage total not resetting?

Originally Posted by ghostbane
Originally Posted by YesImBattman
Perhaps I just don't understand the functionality of the addon but after combat I expect the total to zero out but it does not. Is this proper behavior? Is there a reset function or action?

Thanks!
Your Weapon Damage or Spell Damage will never reach zero, unless you have no gear on. I think you are seeing it as DPS, which would go to zero if no damage is happening.

But what this addon displays is your characters Weapon/Spell Damage level ( available in your character sheet )
Thanks! That makes perfect sense!
Report comment to moderator  
Reply With Quote
Post A Reply



Category Jump: