googlearchive / poly-toolkit-unity

Poly Toolkit for Unity
Apache License 2.0
101 stars 29 forks source link

Issue: Some Models Not Importing at Runtime #23

Open benhylak opened 5 years ago

benhylak commented 5 years ago

First off -- I have the newest release, which incorporates the fix from #22. Although this resolved my import problems with some assets, problems remain.

I am again experiencing problems on just a single asset two assets that I've found so far.

Those assets are:

(I'll continue to update this list as I find more)

It imports just fine in the editor, but my application needs assets to be imported at runtime, based on user input.

Unity version: 2018.2.16f1

Running in editor on macOS mojave.

Most models import okay. This one works, for example: https://poly.google.com/view/cOizz1RJnb3

benhylak commented 5 years ago

I recently found the issue with the Policecar asset. While the puppy asset fails silently, the policecar produces the following error:

Attribute TEXCOORD_0 is missing data: wanted range [0, 10362), count was 3582
UnityEngine.Debug:LogWarningFormat(String, Object[])
PolyToolkitInternal.ImportGltf:CreateMeshPrecursorsFromPrimitive(ImportState, GltfPrimitiveBase) (at Assets/PolyToolkit/Internal/ImportGltf.cs:765)
PolyToolkitInternal.ImportGltf:CreateMeshPrecursorsFromMesh(ImportState, GltfMeshBase) (at Assets/PolyToolkit/Internal/ImportGltf.cs:346)
PolyToolkitInternal.ImportGltf:CreateMeshPrecursorsFromNode(ImportState, GltfNodeBase) (at Assets/PolyToolkit/Internal/ImportGltf.cs:335)
PolyToolkitInternal.ImportGltf:CreateMeshPrecursorsFromScene(ImportState) (at Assets/PolyToolkit/Internal/ImportGltf.cs:329)
PolyToolkitInternal.ImportGltf:BeginImport(GltfSchemaVersion, TextReader, IUriLoader, PolyImportOptions) (at Assets/PolyToolkit/Internal/ImportGltf.cs:244)
PolyToolkitInternal.AsyncImporter:BackgroundThreadProc(Object) (at Assets/PolyToolkit/Internal/AsyncImporter.cs:82)
System.Threading._ThreadPoolWaitCallback:PerformWaitCallback()
benhylak commented 5 years ago

is this still an active project?

anokta commented 5 years ago

Thanks for the report! We're currently investigating the issue.

znebby commented 5 years ago

@bhylak These two assets import just fine for me. I do get the same warning for puppy, but it does import fine.

Unity 2018.3.4 and Poly Toolkit 1.1.2