Feature/vrm10 #596
Merged
Feature/vrm10 #596
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
saturday06
reviewed
Nov 9, 2020
| { | ||
| get | ||
| { | ||
| return String.Format("{0}.{1}", Major, Minor); |
There was a problem hiding this comment.
ここは String.Format("{0}.{1}.beta", Major, Minor) とすることを提案します。理由は、このままだと現在開発中のバージョンで出力したVRMモデルと、仕様と実装が揃った正式版で出力したモデルの違いをプログラムが判定できなくなってしまうからです。
There was a problem hiding this comment.
beta 部分はお好みの任意の文字列。数値と dot のみの制限にしたい場合は 0.999 などのバージョンを使うという方式があったと思います。
There was a problem hiding this comment.
未完成であることがわかるようにします。
saturday06
reviewed
Nov 9, 2020
| } | ||
| } | ||
|
|
||
| public const string VERSION = "1.0"; |
There was a problem hiding this comment.
こっちも上記と同様に変更する必要があるかもです。
|
v0.63.0 でマージ予定です。 |
|
Version 番号のメモ。 現状。 VRM-0.XX のエクスポート。こっちは今後も変えない方針。
VRM-1.0beta のエクスポート
VRM-1.0 のエクスポート
|
ousttrue
force-pushed
the
feature/vrm10
branch
from
0e13d39 to
985c893
Compare
|
#599 マージ後に rebase |
ousttrue
force-pushed
the
feature/vrm10
branch
from
82414db to
c96f033
Compare
|
#625 取り込みで rebase |
ousttrue
force-pushed
the
feature/vrm10
branch
2 times, most recently
from
12bc19a to
8be8d71
Compare
|
コードを整理
|
* MeshUtility を UniGLTF 下に移動 * Assets/VRM10 を追加 * JsonSchemaからのコード生成 UniGLTF/Editor/Generator を追加
ousttrue
force-pushed
the
feature/vrm10
branch
from
8be8d71 to
d53942a
Compare
|
rebase |
hiroj
approved these changes
Jan 7, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
UniVRM-1.0 の実装をこのリポジトリに同居させることに方針を変更し、移植したものです。
おもな変更点。
v0.63.1で先行して UniGLTF 分離と MeshUtility の UniGLTF下移動をしました。この PR は、おもに VRM-1.0 実装の追加になります。
Assets/VRM10 追加
UniVRM-1.0 実装
Separate unigltf #625 で実装した分割シリアライザと接続する VRM10シリアライザ実装(JsonShcmeaからのコード生成)
vrmlib
Format と VrmLib の接続
Assets/VRM10/Runtime/Format/IOVrmLib と Unity の接続
Assets/VRM10/Runtime/UnityBuilderAssets/VRM10/Runtime/VrmConverterAssets/VRM10/Runtime/TextureConvertAssets/VRM, Assets/UniGLTF 更新
VRM-1.0 をロードしてもエラーにならないように修正
Exporter のバリデーションを共通部品化できるように修正