Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autodesk: Add render-selected-edge-from-face global env var #3071

Conversation

erikaharrison-adsk
Copy link
Contributor

Description of Change(s)

To improve the performance of using Geometry Shader in HdStorm, expose an environment variable to enable/disable render-selected-edge-from-face feature, which may trigger geometry shader.

In HdStorm, when rendering mesh using HdMeshGeomStyleSurf/HdMeshGeomStyleHull geomStyle, the Geometry Shader might be enabled to support the render-selected-edge-from-face feature, this could bring performance overhead if the application doesn't need that feature, so, adding an environment variable to enable/disable render-selected-edge-from-face feature in HdStorm,

Fixes Issue(s)

  • N/A
  • I have verified that all unit tests pass with the proposed changes
  • I have submitted a signed Contributor License Agreement

Add new global env var to switch on/off render selected edges from face
@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-9644

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

pixar-oss pushed a commit that referenced this pull request Sep 25, 2024
Exposed an option in HdMeshReprDesc to enable generation of
surface edge ids for surface and hull geom styles which do not
otherwise draw edges.

This option is disabled by default.

Fixes #3071

(Internal change: 2341696)
@tcauchois
Copy link
Contributor

@erikaharrison-adsk we believe that David's commit above addresses the original issue, so we're going to close this out. Please let us know if you have further issues here!

@tcauchois tcauchois closed this Sep 26, 2024
@erikaharrison-adsk erikaharrison-adsk deleted the adsk/feature/add-global-env-render-edge-from-face branch October 11, 2024 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants