irajsb / UE4_Assimp

Assimp mesh importer for Unreal Engine
MIT License
136 stars 29 forks source link

Bug with large files #19

Closed N1q closed 2 years ago

N1q commented 2 years ago

Large files not loading correctly For some reason, some of the objects lose their position and move to the zero position. Also, displaying an imported model uses a lot of resources, which does not happen if the model is simply imported into the unreal engine outside of runtime mode.

To Reproduce Simply try to open .fbx or .gltf (didn't tried any other formats) file with lots of small elements. Used latest release: https://github.com/irajsb/UE4_Assimp/releases/tag/5.4.2CustomFlagsSupport

Screenshots изображение

irajsb commented 2 years ago

Can you provide the sample model for debugging ?

N1q commented 2 years ago

Oh, sorry. Forgot to attach UE4_assimp_big_test_file.zip

irajsb commented 2 years ago

For performance issues : procedural mesh components are inefficient and must not be overused unless its necessary .

The reason that when importing using engine is more efficient is because engine takes about 2 mins to create static mesh data for meshes in this example file's case. Mesh importer only passes vertices and data needed to generate meshes. This process is not related to mesh importer but engine and mesh components .

I recommend trying Runtime Mesh Component which is more efficient .

I'm working on transforms issue at the moment will have to find a solution that produces similar result for all formats

irajsb commented 2 years ago

Update: After a few hours I managed to create an static mesh based solution . an update will be released soon it runs at constant 60fps

irajsb commented 2 years ago

https://github.com/irajsb/UE4_Assimp/releases/tag/StaticMeshSupport Transforms work and static meshes are efficient . I've noticed a bug where only one type of mesh goes missing but that will be a different issue . You can bypass this issue for now by adding Pre transform vertices flag in import function .

N1q commented 2 years ago

Thanks for the quick response! The model is perfectly imported using pre transform flag. But I need each object to be imported separately, so I'm patiently waiting for this fix as well. Should I open another issue for it?

irajsb commented 2 years ago

I'll try again once I find some free time to debug this issue

irajsb commented 2 years ago

https://github.com/irajsb/UE4_Assimp/releases/tag/MeshInstancing