Commit 9175af6f by Alex Bustin Committed by Alex Bustin

Respect 'mesh compression' editor import option in Assimp (ie. FBX) and glTF importers

parent 57d21ebe
...@@ -952,6 +952,9 @@ Error EditorSceneImporterGLTF::_parse_meshes(GLTFState &state) { ...@@ -952,6 +952,9 @@ Error EditorSceneImporterGLTF::_parse_meshes(GLTFState &state) {
return OK; return OK;
} }
bool compress_vert_data = state.import_flags & IMPORT_USE_COMPRESSION;
uint32_t mesh_flags = compress_vert_data ? Mesh::ARRAY_COMPRESS_DEFAULT : 0;
Array meshes = state.json["meshes"]; Array meshes = state.json["meshes"];
for (GLTFMeshIndex i = 0; i < meshes.size(); i++) { for (GLTFMeshIndex i = 0; i < meshes.size(); i++) {
print_verbose("glTF: Parsing mesh: " + itos(i)); print_verbose("glTF: Parsing mesh: " + itos(i));
...@@ -1206,7 +1209,7 @@ Error EditorSceneImporterGLTF::_parse_meshes(GLTFState &state) { ...@@ -1206,7 +1209,7 @@ Error EditorSceneImporterGLTF::_parse_meshes(GLTFState &state) {
} }
//just add it //just add it
mesh.mesh->add_surface_from_arrays(primitive, array, morphs); mesh.mesh->add_surface_from_arrays(primitive, array, morphs, Dictionary(), mesh_flags);
if (p.has("material")) { if (p.has("material")) {
const int material = p["material"]; const int material = p["material"];
...@@ -2951,6 +2954,7 @@ Node *EditorSceneImporterGLTF::import_scene(const String &p_path, uint32_t p_fla ...@@ -2951,6 +2954,7 @@ Node *EditorSceneImporterGLTF::import_scene(const String &p_path, uint32_t p_fla
String version = asset["version"]; String version = asset["version"];
state.import_flags = p_flags;
state.major_version = version.get_slice(".", 0).to_int(); state.major_version = version.get_slice(".", 0).to_int();
state.minor_version = version.get_slice(".", 1).to_int(); state.minor_version = version.get_slice(".", 1).to_int();
state.use_named_skin_binds = p_flags & IMPORT_USE_NAMED_SKIN_BINDS; state.use_named_skin_binds = p_flags & IMPORT_USE_NAMED_SKIN_BINDS;
......
...@@ -279,6 +279,9 @@ class EditorSceneImporterGLTF : public EditorSceneImporter { ...@@ -279,6 +279,9 @@ class EditorSceneImporterGLTF : public EditorSceneImporter {
Map<GLTFNodeIndex, Node *> scene_nodes; Map<GLTFNodeIndex, Node *> scene_nodes;
// EditorSceneImporter::ImportFlags
uint32_t import_flags;
~GLTFState() { ~GLTFState() {
for (int i = 0; i < nodes.size(); i++) { for (int i = 0; i < nodes.size(); i++) {
memdelete(nodes[i]); memdelete(nodes[i]);
......
...@@ -298,6 +298,7 @@ EditorSceneImporterAssimp::_generate_scene(const String &p_path, aiScene *scene, ...@@ -298,6 +298,7 @@ EditorSceneImporterAssimp::_generate_scene(const String &p_path, aiScene *scene,
state.assimp_scene = scene; state.assimp_scene = scene;
state.max_bone_weights = p_max_bone_weights; state.max_bone_weights = p_max_bone_weights;
state.animation_player = nullptr; state.animation_player = nullptr;
state.import_flags = p_flags;
// populate light map // populate light map
for (unsigned int l = 0; l < scene->mNumLights; l++) { for (unsigned int l = 0; l < scene->mNumLights; l++) {
...@@ -829,6 +830,8 @@ EditorSceneImporterAssimp::_generate_mesh_from_surface_indices(ImportState &stat ...@@ -829,6 +830,8 @@ EditorSceneImporterAssimp::_generate_mesh_from_surface_indices(ImportState &stat
Ref<ArrayMesh> mesh; Ref<ArrayMesh> mesh;
mesh.instance(); mesh.instance();
bool has_uvs = false; bool has_uvs = false;
bool compress_vert_data = state.import_flags & IMPORT_USE_COMPRESSION;
uint32_t mesh_flags = compress_vert_data ? Mesh::ARRAY_COMPRESS_DEFAULT : 0;
Map<String, uint32_t> morph_mesh_string_lookup; Map<String, uint32_t> morph_mesh_string_lookup;
...@@ -1266,7 +1269,7 @@ EditorSceneImporterAssimp::_generate_mesh_from_surface_indices(ImportState &stat ...@@ -1266,7 +1269,7 @@ EditorSceneImporterAssimp::_generate_mesh_from_surface_indices(ImportState &stat
morphs[j] = array_copy; morphs[j] = array_copy;
} }
mesh->add_surface_from_arrays(primitive, array_mesh, morphs); mesh->add_surface_from_arrays(primitive, array_mesh, morphs, Dictionary(), mesh_flags);
mesh->surface_set_material(i, mat); mesh->surface_set_material(i, mat);
mesh->surface_set_name(i, AssimpUtils::get_assimp_string(ai_mesh->mName)); mesh->surface_set_name(i, AssimpUtils::get_assimp_string(ai_mesh->mName));
} }
......
...@@ -87,6 +87,9 @@ struct ImportState { ...@@ -87,6 +87,9 @@ struct ImportState {
// this means we can detect // this means we can detect
// what bones are for other armatures // what bones are for other armatures
List<aiBone *> bone_stack; List<aiBone *> bone_stack;
// EditorSceneImporter::ImportFlags
uint32_t import_flags;
}; };
struct AssimpImageData { struct AssimpImageData {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment