xjorma / EarthMeshHoudini

Generate real world meshes in Houdini using the Google API.
Apache License 2.0
207 stars 21 forks source link

KeyError: 'KHR_draco_mesh_compression' #19

Open Piminski opened 1 month ago

Piminski commented 1 month ago

I am getting this error when increasing the min dist and max mesh.

The same coordinates worked with 500 min dist and 5000 max mesh. When I increased to 1500 min dist and 15000 max mesh I get this error.

Error Invalid source /obj/geo1/EarthMesh1/Glb. (Error: Python error: Traceback (most recent call last): File "", line 66, in KeyError: 'KHR_draco_mesh_compression' ).

I am using @joreeves implementation

xjorma commented 1 month ago

I might have sometime today. Could you take give me the exact parameters to get the bug? If I am not able to repro it I won’t be able to fix it.

On Wed, Jul 24, 2024 at 1:05 PM Piminski @.***> wrote:

I am getting this error when increasing the min dist and max mesh.

The same coordinates worked with 500 min dist and 5000 max mesh. When I increased to 1500 min dist and 15000 max mesh I get this error.

Error Invalid source /obj/geo1/EarthMesh1/Glb. (Error: Python error: Traceback (most recent call last): File "", line 66, in KeyError: 'KHR_draco_mesh_compression' ).

I am using @joreeves https://github.com/joreeves implementation

— Reply to this email directly, view it on GitHub https://github.com/xjorma/EarthMeshHoudini/issues/19, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFPHGHQ2YP43G7K6KC2ICUTZN7NEDAVCNFSM6AAAAABLM4XF4SVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQZDQMBXGQ4DINQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Piminski commented 1 month ago

Here are the setting that create the error:

lat: 34.067054 lon: -118.389711

min error:2 mar error:100000 mindist: 1500 maxdist:2e+06

max mesh: 15000

xjorma commented 1 month ago

Awesome I will check that very soon.

On Wed, Jul 24, 2024 at 1:27 PM Piminski @.***> wrote:

Here are the setting that create the error:

lat: 34.067054 lon: -118.389711

min error:2 mar error:100000 mindist: 1500 maxdist:2e+06

max mesh: 15000

— Reply to this email directly, view it on GitHub https://github.com/xjorma/EarthMeshHoudini/issues/19#issuecomment-2248544493, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFPHGHUKIPWHRMANG6WJKLLZN7PWPAVCNFSM6AAAAABLM4XF4SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGU2DINBZGM . You are receiving this because you commented.Message ID: @.***>

Piminski commented 1 month ago

I think maybe I am pushing the process a bit too hard :). It may be some memory or buffer issues.

xjorma commented 1 month ago

No, I found the issue. The problem is a very small mesh, likely just 2 triangles, that doesn't use Draco compression. The code assumes everything uses Draco. The larger the zone, the greater the chance of encountering a faulty mesh. I will probably submit a temporary fix soon. I am testing it, but your scene takes a long time to process.

Piminski commented 1 month ago

Oh I see.

I understand its a long process, thank you for looking into it.

xjorma commented 1 month ago

Temp fix seems to work, It's a long process but houdini handle the charge pretty well. I will submit a fix very soon. But on my version, I will take a look on the @joreeves pull request just after. image

xjorma commented 1 month ago

@Piminski let me know if it works for you.

Piminski commented 1 month ago

OK, thank you! thats awesome. Will take me some time (a couple hours) until I can check on my end.

Piminski commented 1 month ago

@xjorma , I tested your updated HDA in Houdini 20.5. The geometry loads but for some reason I cannot see the texture, not even the mesh (only visible as wireframe and points). I changed the display settings as per the readme but there is some issue. I also tested importing the default settings on the HDA and it had the same issue. This is the first time I use Earthmesh in 20.5 so it may be some new setting I need to change...

I will now check in 20.0.

This screenshot shows the default (your base settings) mesh as it appears with "Flat wire shaded" viewport:

Screenshot 2024-07-25 142657

xjorma commented 1 month ago

Is never tested with 20.5 can upgrade and test.

On Thu, Jul 25, 2024 at 5:25 PM Piminski @.***> wrote:

@xjorma https://github.com/xjorma , I tested your updated HDA in Houdini 20.5. The geometry loads but for some reason I cannot see the texture, not even the mesh (only visible as wireframe and points). I changed the display settings as per the readme but there is some issue. I also tested importing the default settings on the HDA and it had the same issue. This is the first time I use Earthmesh in 20.5 so it may be some new setting I need to change...

I will now check in 20.0.

— Reply to this email directly, view it on GitHub https://github.com/xjorma/EarthMeshHoudini/issues/19#issuecomment-2251422116, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFPHGHQHK43IZ6YW2C5752TZOFUMTAVCNFSM6AAAAABLM4XF4SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJRGQZDEMJRGY . You are receiving this because you were mentioned.Message ID: @.***>