From 71a42d7f8f16814ca24feaa472d1f5c24a154fe3 Mon Sep 17 00:00:00 2001
From: nirmal <nirmalps@iitk.ac.in>
Date: Wed, 7 Oct 2020 21:38:48 +0530
Subject: [PATCH] Fix tests

---
 src/rep/mesh.jl  | 15 ++++++++-------
 test/cuda/rep.jl |  4 +++-
 test/rep.jl      |  2 +-
 3 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/src/rep/mesh.jl b/src/rep/mesh.jl
index 964f127..dd96751 100644
--- a/src/rep/mesh.jl
+++ b/src/rep/mesh.jl
@@ -859,6 +859,7 @@ end
 
 function _compute_verts_packed(m::TriMesh, refresh::Bool = false)
     if refresh || !(m._verts_packed_valid)
+        _compute_verts_list(m)
         verts_packed = _list_to_packed(m._verts_list)
         # avoiding setproperty!, as we are building packed
         # from list and list is always valid
@@ -872,15 +873,15 @@ function _compute_verts_padded(m::TriMesh, refresh::Bool = false)
     if refresh || !(m._verts_padded_valid)
 
         # FIXME: #38 chamfer_loss gradient doesn't reach packed, so using temp fix
-        _packed = get_verts_packed(m)
-        _padded = _packed_to_padded(_packed, m._verts_len, 0)
-        m._verts_padded = _padded
+        _compute_verts_packed(m)
+        verts_padded = _packed_to_padded(m._verts_packed, m._verts_len, 0)
+        # m._verts_padded = _padded
 
-        # _list_to_padded!(m._verts_padded, m._verts_list, 0, (3, m.V))
+        # _list_to_padded!(m._verts_padded, get_verts_list(m), 0, (3, m.V))
 
         # avoiding setproperty!, as we are building padded
         # from list and list is always valid
-        # setfield!(m, :_verts_padded, verts_padded)
+        setfield!(m, :_verts_padded, verts_padded)
         setfield!(m, :_verts_padded_valid, true)
         return nothing
     end
@@ -888,9 +889,9 @@ end
 
 function _compute_verts_list(m::TriMesh, refresh::Bool = false)
     if refresh || !(m._verts_list_valid)
-        if m._verts_packed !== nothing
+        if m._verts_packed_valid
             verts_list = _packed_to_list(m._verts_packed, m._verts_len)
-        elseif m._verts_padded !== nothing
+        elseif m._verts_padded_valid
             verts_list = _padded_to_list(m._verts_padded, m._verts_len)
         else
             error("not possible to contruct list without padded and packed")
diff --git a/test/cuda/rep.jl b/test/cuda/rep.jl
index 1aad31d..65932ad 100644
--- a/test/cuda/rep.jl
+++ b/test/cuda/rep.jl
@@ -90,7 +90,9 @@ end
     # IO Tests
     @testset "IO" begin
         mktempdir() do tmpdir
-            for ext in ["obj", "off", "stl", "ply", "2dm"]
+            # FIXME: MeshIO 2dm save/load breaking #62, #63
+            # for ext in ["obj", "off", "stl", "ply", "2dm"]
+            for ext in ["obj", "off", "stl", "ply"]
                 save_trimesh(joinpath(tmpdir, "test.$(ext)"), m)
                 m_loaded = load_trimesh(joinpath(tmpdir, "test.$(ext)"))
                 @test all(isapprox.(get_verts_packed(m_loaded), verts_list[1]))
diff --git a/test/rep.jl b/test/rep.jl
index 8fa5c3a..19ff7ae 100644
--- a/test/rep.jl
+++ b/test/rep.jl
@@ -98,7 +98,7 @@ end
     # IO Tests
     @testset "IO" begin
         mktempdir() do tmpdir
-            # FIXME: MeshIO 2dm save/load breaking #62
+            # FIXME: MeshIO 2dm save/load breaking #62, #63
             # for ext in ["obj", "off", "stl", "ply", "2dm"]
             for ext in ["obj", "off", "stl", "ply"]
                 save_trimesh(joinpath(tmpdir, "test.$(ext)"), m)