Skip to content

Commit

Permalink
LOAM: early bailout for visibility calculations on no change, reduce …
Browse files Browse the repository at this point in the history
…for_mt overhead

A. There are 4 conditions that can require a visibility/retess check:
1. Camera position change
2. camera direction change
3. heightmap change
4. grounddetail param change
Otherwise we can bail on a lot of expensive things

B. for_mt seems to have a high setup cost, and spins threads needlessly.
for_mt vertex index generation is only useful when a full retesselation is performed, otherwise the number of patches that need their indices regenerated is likely less than the number of threads.

C. side note: /debugview performance is now not strongly affected, side effect of less for_mt?
  • Loading branch information
Beherith authored and ashdnazg committed Apr 10, 2020
1 parent 11c87f3 commit eb3259d
Show file tree
Hide file tree
Showing 2 changed files with 202 additions and 160 deletions.
Loading

0 comments on commit eb3259d

Please sign in to comment.