PixarAnimationStudios / OpenUSD

Universal Scene Description
http://www.openusd.org
Other
5.57k stars 1.16k forks source link

Capsule_1 rendering breaks when top/bottom radii are different #3113

Open crydalch opened 1 month ago

crydalch commented 1 month ago

Description of Issue

The new Capsule_1 primitive seems to have a issues drawing correctly. Compared to adjusting radiusTop and radiusBottom with Cylinder_1, the new cylinder prim has unexpected results that look buggy.

Steps to Reproduce

  1. Download attachment from Bug_Capsule_1.usda.zip , and open in usdview

  2. With radiusBottom of 1.0 and radiusTop of 0.3 on both primitives, it seems reasonable to expect the capsule to resemble the cylinder, except for the rounded ends. But it looks unexpectedly different, maybe broken: image

  3. Open up an Interpeter window, to use for next steps

  4. In usdview, select primitive1, and then select its radiusBottom property, and in the interpreter window, run usdviewApi.property.Set(0.3)

  5. Then select its radiusTop property, and in the interpreter window, run usdviewApi.property.Set(1.0)

  6. This looks entirely broken, like there are assumptions about the values/order of radiusTop and radiusBottom: image

Package Versions

USD 24.05

spiffmon commented 1 month ago

Thanks, @crydalch - do you get the same results with Hydra 2 as with default Hydra 1?

crydalch commented 1 month ago

Hi @spiffmon!

Yes, it appears the same with USDIMAGINGGL_ENGINE_ENABLE_SCENE_INDEX=1

jesschimein commented 1 month ago

Filed as internal issue #USD-9737

marktucker commented 3 weeks ago

Is this something that is already being worked on by Pixar? I was thinking of tackling this, but didn't want to duplicate any work that may already be ongoing.

marktucker commented 3 weeks ago

This was easier than I thought... PR incoming soon. I'll just cross my fingers that there hasn't been any duplicated effort.

pixar-oss commented 3 weeks ago

Thank you @marktucker! We had not started on this yet, so the PR is very welcome!