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

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

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

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

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



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

関連記事

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

WordPressのサーバ引っ越し方法を考える

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

PyTorch3D:3Dコンピュータービジョンライブラリ

AnacondaとTensorFlowをインストールしてVi...

Math Inspector:科学計算向けビジュアルプログラ...

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

Blender 2.8がついに正式リリース!

ゴジラ(2014)のメイキング

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

Mean Stack開発の最初の一歩

Mayaでリアルな布の質感を作るチュートリアル

クラスの基本

Mayaのシェーディングノードの区分

Unity ARKitプラグインサンプルのチュートリアルを読...

Raspberry PiのGPIOを操作するPythonライ...

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

ZBrushでアヴァン・ガメラを作ってみる 頭頂部の作り込み...

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

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

プログラムによる景観の自動生成

Pythonの自然言語処理ライブラリ『NLTK(Natura...

CLO:服飾デザインツール

ZBrushでアヴァン・ガメラを作ってみる 爪とトゲを追加

Theia:オープンソースのStructure from M...

ラクガキの立体化 分割ラインの変更・バランス調整

3Dグラフィックスの入門書

PeopleSansPeople:機械学習用の人物データをU...

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

mentalrayのSubsurface Scatterin...

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

CGのためのディープラーニング

ゴジラ三昧

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

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

トランスフォーマー/ロストエイジのメイキング

ガメラ生誕50周年

PyDataTokyo主催のDeep Learning勉強会

ZBrushでアヴァン・ガメラを作ってみる 下半身のバランス...

ZBrushでアヴァン・ガメラを作ってみる おでこ(?)のバ...

参考書

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

コメント