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

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

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

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

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



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

関連記事

続・ディープラーニングの資料

映画『シン・仮面ライダー』 メイキング情報まとめ

ヒーローに変身なりきりアーケードゲーム『ナレルンダー』

Boost オープンソースライブラリ

Rerun:マルチモーダルデータの可視化アプリとSDK

Leap MotionでMaya上のオブジェクトを操作できる...

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

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

タマムシっぽい質感

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

ZBrushでアヴァン・ガメラを作ってみる モールドの彫り込...

Amazon EC2ログイン用の秘密鍵を無くした場合の対処方...

Blendify:コンピュータービジョン向けBlenderラ...

世界一下品なクマと世界一紳士なクマ

Mechanizeで要認証Webサイトをスクレイピング

OpenCVでカメラ画像から自己位置認識 (Visual O...

Kornia:微分可能なコンピュータービジョンライブラリ

ZBrushでアヴァン・ガメラを作ってみる 脚のポーズ調整

Gource:バージョン管理の履歴を可視化するツール

フリーで使えるスカルプト系モデリングツール『Sculptri...

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

Siggraph Asia 2009 カンファレンスの詳細

シン・ゴジラのファンアート

ZBrush4新機能ハイライト 3DCG CAMP 2010

pythonもかじってみようかと

CEDEC 3日目

AmazonEC2のインスタンスをt1.microからt2....

ポリゴン用各種イテレータと関数セット

Open3D:3Dデータ処理ライブラリ

ドットインストールのWordPress入門レッスン

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

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

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

Unity ARKitプラグインサンプルのドキュメントを読む

GoB:ZBrushとBlenderを連携させるアドオン

OpenVDB:3Dボリュームデータ処理ライブラリ

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

『パシフィック・リム: アップライジング』のVFXブレイクダ...

ラクガキの立体化 目標設定

Structure from Motion (多視点画像から...

Konashiを買った

Verilog HDL

コメント