Bake smoothness and roughnessFactor into textures. #222
Conversation
…nd roughnessFactor as hardcoded-value 1.0 in Unity world (Resolve #212)
|
This is a screenshot when import glTF models with blender. Exported parameters by Unity are |
|
対応はっや！試してみます！ |
|
ウオオーーーーいけたー！ありがとうございます！ |
| a = (byte)(255 - src.g), | ||
| // Bake roughness values into a texture. | ||
| // See: https://github.com/dwango/UniVRM/issues/212. | ||
| a = (byte)(255 - Math.Min(src.g * (1.0f - _smoothness), 255)), |
There was a problem hiding this comment.
こちら、 roughness = 1.0 - smoothness でなく、
roughness = (1.0 - smoothness)^2 とのことですので、修正をお願いします。
以下のような要因で即時の変更が難しい場合、次のPRにまわしていただいても構わないです。
- UniGLTF全編に対する変更が必要なこと
- エクスポート結果が変化すること（正しくはなります）
こちらの記事を参考にどうぞ。
Roughnessは学術的に確立されたパラメータではあるものの、視覚的変化が線形でないため、
SmoothnessというRoughnessを一段覆うパラメータが生まれたようです。
https://blogs.unity3d.com/jp/2016/01/25/ggx-in-unity-5-3/
There was a problem hiding this comment.
おお、ありがとうございます！変更いたします。記事も参考になります 🙏
There was a problem hiding this comment.
c8489af で対応しました。Import/Exportの段階ですべて変換してしまう方針です。
|
Reimport glTF models with Unity.
|
|
|
||
| var pixelSmoothness = src.a * _smoothnessOrRoughness; // smoothness | ||
| // https://blogs.unity3d.com/jp/2016/01/25/ggx-in-unity-5-3/ | ||
| var pixelRoughnessFactor = (1.0f - pixelSmoothness) * (1.0f - pixelSmoothness); |
There was a problem hiding this comment.
| var pixelRoughnessFactor = (1.0f - pixelSmoothness) * (1.0f - pixelSmoothness); | |
| var pixelRoughnessFactorSqrt = (1.0f - pixelSmoothness); | |
| var pixelRoughnessFactor = pixelRoughnessFactorSqrt * pixelRoughnessFactorSqrt; |
めちゃ細かいですが……ｗ
|
式の修正、レビュー対応済みです。 |
yutopp
deleted the
feature/bake_smoothness_and_roughness_factor_into_textures
branch
Resolve #212.
Now some conversion logic are introduced when converting MetallicRoughness maps.
Unity world to glTF world
smoothnessvalue is baked into exported textures when exportingroughnessFactoris hard-coded as1.0fin glTF (because this values are already baked into textures)glTF world to Unity world
roughnessFactorvalue is baked into imported textures when importingsmoothnessis hard-coded as1.0fin Unity (because this values are already baked into textures)