Changes between Initial Version and Version 1 of Ticket #1987, comment 11


Ignore:
Timestamp:
Jul 5, 2013, 5:37:30 AM (11 years ago)
Author:
historic_bruno

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1987, comment 11

    initial v1  
    11Damn, that's right. Sorry, I made a mistake. My method seems to prefer terrain with a bigger slope, so it's not mathematically equivalent. I guess we'll leave it this way.
    22
    3 But now that I'm inspecting it a bit better, shouldn't CCmpTerrain->CalcNormal use a fixed version of the method Terrain->CalcExactNormal? As that method is less CPU intensive (also when ported to fixeds), and it seems to do a better job at calculating the normal (as it takes account of the triangle you're on). That method does almost what I did, a single sqrt, and a single cross product.
     3But now that I'm inspecting it a bit better, shouldn't `CCmpTerrain->CalcNormal` use a fixed version of the method `Terrain->CalcExactNormal`? As that method is less CPU intensive (also when ported to fixeds), and it seems to do a better job at calculating the normal (as it takes account of the triangle you're on). That method does almost what I did, a single sqrt, and a single cross product.
    44