-
Notifications
You must be signed in to change notification settings - Fork 32
Misc Feature Updates #149
base: main
Are you sure you want to change the base?
Misc Feature Updates #149
Conversation
…nsform instead of application through render object properties
… use native function.
…g algorithm slightly.
…tion on the bone edit window and associated functions.
…ct vectors for all cases relating to game geometry (ie everything but where required by Imgui) Update vector extension methods accordingly
…es. Split aliased bones into two separate kinds of subbone that can be checked for individually via pattern matching.
Co-authored-by: chirp <[email protected]>
…tract resolver to keep the serialized information from blowing up with redundant data from the ffxivclientstruct vector3 class
…d null references
…nd them more tightly to their real counterparts. Fix a bug wherein the jaw bone/s couldn't be edited without the values snapping back.
…osed more elegantly. Change name of reference pose public access variable. Add primitive weapon editing.
Oh damn, hella lotta stuff right here nice job!
|
|
Am I correct that with this the Movement hook basically does nothing anymore entirely? |
Whoops, yeah. Didn't delete it in the moment because I didn't feel like cleaning up all the related bits and then I just forgot. |
All cool! Its nice that I noticed this way instead of "wait, that was there just recently where did it go" haha |
It has a build error for me in GameDataHelper.cs line 87, FindModelByName doesnt exist. |
I'm pretty sure that's because of the last commit where structurerework was merged into it. |
Ah thats where I read it recently, will test |
For this ill propose we put the Parenting parts in Debug only mode for now, and the rest should be good to go then? Hmm, Posing mode does not appear to be fully compatible now.... |
Hmm as more things crop up, could we extract the Root Bone fix from this into its own PR, merge that and then release it in testing into hopefully a full release? |
I think chirp's gpose fix is still not a part of structureRework, so that's something that needs to be cherry picked out of this PR as well if we are going that route. |
Converted this to draft, hope thats okay with you! Doesnt really change anything just because of the fixes we had to do to get the previous version launch ready this will need some fixes, if you still want to work on it at all that is! |
Stuff that's (mostly) in this PR:
Stuff that's still weird:
Stuff that this PR doesn't have:
Data/BoneData
class is no longer needed (right now we really only care about human-readable bone names and their overarching categories), and the rest would probably work better as a JSON or some other external resource file