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

Animation research #66

Open
wants to merge 9 commits into
base: an2-research
Choose a base branch
from

Conversation

quarkblitz
Copy link

Animation research, based on @RoadTrain's work.

@RoadTrain
Copy link
Contributor

@quarkblitz How well is the decoding working now?
Maybe I can come up with a way to import/export these to simplify further research/debug

@quarkblitz
Copy link
Author

@quarkblitz How well is the decoding working now? Maybe I can come up with a way to import/export these to simplify further research/debug

The code is based on a dumper I wrote earlier. The output of said dumper looked fine (as in no NaNs or anything weird, at least from what I noticed) for the (few) animations I tested it with. However I haven't tested playback of the decoded data on a model or something along those lines so maybe there's still something off that's hard to spot by just staring at the raw dump.

That being said, the code is entirely untested as Animation2Loader is not yet wired up anywhere in the project. It's basically a port of the loader code from the dumper, but it's well possible I got some bits wrong along the way, like range accessors for instance.

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

Successfully merging this pull request may close these issues.

None yet

2 participants