Skip to content

Going to fix AnbMaker, not yet done #1046

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.

Sign up for GitHub

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

Jump to bottom
Draft
kenjiuno wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Going to fix AnbMaker, not yet done #1046

kenjiuno wants to merge 5 commits into from

Conversation

@kenjiuno
Copy link
Collaborator

@kenjiuno kenjiuno commented Jun 8, 2024 •
edited
Loading

Tasks

  • Implement command dump-srt in order to inspect the current Rx Ry Rz of ANB/MSET's FCurveFK/IK. 
    OpenKh.Command.AnbMaker.exe dump-srt C:\xxx\obj\P_EX100.mset
  • Check about Rx Ry Rz.

Plans

  • Add a non-Assimp importer to anb-ex.
    • Assimp animation system internally uses Quaternion for rotation channel Assimp.NodeAnimationChannel.RotationKeys. It needs another importer in order to respect original Rx/Ry/Rz rotation channels.
    • Assimp animation deals with baked motions instead of Keyframe-based motion. It needs another importer in order to access the original Keyframe-based motion.

kenjiuno added 5 commits June 8, 2024 13:14
@kenjiuno
Implement command dump-srt
1edd1a4
@kenjiuno
Fix bug of Time column
120eea7
@kenjiuno
Merge remote-tracking branch 'remotes/origin/master' into anbmaker-fix2
e9ba120
@kenjiuno
Changes WIP:
f067a60 
- Make some reference types to nullable due to comformance with `<Nullable>enable</Nullable>`.
- Adding `--use-gltf` option WIP. It is said that the representation of rotation of glLF is quaternion. It is not an Euler angle. So glTF may not help.
- Adding `dump-srt` command for inspection purpose, for anb/mset.
@kenjiuno
Adding dump-mdlx-srt command for debug something.
d96a3ab
@TopazTK
Copy link
Contributor

TopazTK commented Jan 23, 2025

Pinging this for updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kenjiuno @TopazTK