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

Docs #1335

Merged
merged 21 commits into from
Nov 28, 2024
Merged

Docs #1335

merged 21 commits into from
Nov 28, 2024

Conversation

anatawa12
Copy link
Owner

@anatawa12 anatawa12 commented Nov 7, 2024

Fix #1321
Fix #1207
Fix #1229
Fix #1321
Fix #1337
Fix #1297
Fix #1144
Fix #1175
Fix #1004

@anatawa12 anatawa12 added the documentation Improvements or additions to documentation label Nov 7, 2024
@anatawa12 anatawa12 marked this pull request as ready for review November 7, 2024 11:08
@anatawa12
Copy link
Owner Author

#1337 もやる

@anatawa12 anatawa12 marked this pull request as draft November 8, 2024 05:19
@anatawa12 anatawa12 marked this pull request as ready for review November 8, 2024 06:11
@anatawa12
Copy link
Owner Author

レビュー途中だけど #1144 追加します

In previous version, I don't stated NDMF Pass since it's a special version of `IVRCSDKPreprocessAvatarCallback` but Sayamame told me it's not explict enough so I did.
@anatawa12
Copy link
Owner Author

ほぼそのままだけどwarnなどで書き直した

@Sayamame-beans
Copy link
Collaborator

他の場所だけれど、2点抜けてることがあったから足したい

  • Manual Bakeについてどこかで言及
    • 今はhard limitに引っ掛かったときの話でしか出てきてないっぽい
  • Remove Mesh By Mask、作成押したら新規生成されるから、基本それで足りる旨を書きたい
    • (メインテクスチャを複製してマスクにしている例を観測したため。UV表示されてるし必須ではない気がする)

@anatawa12
Copy link
Owner Author

Manual Bake、Hard Limitくらいしか使わなくない?

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Nov 24, 2024

Manual Bake、Hard Limitくらいしか使わなくない?

VRMとか?

@anatawa12
Copy link
Owner Author

他の場所だけれど、2点抜けてることがあったから足したい

* Manual Bakeについてどこかで言及
  
  * 今はhard limitに引っ掛かったときの話でしか出てきてないっぽい

* Remove Mesh By Mask、作成押したら新規生成されるから、基本それで足りる旨を書きたい
  
  * (メインテクスチャを複製してマスクにしている例を観測したため。UV表示されてるし必須ではない気がする)

これってさやまめさんがやると認識してたけど、私がやってくださいってことだった?

@Sayamame-beans
Copy link
Collaborator

忘れてた

@anatawa12
Copy link
Owner Author

今月中にマージ & リリースしたいので、模試間に合わなさそうであればできれば入れないで一旦リリースにしたい。(どうせバグ報告来るのでそれの修正リリースがあとに来るので)

@Sayamame-beans
Copy link
Collaborator

トップページにも問い合わせ先書いても良いかも(書き方どうしたら良いか分からない)
(あとオフトピだけどBoothページの説明更新忘れてるかも)

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Nov 27, 2024

FAQのOSCの話、Asset Descriptionにパラメーター情報追加したときに更新忘れてるっぽい
変えて良さそう(変える場合はそちらで草案お願いします 🙏 )

@Sayamame-beans
Copy link
Collaborator

他の場所だけれど、2点抜けてることがあったから足したい

* Manual Bakeについてどこかで言及
  
  * 今はhard limitに引っ掛かったときの話でしか出てきてないっぽい

* Remove Mesh By Mask、作成押したら新規生成されるから、基本それで足りる旨を書きたい
  
  * (メインテクスチャを複製してマスクにしている例を観測したため。UV表示されてるし必須ではない気がする)

これってさやまめさんがやると認識してたけど、私がやってくださいってことだった?

やりました(その他の変更点も含めてレビューお願いします 🙏 )

@anatawa12
Copy link
Owner Author

OSC更新しました。

.docs/content/docs/tutorial/basic-usage/index.ja.md Outdated Show resolved Hide resolved
.docs/content/docs/tutorial/basic-usage/index.md Outdated Show resolved Hide resolved
.docs/content/docs/faq/index.ja.md Outdated Show resolved Hide resolved
.docs/content/docs/faq/index.ja.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

2点だけ

.docs/content/docs/faq/index.md Outdated Show resolved Hide resolved
.docs/content/docs/faq/index.ja.md Outdated Show resolved Hide resolved
@anatawa12
Copy link
Owner Author

(あとオフトピだけどBoothページの説明更新忘れてるかも)

はどういう糸化聞いてなかった

@Sayamame-beans
Copy link
Collaborator

連絡方法とかメッシュ結合のこととか言及書いてあったから若干更新した方が良いのかも?って
画像はまあどっちでもいい(最適化結果変わってるかもだけど)

@anatawa12
Copy link
Owner Author

連絡先はたしかにそうね。NDMF Discordを書いたほうがいいかも。

@Sayamame-beans
Copy link
Collaborator

とりあえずmerge可能です
また何かあったら変えるかもだけど一旦良いと思う

@anatawa12
Copy link
Owner Author

merge~ RC打ちます

@anatawa12 anatawa12 merged commit aab7fff into master Nov 28, 2024
6 checks passed
@anatawa12 anatawa12 deleted the docs branch November 28, 2024 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment