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

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

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

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

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



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

関連記事

粘土をこねるようなスカルプトモデリング

ZBrushでアヴァン・ガメラを作ってみる パーツ分割

Open Shading Language (OSL)

Composition Rendering:Blenderに...

MPC社によるゴジラ(2014)のVFXブレイクダウン

ZBrushで人型クリーチャー

ラクガキの立体化

ディープラーニング

SONY製のニューラルネットワークライブラリ『NNabla』

HD画質の無駄遣い

OpenCVのバージョン3が正式リリースされたぞ

Mixamo:人型3Dキャラクターアニメーション制作サービス

Mask R-CNN:ディープラーニングによる一般物体検出・...

頭蓋骨からの顔復元と進化過程の可視化

OpenGVの用語

Google製オープンソース機械学習ライブラリ『Tensor...

映画『ブレードランナー 2049』のVFX

iPhone 3GSがますます欲しくなる動画

Photogrammetry (写真測量法)

素敵なパーティクル

オープンソースのネットワーク可視化ソフトウェアプラットフォー...

UnityのTransformクラスについて調べてみた

Maya LTのQuick Rigを試す

この本読むよ

UnityでLight Shaftを表現する

ZBrushでゴジラ2001を作ってみる 側頭部のボリューム...

ガメラ生誕50周年

ZBrushの練習 手のモデリング

ManuelBastioniLAB:人体モデリングできるBl...

Webサイトのワイヤーフレームが作成できるオンラインツール

プロシージャル手法に特化した本が出てるみたい(まだ買わないけ...

デザインのリファイン再び

ROSの薄い本

日立のフルパララックス立体ディスプレイ

OpenGVのライブラリ構成

ラクガキの立体化 1年半ぶりの続き

ブログをGoogle App EngineからAmazon ...

『スター・ウォーズ フォースの覚醒』のVFXブレイクダウン ...

Google Earth用の建物を簡単に作れるツール Goo...

Unite 2014の動画

MeshroomでPhotogrammetry

OpenCVの三角測量関数『cv::triangulatep...

コメント