Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent nil error which occures sometimes at end of M+ run #261

Open
wants to merge 755 commits into
base: develop
Choose a base branch
from

Conversation

doadin
Copy link
Contributor

@doadin doadin commented Dec 2, 2023

796x RaiderIO/core.lua:7943: attempt to perform arithmetic on field 'killed' (a nil value)
[string "@RaiderIO/core.lua"]:7943: in function Update' [string "@RaiderIO/core.lua"]:7927: in function Setup'
[string "@RaiderIO/core.lua"]:9922: in function SetUIBosses' [string "@RaiderIO/core.lua"]:9616: in function OnBossKill'
[string "@RaiderIO/core.lua"]:8486: in function GetSummary' [string "@RaiderIO/core.lua"]:9671: in function UpdateShown'
[string "@RaiderIO/core.lua"]:10199: in function callbackFunc' [string "@RaiderIO/core.lua"]:1126: in function SendEvent'
[string "@RaiderIO/core.lua"]:1061: in function <RaiderIO/core.lua:1057>

Locals:
Skipped (In Encounter)

prevent the nil error by verifying the value exists.

- Made the replay timer count up and include the death penalty when a replay is running

- Made the ahead/behind colors show, so it only colorizes when there is a difference > 0

- Show the replay frame when in Mythic dungeons, because this will be when people will probably want to choose which replay to run against.

- Hide "your side" of the comparison when not in an active replay

- Reduced font size

- Always showing Boss count
added hover tooltip to the replay frame that shows the title of the replay you are going against

added in-combat indicator on the main section of the replay frame
trash only matters if it gets to 100%, so make sure that we do not show as being behind if live = 100 and replay > 100
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants