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

Spine does not account for phaseAtlas #248

Open
PirateIzzy opened this issue Jan 30, 2024 · 2 comments
Open

Spine does not account for phaseAtlas #248

PirateIzzy opened this issue Jan 30, 2024 · 2 comments

Comments

@PirateIzzy
Copy link
Contributor

PirateIzzy commented Jan 30, 2024

When rendering Spine atlas files, phaseAtlas is not accounted for, meaning that certain boss phases (Chaos Zakum and Ursus) do not allow for rendering. I have recorded a video to showcase how the phaseAtlas shows additional phase data for Chaos Zakum (8800102) and Ursus (8881000), but WCR2 does not have an option to display these phases. Certain other Spine bosses, namely Elunite Elemental (9402197) and Julieta (9402280) are coded slightly differently and use a different phase technique from phaseAtlas.
https://www.youtube.com/watch?v=HKoDzwWM1xU

For reference, this is how phaseAtlas appears in game for Chaos Zakum.
Regular:
image

Phase 3 (the red crest is due to one of Zakum's buffs):
image

Phase 4:
image

The corresponding spritesheets are shown at 0:36 - 0:50 in the video, and I'll share them here:
Regular:
Mob 8800102 img zakum png

Phase 3:
Mob 8800102 img phaseAtlas zakum png 3

Phase 4:
Mob 8800102 img phaseAtlas zakum png 4

@Kagamia
Copy link
Owner

Kagamia commented Feb 3, 2024

Zakum and Ursus were introduced at a very early stage, it didnot fully use all spine features like Skins. So it designed another data structure (phaseAtlas) for the same approach.

Checked the entire CMS204 client, there're only 4 cases:
image

Do we really need the feature for them two🤣? Let me file a work item for it.

@PirateIzzy
Copy link
Contributor Author

PirateIzzy commented Feb 3, 2024

Up to you if you think it's worthwhile. Seems like Nexon doesn't add a lot of bosses with Spine animation anymore so might not be a high priority. Plus, Ursus is being removed so that's one less boss.

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

No branches or pull requests

2 participants