-
Notifications
You must be signed in to change notification settings - Fork 45
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
raidboss: replace GameLog lines (HW) #95
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -343,11 +343,12 @@ const triggerSet: TriggerSet<Data> = { | |
}, | ||
}, | ||
}, | ||
// There is a GameLog message (en: The plasma shield is shattered!), but no corresponding | ||
// SystemLogMessage. The 0x19 (NetworkDeath) line shows up >2 seconds later (too late). | ||
{ | ||
id: 'A11S Plasma Shield Shattered', | ||
type: 'GameLog', | ||
netRegex: { line: 'The plasma shield is shattered.*?', capture: false }, | ||
|
||
type: 'NetworkEffectResult', | ||
netRegex: { name: 'Plasma Shield', currentHp: '0', capture: false }, | ||
response: Responses.spread(), | ||
}, | ||
Comment on lines
+346
to
353
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also not really directly related to this PR, but... Relevant log lines that could potentially be used instead. Maybe we add
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I wasn't wild about using the 0x25 line (even less so about using the 0x26 line), but having another option long-term would be nice. I'm not sure if |
||
{ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely unrelated to this PR but if we didn't want to use a
SystemLogMessage
here the only other option is the floor animation which is done viaActorControl
:There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. How "spammy" of a category is
EObjAnimation
? Seems like another potential candidate for adding to OP.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OverlayPlugin/OverlayPlugin#276 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guessing based on volume that the time period for those totals encompasses multiple days/log files (sorry if that's mentioned somewhere in the comment thread and I missed it). But 795 for
EObjAnimation
doesn't seem super heavy.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that was over like a month's worth of logs, but it was also mainly level 90 roulette content or UWU/TEA IIRC.