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

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

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

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

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



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

関連記事

ZBrushのお勉強

openMVGをWindows10 Visual Studi...

ZBrushと液晶ペンタブレットでドラゴンをモデリングするチ...

ハリウッド版「GAIKING」パイロット映像

『スター・ウォーズ 最後のジェダイ』のVFXブレイクダウン ...

Swark:コードからアーキテクチャ図を作成できるVSCod...

顔モデリングのチュートリアル

Maya には3 種類のシェーダSDKがある?

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

CycleGAN:ドメイン関係を学習した画像変換

Raspberry Pi 2を買いました

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

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

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

Maya LTのQuick Rigを試す

ZBrushのZmodelerとDynamic Subdiv...

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

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

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

ZBrush用トポロジー転送プラグイン『ZWrap Plug...

BlenderProc:Blenderで機械学習用の画像デー...

グローバルイルミネーションに手を出す前に、やり残したことがあ...

フリーのUV展開ツール Roadkill UV Tool

仮想関数

Google App EngineでWordPress

ZBrushでゴジラ2001を作ってみる 身体のSubToo...

LuxCoreRender:オープンソースの物理ベースレンダ...

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

Subsurface scatteringの動画

OpenGVの用語

組み込み向けのWindows OS 『Windows Emb...

PyDataTokyo主催のDeep Learning勉強会

自前Shaderの件 解決しました

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

GAN (Generative Adversarial Ne...

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

ZBrushでゴジラ2001を作ってみる 身体のシルエット出...

ArUco:OpenCVベースのコンパクトなARライブラリ

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

Autodesk Mementoでゴジラを3次元復元する

ZBrushで人型クリーチャー

ZBrushでアヴァン・ガメラを作ってみる 頬の突起を作り始...

コメント