1. 21 Feb, 2020 2 commits
    • Jeremy Reizenstein's avatar
      lint things · e491efb8
      Jeremy Reizenstein authored
      Summary:
      Lint related fixes: Improve internal/OSS consistency. Fix the fight between black and certain pyre-ignore markers by moving them to the line before.
      Use clang-format-8 automatically if present. Small number of pyre fixes.
      
      arc doesn't run pyre at the moment, so I put back the explicit call to pyre. I don't know if there's an option somewhere to change this.
      
      Reviewed By: nikhilaravi
      
      Differential Revision: D19780518
      
      fbshipit-source-id: ef1c243392322fa074130f6cff2dd8a6f7738a7f
      e491efb8
    • merayxu's avatar
      Fixed windows MSVC build compatibility (#9) · 9e21659f
      merayxu authored
      Summary:
      Fixed a few MSVC compiler (visual studio 2019, MSVC 19.16.27034) compatibility issues
      1. Replaced long with int64_t. aten::data_ptr\<long\> is not supported in MSVC
      2. pytorch3d/csrc/rasterize_points/rasterize_points_cpu.cpp, inline function is not correctly recognized by MSVC.
      3. pytorch3d/csrc/rasterize_meshes/geometry_utils.cuh
      const auto kEpsilon = 1e-30;
      MSVC does not compile this const into both host and device, change to a MACRO.
      4. pytorch3d/csrc/rasterize_meshes/geometry_utils.cuh,
      const float area2 = pow(area, 2.0);
      2.0 is considered as double by MSVC and raised an error
      5. pytorch3d/csrc/rasterize_points/rasterize_points_cpu.cpp
      std::tuple<torch::Tensor, torch::Tensor> RasterizePointsCoarseCpu() return type does not match the declaration in rasterize_points_cpu.h.
      Pull Request resolved: https://github.com/facebookresearch/pytorch3d/pull/9
      
      Reviewed By: nikhilaravi
      
      Differential Revision: D19986567
      
      Pulled By: yuanluxu
      
      fbshipit-source-id: f4d98525d088c99c513b85193db6f0fc69c7f017
      9e21659f
  2. 19 Feb, 2020 1 commit
    • Jeremy Reizenstein's avatar
      MACOSX_DEPLOYMENT_TARGET=10.14 · bdc2bb57
      Jeremy Reizenstein authored
      Summary:
      pybind now seems to need C++17 on a mac, so advise people to use it. (Also delete an unused variable to silence a warning I got on a mac build.)
      
      Reported in github issue #68.
      
      Reviewed By: nikhilaravi
      
      Differential Revision: D19970512
      
      fbshipit-source-id: f9be20c8ed425bd6ba8d009a7d62dad658dccdb1
      bdc2bb57
  3. 07 Feb, 2020 1 commit
  4. 31 Jan, 2020 1 commit
    • Haoqi Fan's avatar
      update install.md · 25c2f340
      Haoqi Fan authored
      Reviewed By: bottler, wanyenlo
      
      Differential Revision: D19658045
      
      fbshipit-source-id: a623a81c1ed1fa4054ea55bf06a2926e297b7966
      25c2f340
  5. 24 Jan, 2020 1 commit
  6. 23 Jan, 2020 1 commit