Skip to content
Snippets Groups Projects
Commit e5eacb97 authored by Yoel's avatar Yoel
Browse files

Light Multi is now correctly constant

parent 61a9c9cd
No related branches found
No related tags found
No related merge requests found
#define _USE_MATH_DEFINES
#include "DiffuseMaterial.h" #include "DiffuseMaterial.h"
#include <math.h>
#include <cassert> #include <cassert>
#include "../tools/Random.h" #include "../tools/Random.h"
...@@ -32,10 +35,7 @@ util::Vec3 DiffuseMaterial::scattered_d(const util::Vec3& d, ...@@ -32,10 +35,7 @@ util::Vec3 DiffuseMaterial::scattered_d(const util::Vec3& d,
const util::Vec3& n) const { const util::Vec3& n) const {
util::Vec3 rand = util::rand_vec3_in_circle(1); util::Vec3 rand = util::rand_vec3_in_circle(1);
util::Vec3 result = n + rand; util::Vec3 result = n + rand;
/*if (util::dot(n, result) <= 0) {
std::cout << n << " " << rand << std::endl;
assert(false);
}*/
return result; return result;
} }
bool DiffuseMaterial::scatter(const util::Vec3& d, const util::Vec3& n) const { bool DiffuseMaterial::scatter(const util::Vec3& d, const util::Vec3& n) const {
...@@ -44,6 +44,6 @@ bool DiffuseMaterial::scatter(const util::Vec3& d, const util::Vec3& n) const { ...@@ -44,6 +44,6 @@ bool DiffuseMaterial::scatter(const util::Vec3& d, const util::Vec3& n) const {
float DiffuseMaterial::calculateLightMultiplier(const util::Vec3& d_in, float DiffuseMaterial::calculateLightMultiplier(const util::Vec3& d_in,
const util::Vec3& d_out, const util::Vec3& d_out,
const util::Vec3& n) const { const util::Vec3& n) const {
return util::dot(n, d_in); return M_1_PI;
} }
} // namespace material } // namespace material
\ No newline at end of file
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment