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

[1.0] Fix springbone #974

Merged
merged 2 commits into from
May 17, 2022
Merged

[1.0] Fix springbone #974

merged 2 commits into from
May 17, 2022

Conversation

0b5vr
Copy link
Contributor

@0b5vr 0b5vr commented May 12, 2022

Description

  • Reverted a part of the PR [1.0] Clean up spring bone #945.
    • I believe I multiplied center space rotation to local space by mistake
  • Fixed several doc comments on private fields.
    • __prevTail , __currentTail , __nextTail are in center space instead of world space

ついでに

VRMの仕様の方に書いた文書も間違っていそうです……

https://github.com/vrm-c/vrm-specification/tree/master/specification/VRMC_springBone-1.0-beta#applying-rotation

0b5vr added 2 commits May 12, 2022 19:25
prevTail・currentTail・nextTailはcenter space
…ulate rotation"

The original commit: 0b8985e

center spaceとlocal spaceを混同して掛け算していそう
@0b5vr 0b5vr added bug Something isn't working VRM 1.0 labels May 12, 2022
@0b5vr 0b5vr added this to the VRM 1.0 milestone May 12, 2022
@0b5vr 0b5vr requested a review from nyamadan May 12, 2022 10:30
@0b5vr 0b5vr self-assigned this May 12, 2022
Copy link
Contributor

@nyamadan nyamadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@0b5vr 0b5vr merged commit 6ecc650 into 1.0 May 17, 2022
@0b5vr 0b5vr deleted the fix-springbone branch May 17, 2022 02:55
@0b5vr
Copy link
Contributor Author

0b5vr commented May 17, 2022

VRM1.0仕様側にもPR出しました: vrm-c/vrm-specification#372

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working VRM 1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants