search

lighttransport / nanort

NanoRT, single header only modern ray tracing kernel.
MIT License
1.08k stars 90 forks source link

This might be typo. #13

Closed regrow1123 closed 7 years ago

regrow1123 commented 7 years ago

1130th line on nanort.h

pos = bmin[j] + (i + 0.5f) * bstep[j]; should be pos = bmin[j] + (i + 1.0f) * bstep[j];

There are some comments above this statement.

  //
  // Split pos bmin + (i + 1) * (bsize / BIN_SIZE)
  // +1 for i since we want a position on right side of the cell.
  //

This comment dosen't correspond to the below line. I guess one of them is wrong.

syoyo commented 7 years ago

Thank you for finding an issue!

Its now fixed.