-
Notifications
You must be signed in to change notification settings - Fork 21
Samples are starting at the boundaries of the global AABB #217
Conversation
retest this please |
@@ -164,6 +164,14 @@ struct RayCastRenderer::Impl | |||
tParamNameGL = glGetUniformLocation( program, "invModelViewMatrix" ); | |||
glUniformMatrix4fv( tParamNameGL, 1, false, frustum.getInvModelViewMatrix( ).array ); | |||
|
|||
tParamNameGL = glGetUniformLocation( program, "globalAabbMin" ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
globalAABBMin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a note while you are dividing by two.
add a changelog to notify which fix number you have fixed. I did not do that before myself on the previous 3 fixes, but you can do it :) |
I am a bit lost... There is no issue opened for this one and also this issue was not existing in the previous release because it was introduced by this recent nyquist limit computation Should I write in the changeog something like: "Imporvment: Samples start from the boundaries of the global AABB" ??? |
Ready to merge ?? :) |
@@ -164,6 +164,16 @@ struct RayCastRenderer::Impl | |||
tParamNameGL = glGetUniformLocation( program, "invModelViewMatrix" ); | |||
glUniformMatrix4fv( tParamNameGL, 1, false, frustum.getInvModelViewMatrix( ).array ); | |||
|
|||
// Because the volume is centered to the origin we can compute the volume AABB by using | |||
// the volume total size. | |||
Vector3f halfWorldSize = _volInfo.worldSize / 2.0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const
Updated |
+1 |
Samples are starting at the boundaries of the global AABB
No description provided.