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

MAYA-106937 Prototype support for GPU Normals and OSD using glsl. #966

Merged

Conversation

williamkrick
Copy link
Contributor

The code is disabled by default.

The GPU evaluation prototype is two separate parts, the normal calculation code and the OSD
code.

The normal calculation code is enabled by setting HDVP2_USE_GPU_NORMAL_COMPUTATION=1 at runtime.
The normal calculation code is close to being stable enough for general use, but hasn't had
enough polish to enable by default.

The OSD code requires the normal calculation code to be enabled to use. OSD is enabled
by compiling with HDVP2_ENABLE_GPU_OSD. The OSD code is much less stable then the normals
calculation code and comes with a number of huge.

OSD Limitations:
* No OSD adaptive support
* scenes with animation behave poorly
* selection in the viewport is very slow
* toggling VP2 consolidation world off and on will cause crashes
* some objects draw with incorrect indexing

@williamkrick williamkrick added the ready-for-merge Development process is finished, PR is ready for merge label Dec 7, 2020
@kxl-adsk kxl-adsk added do-not-merge-yet Development is not finished, PR not ready for merge pending beta This issue or PR will be testable in next beta release and removed ready-for-merge Development process is finished, PR is ready for merge labels Dec 7, 2020
@kxl-adsk kxl-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed do-not-merge-yet Development is not finished, PR not ready for merge pending beta This issue or PR will be testable in next beta release ready-for-merge Development process is finished, PR is ready for merge labels Dec 15, 2020
Copy link

@kxl-adsk kxl-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change doesn't compile with USD 20.02. I'm getting the following error:

maya-usd_pr\lib\mayaUsd\render\vp2RenderDelegate\meshViewportCompute.cpp(250): error C2065: 'PxOsdOpenSubdivTokens': undeclared identifier

@kxl-adsk kxl-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Dec 18, 2020
@kxl-adsk kxl-adsk merged commit 75d7cdd into dev Dec 18, 2020
@kxl-adsk kxl-adsk deleted the krickw/MAYA-106937/prototype_compute_normals_and_OSD_on_the_gpu branch December 18, 2020 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge vp2renderdelegate Related to VP2RenderDelegate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants