サンプルコードにも間違いはある?

MayaのDGNodeを使ったShaderの書き方をサンプルコード読みながら勉強してたんだけど、そもそも参考にしてたサンプルコード自体に間違いがあることに気づいた。
前にも貼ったPhongShaderのサンプル
サンプルをそのままビルドして実行してみればわかるんだけど、specular処理がちょっとおかしなことになる。

これはごく初歩的なミスで、要するに値の丸め忘れ。
サンプルコードだと、compute関数の中でif文使って( blindDataがNullな時とそうでない場合を)振り分けた後に、ほぼ同じShading処理を二箇所に書く構造になっていて、その内の一箇所に丸め忘れがある。
不思議なことにコードを見てると、二箇所のShading処理で別々の人が書いたように書き方が変わってたりする。(コメントの書き方も何か違う)

それで思ったんだけど、そもそもこのコードの書き方がミスを犯しやすい構造になってる気がする。if文書く位置をちょっと変えるだけでもっとメンテナンスしやすいのに…(この方が実行速度が速いんだろうか)
とか書いてたらまたおかしいところ見つけたぞ。(まあ良いか)
たぶん、最初に誰かが書いたのをバージョンアップか何かで他の誰かが書き換えて行くうちにおかしなことになっちゃったんだろう。

そんで、コード直すついでにDiffuseに関してはこの考えに従って書き換えてみた。(デザイン系ではあまり考えなくて良い部分ではある)



さて、CookTorranceがうまくいかないわけだが…。

関連記事

BlenderのRigifyでリギング

オープンソースの顔の動作解析ツールキット『OpenFace』

スターウォーズ エピソードVIIの予告編

OpenCV3.3.0でsfmモジュールのビルドに成功!

AfterEffectsプラグイン開発

iOSデバイスのためのフィジカル・コンピューティングツールキ...

SIGGRAPH ASIAのマスコット

素敵なパーティクル

Mitsuba 3:オープンソースの研究向けレンダラ

ZBrushでアヴァン・ガメラを作ってみる 歯を配置

Faceshiftで表情をキャプチャしてBlender上でM...

Raspberry Pi 2を買いました

Unite 2014の動画

VGGT: Visual Geometry Grounded...

OpenCV

書籍『3次元コンピュータビジョン計算ハンドブック』を購入

UnityでPoint Cloudを表示する方法

Math.NET Numerics:Unityで使える数値計...

MeshLab:3Dオブジェクトの確認・変換に便利なフリーウ...

viser:Pythonで使える3D可視化ライブラリ

『ピクサー展』へ行ってきた

geometry3Sharp:Unity C#で使えるポリゴ...

R-CNN (Regions with CNN featur...

ポリゴンジオメトリ処理ライブラリ『pmp-library (...

OpenGVの用語

参考になりそうなサイト

openMVG:複数視点画像から3次元形状を復元するライブラ...

ZBrushでアヴァン・ガメラを作ってみる 全体のバランス調...

ZBrushでゴジラ2001を作ってみる 姿勢の変更

ZBrushでアヴァン・ガメラを作ってみる 壊れたデータの救...

OpenCVで顔のランドマークを検出する『Facemark ...

機械学習に役立つPythonライブラリ一覧

自前Shaderの件 解決しました

SIGGRAPH ASIA 2009で学生ボランティア募集し...

HTML5・WebGLベースのグラフィックスエンジン『Goo...

pythonの機械学習ライブラリ『scikit-learn』

Manim:Pythonで使える数学アニメーションライブラリ

KelpNet:C#で使える可読性重視のディープラーニングラ...

ZBrushでアヴァン・ガメラを作ってみる 全体のバランス調...

Adobe Photoshop CS5の新機能

UnityでARKit2.0

IronPythonを使ってUnity上でPythonのコー...

コメント