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

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

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

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

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



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

関連記事

マジョーラ

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

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

ブログのデザイン変えました

Kubric:機械学習用アノテーション付き動画生成パイプライ...

Mean Stack開発の最初の一歩

Stanford Bunny

ジュラシック・パークの続編『ジュラシック・ワールド』

なんかすごいサイト

ちょっと凝り過ぎなWebキャンペーン:全日本バーベイタム選手...

Verilog HDL

『ゴジラ キング・オブ・モンスターズ』のVFXブレイクダウン

「ベンジャミン·バトン数奇な人生」でどうやってCGの顔を作っ...

2D→3D復元技術で使われる用語まとめ

チャットツール用bot開発フレームワーク『Hubot』

Unityで360度ステレオVR動画を作る

Paul Debevec

OpenCVでPhotoshopのプラグイン開発

AMIMOTO(PVM版)で作成したインスタンスをAMIMO...

OpenGV:画像からカメラの3次元位置・姿勢を推定するライ...

Pythonのソースコードに特化した検索エンジン『Nulle...

ドラマ『ファーストクラス』のモーショングラフィックス

Iterator

オープンソースのテクスチャマッピングシステム『Ptex』

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

C#で使える遺伝的アルゴリズムライブラリ『GeneticSh...

UnityのGameObjectの向きをScriptで制御す...

CreativeCOW.net

生物の骨格

AndroidもopenGLも初心者さ (でもJavaは知っ...

Webスクレイピングの勉強会に行ってきた

Transformers ”Reference & Buli...

OpenCV 3.1のsfmモジュールのビルド再び

Raspberry Piでセンサーの常時稼働を検討する

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

3Dスキャンしたテクスチャから照明を除去するUnityツール...

ZBrushで仮面ライダー3号を造る 仮面編

ZBrushのZmodelerとDynamic Subdiv...

イタリアの自動車ブランドFiatとゴジラがコラボしたCMのメ...

ブログの復旧が難航してた話

ニューラルネットワークで画像分類

Unity MonoBehaviourクラスのオーバーライド...

コメント