-
Notifications
You must be signed in to change notification settings - Fork 596
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
Identify some unaccounted and blob asset data #1953
base: main
Are you sure you want to change the base?
Conversation
@@ -12,6 +12,6 @@ | |||
<DList Name="object_door_killer_DL_001550" Offset="0x1550"/> | |||
<DList Name="object_door_killer_DL_0017B8" Offset="0x17B8"/> | |||
<DList Name="object_door_killer_DL_001A58" Offset="0x1A58"/> | |||
<Skeleton Name="object_door_killer_Skel_001BC8" Type="Flex" LimbType="Standard" Offset="0x1BC8"/> | |||
<Skeleton Name="object_door_killer_Skel_001BC8" Type="Flex" LimbType="LOD" Offset="0x1BC8"/> |
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.
lod type? this seems weird, i dont think the code that draws it agrees. can the non-lod draw functions draw lod limbs?
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.
The struct layout for StandardLimb and LodLimb is the same except for an extra Gfx*
pointer, so I think non-lod draw functions would work on LodLimb. I'm not really sure what to make of this, for these limbs the two display lists are the same so it doesn't seem very LOD-y
SkelAnime_Init(play, &this->skelAnime, &gBarinadeBariSkel.sh, &gBarinadeBariAnim, NULL, NULL, 0); | ||
this->actor.shape.yOffset = 400.0f; |
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.
these cases of using the sh
field feel weird, wasnt aware there were cases of the wrong skelanime function being used. I guess if it ends up working in practice its fine
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.
An alternative could be: (SkeletonHeader*)&gBarinadeBariSkel
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.
right yeah I was thinking this as well. I think i like a cast more but someone else should prob weigh in too
Feedback from discord: skeleton weirdness should get Also this conflicts with #1973, I'd rather that get merged first and I'll update this PR after |
do we want to try reviving this soon? |
I'll get to this eventually sorry, just that doing this for other versions too is not the most exciting |
You don't have to do it for everything, assets are still very much best effort territory |
yeah I just mean solving merge conflicts with all the _v2.xml files |
I tried to go through all "unaccounted" or "blob" assets (except for room files). I kept track of things with a spreadsheet (notes with question marks are things I didn't solve in this PR).
Common problems:
gsDPLoadTLUT_pal256
. I left these as-is for now.