Error in AssimpImporter when loading DAE model about assimpkit HOT 8 CLOSED

@jirisamek I am working on a fix for this problem, thanks for reporting this. See PR #59. I think I may have to split the elements into smaller buffers internally, as this fix works for #56, but right now not for #58; but this is the correct fix with more implementation tweaks required for your case.

Are you OK with me using the model you have provided as part of AssimpKit test cases? The AssimpKit licenses explicitly mention the usage of such files only for testing purpose. Let me know. Thanks!

from assimpkit.

@dmsurti thank you very much for quick response. The model is not mine. I have downloaded it from https://free3d.com/3d-model/cyprys-house-23761.html, and the licence tag says: "Commercial use License". I didn't find actual text of license so I don't know if it is possible to use it in open source as model in test cases. Also this specific dae seems to have broken textures/materials.

from assimpkit.

Fixed in #59. In addition to mallocing scene buffers, it also requires ignoring the faces which are not triangulated!

This model contains material colors and/or embedded textures, which is being fixed in #62 and #64.

from assimpkit.

With #64, the house now has textures as well.

Also this specific dae seems to have broken textures/materials.

@jirisamek Actually this file has all colors for material properties, #64 ensure correct texture and colors are applied to material properties.

from assimpkit.

Hi, I've same issue with newest code in master branch .

It's throw exc_bad_access in the makeTangentGeometrySourceForNode:
SCNGeometrySource *tangentSource = [SCNGeometrySource geometrySourceWithData:[NSData dataWithBytes:scnTangents length:nVertices * 3 * sizeof(float)] semantic:SCNGeometrySourceSemanticTangent vectorCount:nVertices floatComponents:YES componentsPerVector:3 bytesPerComponent:sizeof(float) dataOffset:0 dataStride:3 * sizeof(float)];
But this crash is only happen on iOS8, if I running it in iOS11, everything work fine

from assimpkit.

I'm using a model by obj+mtl+jpg

from assimpkit.

oh, I found the reason why it would crash on iOS 8, because in , it using a constant that only available after iOS 10, it is SCNGeometrySourceSemanticTangent, here is the define:
FOUNDATION_EXTERN SCNGeometrySourceSemantic const SCNGeometrySourceSemanticTangent API_AVAILABLE(macos(10.12), ios(10.0), tvos(10.0));

from assimpkit.

@azolyoung AssimpKit minimum iOS requirement is iOS 10.0 or later, which is listed in the README.md and in the project home page.

from assimpkit.