Changes between Initial Version and Version 1 of Ticket #1086, comment 1


Ignore:
Timestamp:
Sep 18, 2021, 7:46:12 PM (3 years ago)
Author:
mdavis

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #1086, comment 1

    initial v1  
    1 I doubt we'll be able to actually handle very small densify fractions, since (a) math precision issues are hard and (b) the implemented algorithm is O(nm) and uses the stack heavily, so is liable to OOM errors for very large numbers of vertices (which the small densify fraction effectively produces).
     1I doubt we'll be able to handle very small densify fractions, since (a) math precision issues are hard and (b) the implemented algorithm is O(nm) and uses the stack heavily, so is liable to OOM errors for very large numbers of vertices (which the small densify fraction effectively produces).
    22
    33We could hard-limit the densify fraction to something reasonable (e.g. 1E-4).  Or document and raise an error if the input exceeds this limit. 
    44
    55Of course, you can also do this in the client code.
     6
     7Is this just erroneous/fuzzed input, or is there a perceived need for this small a fraction?