Skip to content

Add the acrolith family of skills#7759

Open
AJBats wants to merge 1 commit intoLandSandBoat:basefrom
AJBats:Acrolith_family_skills_2
Open

Add the acrolith family of skills#7759
AJBats wants to merge 1 commit intoLandSandBoat:basefrom
AJBats:Acrolith_family_skills_2

Conversation

@AJBats
Copy link
Contributor

@AJBats AJBats commented Jul 28, 2025

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Addresses #5494 by adding the acrolith family skills. There are TODOs left where more capture is needed.
There are still some capture TODOs:

  • Follow up adding acrolith.lua mixin to every acrolith type mob in the game. This change starts with just the one in Al Zahbi besieged.
  • Verify all of the ranges and AOE ranges of these abilities.
  • Tune skill power
  • Use the correct animation id for left arm and body dismemberment anims.

Existential TODOs:

  • How does a client zoning in know the disassembled state of an Acrolith that it wasn't present to witness?

Steps to test these changes

!zone al zahbi
!spawnmob 16974118 (spawns near the gate to outside)
Feed TP until debug logging reports all of the limbs have been exhausted (Only way to do this since we still need to capture the proper limb falling animations)
Acrolith will only do earthshatter with no limbs or body left.

@AJBats
Copy link
Contributor Author

AJBats commented Jul 28, 2025

There's a lot of capture TODOs here. I'm very interested in learning more about how this is done. If there are community resources for how to get these captures I'd be interested in helping out to make this PR more complete.

@AJBats AJBats force-pushed the Acrolith_family_skills_2 branch 2 times, most recently from ce854c4 to ca42d5f Compare July 29, 2025 13:32
@UmeboshiXI
Copy link
Contributor

UmeboshiXI commented Jul 29, 2025

How does a client zoning in know the disassembled state of an Acrolith that it wasn't present to witness?

Can use mob:setAnimationSub() to switch between the different states (Missing arms etc) so other players or yourself will see the mob animations update correctly after moving out of range/zoning.

You could probably set it using a listener in your mixin to trigger it (Likely on skill finish).

Edit:

Using different animation sub values, I could not find one where the left arm is destroyed. Might need to get a full capture to see if this actually exists.

@AJBats AJBats marked this pull request as ready for review July 29, 2025 14:53
@AJBats
Copy link
Contributor Author

AJBats commented Jul 29, 2025

Can use mob:setAnimationSub() to switch between the different states (Missing arms etc) so other players or yourself will see the mob animations update correctly after moving out of range/zoning.

Awesome! TIL about animation sub. But yeah, if you have any info on how to do the captures I want to learn. This PR is functionally equivalent game play (as for the limb rules) but the animations are all wrong.

@sruon
Copy link
Contributor

sruon commented Jul 29, 2025

Can use mob:setAnimationSub() to switch between the different states (Missing arms etc) so other players or yourself will see the mob animations update correctly after moving out of range/zoning.

Awesome! TIL about animation sub. But yeah, if you have any info on how to do the captures I want to learn. This PR is functionally equivalent game play (as for the limb rules) but the animations are all wrong.

There's a capture discord where you can find guides but I can't add you. If you ask around the FFXI Private Servers discord maybe someone can add you.

This addon for Windower is generally used to capture information. An Ashita v4 version will be available in a few weeks.

@Xaver-DaRed Xaver-DaRed added the hold On hold, pending further action/info label Aug 1, 2025
Addresses LandSandBoat#5494 by adding the acrolith family skills. There are TODOs left where more capture is needed.
@AJBats AJBats force-pushed the Acrolith_family_skills_2 branch from ca42d5f to 9089ccd Compare September 17, 2025 00:58
@AJBats
Copy link
Contributor Author

AJBats commented Sep 17, 2025

I wont be able to make more progress on this, but I have done some capture and study of retail Acrolith I think is worth sharing if anyone picks up the torch in the future. My notes are here: https://docs.google.com/document/d/184LPAs4-fFE9Zzh5eIBKowlmyB7IaFnKXCvp6ry3ajU/edit?tab=t.0

  • The rate of body part fall apart is likely after a # of crits, this rate needs to be sourced from more capture trials.
  • I have never seen the animation sub for losing the left arm. Either by capture or by enumerating through the animation sub values experimentally. Does it really exist?
  • The wiki says the Acrolith can dismemberment themselves from right arm -> left arm -> body, but through all of my experimentation I only observed the right arm.

I have made this commit match roughly what I observed, its approximately the behaviors of the Acrolith in Salvage II. I never was able to test against an Acorlith in besieged as I could never play when Besieged was ongoing.

I hope this helps my future self, or another supporter who takes on #5494.

@MowFord
Copy link
Contributor

MowFord commented Sep 17, 2025

Fwiw I don't mind fielding questions in discord if you'd like to learn more. I just did a PR that handles animation subs changed during mob skills: #7972

Separately there's a bit of a learning curve for passing cicd checks. The reports are mostly self explanatory if you click the failed jobs, but happy to help if you need it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold On hold, pending further action/info

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants