Rename.

Add some missing files.

examples/mesh-conv/Makefile

@@ -2,4 +2,4 @@
 EXTRA_CXXFLAGS :=
 
 all:
-	clang++ -std=c++11 -g -O1 -I../../ -I../common $(EXTRA_CXXFLAGS) -o mesh-modify mesh-modify.cc mesh-util.cc tinygltf_impl.cc
+	clang++ -std=c++11 -g -O1 -I../../ -I../common $(EXTRA_CXXFLAGS) -o mesh-conv mesh-conv.cc mesh-util.cc tinygltf_impl.cc

examples/mesh-conv/Makefile.meson

@@ -0,0 +1,5 @@
+all:
+	ninja
+
+clean:
+	ninja -t clean

examples/mesh-conv/README.md

@@ -0,0 +1,43 @@
+# Mesh modify experiment
+
+Sometimes we want to tweak mesh attributes(e.g. vertex position, uv coord, etc).
+glTF itself does not allow ASCII representation of such data.
+
+This example show how to
+
+- Export mesh data from .bin to .obj
+- Import mesh data to .bin(update corresponding buffer data) from .obj
+
+## Usage
+
+### Wavefront .obj to glTF
+
+```
+$ mesh-modify obj2gltf input.obj
+```
+
+All shapes in .obj are concatenated and create single glTF mesh.
+(tinyobjloader's skin weight extension `vw` supported)
+
+#### Limitation
+
+Buffer is stored as external file(`.bin`)
+
+### glTF to Wavefront .obj
+
+```
+$ mesh-modify gltf2obj input.gltf
+```
+
+.obj will be created for each glTF Mesh.
+(Skin weight data is exported to .obj using tinyobjloader's skin weight extension `vw`)
+
+#### Limitation
+
+Buffer is stored as external file(`.bin`)
+
+## TODO
+
+* [ ] obj2gltf: Assign glTF material from .mtl
+* [ ] modify/patch mesh geometry in glTF
+  * By using JSON Patch or JSON Merge feature

examples/mesh-conv/bootstrap-meson.sh

@@ -0,0 +1,3 @@
+rm -rf build
+CXX=clang++ CC=clang meson build -Db_sanitize=address -Db_lundef=false --buildtype=debug
+cp Makefile.meson build/Makefile

examples/mesh-conv/mesh-conv.cc

@@ -148,13 +148,11 @@ struct MeshPrim {
 };
 #endif
 
-#if 0
 static std::string GetFilePathExtension(const std::string &FileName) {
   if (FileName.find_last_of(".") != std::string::npos)
     return FileName.substr(FileName.find_last_of(".") + 1);
   return "";
 }
-#endif
 
 static size_t ComponentTypeByteSize(int type) {
   switch (type) {
@@ -296,13 +294,11 @@ static std::string FindFile(const std::vector<std::string> &paths,
   return std::string();
 }
 
-#if 0
 static std::string GetBaseDir(const std::string &filepath) {
   if (filepath.find_last_of("/\\") != std::string::npos)
     return filepath.substr(0, filepath.find_last_of("/\\"));
   return "";
 }
-#endif
 
 static int GetSlotId(const std::string &name) {
   if (name.rfind("TEXCOORD_", 0) == 0) {
@@ -575,7 +571,6 @@ static bool DumpMesh(const tinygltf::Model &model, const tinygltf::Mesh &mesh,
   return true;
 }
 
-#if 0
 static bool ExtractMesh(const std::string &asset_path, tinygltf::Model &model,
                         std::vector<example::MeshPrim> *outs) {
   // Get .bin data
@@ -623,28 +618,34 @@ static bool ExtractMesh(const std::string &asset_path, tinygltf::Model &model,
 
   return true;
 }
-#endif
 
 }  // namespace
 
 int main(int argc, char **argv) {
-  if (argc < 2) {
+  if (argc < 3) {
-    std::cout << "mesh-dump input.gltf" << std::endl;
+    std::cout << "mesh-modify <op> <args>" << std::endl;
+    std::cout << "  op\n\n";
+    std::cout << "  gltf2obj input.gltf <flip_texcoord_y>\n";
+    std::cout << "  obj2gltf input.obj <verbose>\n";
+
+    return EXIT_FAILURE;
   }
 
-#if 0
+  std::string op = argv[1];
+
+  if (op == "gltf2obj") {
+
+    bool flip_texcoord_y = true;
+    if (argc > 3) {
+      flip_texcoord_y = (std::atoi(argv[3]) > 0) ? true : false;
+    }
 
     tinygltf::Model model;
     tinygltf::TinyGLTF loader;
     std::string err;
     std::string warn;
 
-#ifdef _WIN32
+    std::string input_filename(argv[2]);
-  std::string input_filename(argv[1] ? argv[1]
-                                     : "../../../models/Cube/Cube.gltf");
-#else
-  std::string input_filename(argv[1] ? argv[1] : "../../models/Cube/Cube.gltf");
-#endif
 
     std::string ext = GetFilePathExtension(input_filename);
 
@@ -717,8 +718,7 @@ int main(int argc, char **argv) {
         filename = mesh.name + ".obj";
       }
 
-    bool flip_y = true; // flip texcoord Y?
+      bool ok = example::SaveAsObjMesh(filename, mesh, flip_texcoord_y);
-    bool ok = example::SaveAsObjMesh(filename, mesh,);
       if (!ok) {
         return EXIT_FAILURE;
       }
@@ -726,14 +726,19 @@ int main(int argc, char **argv) {
     }
 
     return ret ? EXIT_SUCCESS : EXIT_FAILURE;
-#else
+  } else if (op == "obj2gltf") {
+    std::string input_filename(argv[2]);
 
-  {
+    bool verbose = false;
-    std::string input_filename(argv[1]);
+    if (argc > 3) {
+      verbose = (std::atoi(argv[3]) > 0) ? true : false;
+    }
 
     // Require facevarying layout?
-    // false = try to keep GL-like mesh data as much as possible.
+    // facevarying representation is required if a vertex can have multiple normal/uv value.
-    // true = reorder vertex data and re-assign vertex indices.
+    // drawback of facevarying is mesh data increases.
+    // false = try to keep shared vertex representation as much as possible.
+    // true = reorder vertex data and re-assign vertex indices for facevarying data layout.
     bool facevarying = false;
 
     example::MeshPrim mesh;
@@ -742,7 +747,9 @@ int main(int argc, char **argv) {
       return EXIT_FAILURE;
     }
 
+    if (verbose) {
       PrintMeshPrim(mesh);
+    }
 
     std::string output_filename("output.gltf");
 
@@ -752,9 +759,11 @@ int main(int argc, char **argv) {
       return EXIT_FAILURE;
     }
     std::cout << "Write glTF: " << output_filename << "\n";
-  }
-
     return EXIT_SUCCESS;
-#endif
+  } else {
+
+    std::cerr << "Unknown operation: " << op << "\n";
+    return EXIT_FAILURE;
+  }
 
 }

examples/mesh-conv/mesh-util.hh

@@ -54,7 +54,7 @@ struct MeshPrim {
 ///
 /// Save MeshPrim as wavefront .obj
 ///
-bool SaveAsObjMesh(const std::string &filename, const MeshPrim &mesh);
+bool SaveAsObjMesh(const std::string &filename, const MeshPrim &mesh, bool flip_texcoord_y = true);
 
 ///
 /// Save MeshPrim as glTF mesh

examples/mesh-conv/meson.build

@@ -0,0 +1,7 @@
+project('mesh-conv', 'cpp', default_options : ['cpp_std=c++11'])
+
+thread_dep = dependency('threads')
+
+incdir = include_directories(['../../', '../common'])
+
+executable('mesh-conv', ['mesh-conv.cc', 'mesh-util.cc', 'tinygltf_impl.cc'], include_directories : incdir, dependencies : thread_dep)

examples/mesh-conv/tinygltf_impl.cc

@@ -0,0 +1,9 @@
+#define TINYGLTF_IMPLEMENTATION
+#define STB_IMAGE_IMPLEMENTATION
+#define STB_IMAGE_WRITE_IMPLEMENTATION
+
+#ifdef _WIN32
+#include "../../tiny_gltf.h"
+#else
+#include "tiny_gltf.h"
+#endif

examples/mesh-modify/README.md

@@ -1,13 +0,0 @@
-# Mesh modify experiment
-
-Sometimes we want to tweak mesh attributes(e.g. vertex position, uv coord, etc).
-glTF itself does not allow ASCII representation of such data.
-
-This example show how to
-
-- Export mesh data from .bin to .obj
-- Import mesh data to .bin(update corresponding buffer data) from .obj
-
-## Requirement
-
-Assume Buffer is stored as external file(`.bin`)

examples/mesh-modify/meson.build

@@ -1,7 +0,0 @@
-project('mesh-modify', 'cpp', default_options : ['cpp_std=c++11'])
-
-thread_dep = dependency('threads')
-
-incdir = include_directories(['../../', '../common'])
-
-executable('mesh-modify', ['mesh-modify.cc', 'mesh-util.cc', 'tinygltf_impl.cc'], include_directories : incdir, dependencies : thread_dep)