Skip to content
Snippets Groups Projects
  • Bastien Montagne's avatar
    5352ac7e
    Revert "FBX Import: corrected import camera. Added setting camera (invert or not invert)." · 5352ac7e
    Bastien Montagne authored
    While it's nice to see attempts to fix cameras (their orientations are know
    broken in some case for ages), this commit has several issues:
    
     - It did not get any review.
     - It changes default behavior.
     - It adds yet another parameter.
     - It does not actually fixes anything, nor does it explain anything.
    
    The first two points in particular are red lights.
    
    But the last two are also more and more annoying, unless someone can provide
    a good, valid understanding of how camera orientation is supposed to work
    in FBX, am fairly not keen on accepting any more hack like that.
    This is just adding more parameters that users just don’t understand,
    and which generates by themselves even more bug reports.
    
    This reverts commit 9eddf664.
    5352ac7e
    History
    Revert "FBX Import: corrected import camera. Added setting camera (invert or not invert)."
    Bastien Montagne authored
    While it's nice to see attempts to fix cameras (their orientations are know
    broken in some case for ages), this commit has several issues:
    
     - It did not get any review.
     - It changes default behavior.
     - It adds yet another parameter.
     - It does not actually fixes anything, nor does it explain anything.
    
    The first two points in particular are red lights.
    
    But the last two are also more and more annoying, unless someone can provide
    a good, valid understanding of how camera orientation is supposed to work
    in FBX, am fairly not keen on accepting any more hack like that.
    This is just adding more parameters that users just don’t understand,
    and which generates by themselves even more bug reports.
    
    This reverts commit 9eddf664.