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] change: new constraint spec #355

Merged
merged 8 commits into from
Mar 17, 2022
Merged

[1.0] change: new constraint spec #355

merged 8 commits into from
Mar 17, 2022

Conversation

0b5vr
Copy link
Contributor

@0b5vr 0b5vr commented Feb 25, 2022

Description

前回の技術委員会での議論に基づき、コンストレイントの仕様に対して大幅な改訂を行いました。

今回の仕様ですが、Domain Specificに、使用用途に直結するコンストレイントの仕様を定義することを目指しました。
これまでのコンストレイントの議論での、環境間の挙動の差異の吸収・厳密な挙動の定義を行うことの難しさを踏まえ、
「このコンストレイントはこう使う」という使用用途に最適化したコンストレイントへの仕様の変更を行いました。
そのため、実装間での挙動の差異についてはある程度寛容な仕様となっています。
加えて、設定項目も最低限に留めることにより、実装・使用共により簡単となることを目指しています。

TODOs

  • 英語仕様
  • 仕様・スキーマ見直し

Points need review

  • スキーマ問題ないですか?
    • パラメタ忘れ・ドキュメンテーションとの乖離・required忘れなど
  • 仕様、十分に明解ですか?
  • 英語、パッと見問題なさそうですか?
  • 各実装、問題なく追従できそうですか?

@0b5vr 0b5vr added documentation Improvements or additions to documentation schema Constraint labels Feb 25, 2022
@0b5vr 0b5vr added this to the v1.0 milestone Feb 25, 2022
@0b5vr 0b5vr requested a review from ousttrue February 25, 2022 12:06
@0b5vr 0b5vr self-assigned this Feb 25, 2022
fix required field of constraint
doctoc README.ja.md
fix wrong required spec on explainer
add English explainer
@0b5vr
Copy link
Contributor Author

0b5vr commented Feb 28, 2022

英語仕様まで書き終わりました。

@0b5vr 0b5vr marked this pull request as ready for review February 28, 2022 10:04
UpperArm-LowerArmはAimとRollを組み合わせないと難しそうなので、これの言及は仕様内ではやめておく
fromToQuatをローカル空間に戻すのを忘れていた
@0b5vr
Copy link
Contributor Author

0b5vr commented Mar 3, 2022

ちょっとアップデートしました:

b89de38: UpperArm-LowerArmのツイストですが、AimとRollを2トランスフォーム組み合わせて実装しないと実現しなさそうな様子だったので、これについての仕様内での言及を避けました。

c518a9d: AimConstraintのExample Implementationについて、ワールド空間からローカル空間への変換が漏れていたため、これを追加しました。

Copy link
Contributor

@ousttrue ousttrue left a comment

Choose a reason for hiding this comment

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

よさそう

0b5vr added a commit to pixiv/three-vrm that referenced this pull request Mar 4, 2022
0b5vr added a commit to pixiv/three-vrm that referenced this pull request Mar 4, 2022
@0b5vr 0b5vr changed the title [1.0] WIP change: new constraint spec [1.0] change: new constraint spec Mar 17, 2022
@0b5vr 0b5vr merged commit 5cec3ce into vrm-c:master Mar 17, 2022
@0b5vr 0b5vr deleted the new-constraint branch March 17, 2022 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Constraint documentation Improvements or additions to documentation schema
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants