Skip to content

More XBM updates - #277

Open
xiashtra wants to merge 5 commits into
xivdev:latestfrom
xiashtra:xbm-info
Open

xiashtra wants to merge 5 commits into
xivdev:latestfrom
xiashtra:xbm-info

Conversation

@xiashtra

Copy link
Copy Markdown

No description provided.

@github-actions

This comment has been minimized.

misled by stale data in XIViewer
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@GurgumXIV

Copy link
Copy Markdown

@xiashtra , I was looking at the "Inflict" columns, and can't figure out what status effect (if any) corresponds to "InflictDeath". If that's related to a status effect (maybe Doom?), could the column name use that, and if it means something else, could a comment mention that? Thanks!

@wolfcomp wolfcomp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some stuff from one of the ClientStructs devs so it gets talked about on github

Comment thread XBMPet.yml
Comment on lines +32 to +42
- name: InflictsSlow # next 11 use the same indexer as BNpcResist
- name: InflictsPetrify
- name: InflictsParalysis
- name: InflictsInterrupt
- name: InflictsBlind
- name: InflictsPoison
- name: InflictsStun
- name: InflictsSleep
- name: InflictsBind
- name: InflictsHeavy
- name: InflictsDeath

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are accessed as an array in the exe and not as individual fields

@xiashtra xiashtra Sep 12, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm aware that they are part of an array, but InflictsStatus[idx] is far less informative than giving them a name according to what they are actually used for.

Comment thread BNpcResist.yml
Comment on lines +3 to +13
- name: ResistsSlow
- name: ResistsPetrify
- name: ResistsParalysis
- name: ResistsInterrupt
- name: ResistsBlind
- name: ResistsPoison
- name: ResistsStun
- name: ResistsSleep
- name: ResistsBind
- name: ResistsHeavy
- name: ResistsDeath

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as XBMPet

@xiashtra

Copy link
Copy Markdown
Author

@xiashtra , I was looking at the "Inflict" columns, and can't figure out what status effect (if any) corresponds to "InflictDeath". If that's related to a status effect (maybe Doom?), could the column name use that, and if it means something else, could a comment mention that? Thanks!

The in-game filtering is called "Flat Damage/Death". It's not limited to a single status effect; it includes things that cause instant death, like Doom, but also things that are considered "flat damage", like Tail Screw. It's equivalent to the same category on BLU, which we previously labeled as CauseDeath, so I kept the naming similar.

@GurgumXIV

Copy link
Copy Markdown

@xiashtra , I was looking at the "Inflict" columns, and can't figure out what status effect (if any) corresponds to "InflictDeath". If that's related to a status effect (maybe Doom?), could the column name use that, and if it means something else, could a comment mention that? Thanks!

The in-game filtering is called "Flat Damage/Death". It's not limited to a single status effect; it includes things that cause instant death, like Doom, but also things that are considered "flat damage", like Tail Screw. It's equivalent to the same category on BLU, which we previously labeled as CauseDeath, so I kept the naming similar.

Ah, good to know. Appreciate the details!

@github-actions

Copy link
Copy Markdown
Contributor

✅ Validation succeeded

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.

3 participants