[SM64] Add gltf support to animations#700
Draft
Lilaa3 wants to merge 11 commits into
Draft
Conversation
also bug fix for duplicate_name
This gives enough info for a "compliant" animation exporter, but not more. also change the naming of the extensions cause it was out of spec lol
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Introduce as much info as needed for a "compliant" animation converter, repo side. This is being done instead of other glTF impls for two reasons:
This has the feature of porting over existing animation tables via reference, it does not attempt to manually fix the whole anims/ enviroment because that would take importing, which assuming a repo where this would be needed thats not an option, ideally a repo would have glTF to begin with and fast64 wouldnt have to handle anything.
Animations in glTF mode are basically identical to their C counterpart in settings, tho they allow you to reference a table via file instead of the c table names.
Two very early impls of FAST64_joint_sm64_geo and FAST64_scene_sm64_settings are also present. These are not and are not intended to be feature complete in this PR. The focus is allowing a repo to do exporting compliant to what fast64 does, nothing else nothing more.
There is some minor cleanups, only really in the animations folder for minor bugs and structure.
All things considered this is not a perfect implementation by any means and docs were kinda rushed.