Skip to content

doc: CCDIKSolver with generic SkinnedMesh#23449

Merged
Mugen87 merged 11 commits intomrdoob:devfrom
abernier:ccdiksolver-doc-example
Apr 27, 2022
Merged

doc: CCDIKSolver with generic SkinnedMesh#23449
Mugen87 merged 11 commits intomrdoob:devfrom
abernier:ccdiksolver-doc-example

Conversation

@abernier
Copy link
Copy Markdown
Contributor

@abernier abernier commented Feb 9, 2022

Note Merged
see CCDIKSolver doc

Align the CCDIKSolver documentation with its description, which says it works with a generic SkinnedMesh (not only MMDLoader).

Adding an embed example, based on the SkinnedMesh one.

See:

@Mugen87
Copy link
Copy Markdown
Collaborator

Mugen87 commented Apr 24, 2022

@Mugen87
Copy link
Copy Markdown
Collaborator

Mugen87 commented Apr 24, 2022

Looks good to me!

@takahirox Are you fine with this change?

@Mugen87 Mugen87 added this to the r140 milestone Apr 24, 2022
Comment thread docs/examples/en/animations/CCDIKSolver.html
Comment thread docs/examples/en/animations/CCDIKSolver.html Outdated
Comment thread docs/examples/en/animations/CCDIKSolver.html Outdated
@takahirox
Copy link
Copy Markdown
Collaborator

@takahirox Are you fine with this change?

Yes, this change looks good. CCDIKSolver is originally made for MMD but also designed for generic use. And we have good document now. So mentioning that it works with a generic SkinnedMesh (not only MMDLoader) sounds good to me.

@Mugen87 Mugen87 merged commit d4ac841 into mrdoob:dev Apr 27, 2022
@mrdoob
Copy link
Copy Markdown
Owner

mrdoob commented Apr 28, 2022

Thanks!

abernier added a commit to abernier/three.js that referenced this pull request Sep 16, 2022
* doc: CCDIKSolver with generic SkinnedMesh

* whitespaces fix

* tabs indentation

* tweaks

* Update CCDIKSolver.html

* typo

* lint fixes

* zh version

* oops
@abernier abernier deleted the ccdiksolver-doc-example branch September 22, 2022 11:02
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.

5 participants