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

[vrm10] vrmlib による mesh 処理を gltf/vrm0.x のものと同じものに #2478

Merged
merged 4 commits into from
Nov 6, 2024

Conversation

ousttrue
Copy link
Contributor

@ousttrue ousttrue commented Nov 1, 2024

fixed #2474

vrmlib は少しずつ削除していこうと思っております。
とりあえず importer から削除。

1.0 は当初 vrmlib を主流に vrm-0.x 系の mesh 処理は置き換える予定だったのだけど、
あとから glb/vrm-0.x の importer も刷新されて良くなりました。
メンテナンスコスト削減のため vrmlib の方を削除して一本化する方針です。

  • migration
  • export

で使っているところがあるのでまだ削除はできません。

@ousttrue ousttrue added this to the next milestone Nov 1, 2024
@ousttrue ousttrue changed the title [vrm10] vrm10の importer の mesh 処理を gltf のものと同じものに [vrm10] vrmlib による mesh 処理を gltf/vrm0.x のものと同じものに Nov 1, 2024
@@ -3,7 +3,6 @@
using System.Linq;
using UniGLTF.Extensions.VRMC_vrm;
using UnityEngine;
using VrmLib;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

using VrmLib を減らしていく

@@ -38,18 +35,5 @@ async void Run()
Debug.Log($"write : {path}");
File.WriteAllBytes(path, exportedBytes);
}

static void Printmatrices(Model model)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

未使用の debug print ぽい

// ヘビーなため、別スレッドで Run する.
await awaitCaller.Run(() =>
{
// bin に対して右手左手変換を破壊的に実行することに注意 !(bin が変換済みになる)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

この設計がエッジケースでささる。

@ousttrue ousttrue requested a review from Santarh November 5, 2024 04:16
@ousttrue ousttrue merged commit 129af09 into vrm-c:master Nov 6, 2024
1 check passed
@ousttrue ousttrue deleted the fix/remove_vrmlib branch November 6, 2024 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UV sometimes flips when a same accessor is referred by multiple meshes
2 participants