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

Extract Animation Table Patch #124

Merged
merged 2 commits into from Jul 18, 2021
Merged

Extract Animation Table Patch #124

merged 2 commits into from Jul 18, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jun 7, 2021

Adds a patch that extracts the hardcoded animation tables and put them in one file, BALANCE/anim.bin, for more flexibility.

Testing required for this, as usual.

BALANCE/anim.bin file specs:

  • 0x14 bytes header (5 pointers * 4 bytes, in that order: TrapTablePtr, ItemTablePtr, MoveTablePtr, GeneralTablePtr, SpecialMoveTablePtr; each table ends where the next table starts, except the last one which ends at EoF, tables may be 4 bytes aligned)
  • TrapTable: 0x2 bytes / trap: animation_id of an animation in the GeneralTable for each trap
  • ItemTable: 0x4 bytes / item: two animation_id of animations in the GeneralTable for each item; I don't remember which one is for which case
  • MoveTable: same as the Move Animation Table described in the animation doc (Table 1)
  • GeneralTable: same as the Global Animation Table described in the animation doc (Table 2)
  • SpecialMoveTable: same as the Special Move Animation Table described in the animation doc (Table 3)

@theCapypara
Copy link
Member

Thank you!!

@theCapypara theCapypara self-assigned this Jun 8, 2021
@theCapypara
Copy link
Member

Relates to SkyTemple/skytemple#274

@theCapypara theCapypara added this to Open in 1.4 via automation Jun 11, 2021
@theCapypara theCapypara added this to In Progress in Core: All open issues via automation Jun 11, 2021
@theCapypara theCapypara moved this from Open to Done just in Files in 1.4 Jun 11, 2021
@theCapypara theCapypara removed this from Done just in Files in 1.4 Jul 17, 2021
@theCapypara theCapypara added this to Open in 1.3 via automation Jul 17, 2021
@theCapypara theCapypara merged commit bcc4eac into SkyTemple:master Jul 18, 2021
Core: All open issues automation moved this from In Progress to Done Jul 18, 2021
@ghost ghost deleted the animations branch July 18, 2021 19:51
@theCapypara theCapypara moved this from Open to Done in 1.3 Jul 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
1.3
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants