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-105968] Nvidia attic stage - super slow to load #460

Closed
Xaia opened this issue Apr 24, 2020 · 6 comments · Fixed by #1043
Closed

[MAYA-105968] Nvidia attic stage - super slow to load #460

Xaia opened this issue Apr 24, 2020 · 6 comments · Fixed by #1043
Assignees
Labels
bug Something isn't working reproduced Issue has been reproduced

Comments

@Xaia
Copy link

Xaia commented Apr 24, 2020

Describe the bug
I takes over 30 seconds to load simple nvidia attic usd stage.

Expected behavior
Quick load

Attachments
GEknl2Hqh3

Specs (if applicable):

  • Windows 10
  • Maya PR114
@Xaia Xaia added the bug Something isn't working label Apr 24, 2020
@Xaia Xaia changed the title Nvidia attic - super slow to load Nvidia attic stage - super slow to load Apr 24, 2020
@kxl-adsk
Copy link

@Xaia Can you measure in USD view and compare? The pure load of USD assets depends on USD library. When reporting performance issues like this one, please compare first.

@Xaia
Copy link
Author

Xaia commented Apr 24, 2020

@kxl-adsk Well, it is embarrassing but I have not clue how to run usdview from MayaPR114 build.
I now that clarisse and houdini opens the file in 1 second but does not load any textures. Pretty sure this is what makes the scene load so long in Maya.

It cannot be that once you have some shaders and textures in, scene loads forever. Checkbox for loading textures only on demand would be nice.
Cheers

@santosd
Copy link
Collaborator

santosd commented Jul 28, 2020

Hi @Xaia and @kxl-adsk, I took a look at this and found that loading in USD view and in Maya take the same amount of time about a minute for the Attic File for me. I think like you said Xaia this is related to textures. Maybe something we could look into speeding up. I noticed if I turn on the use default material option the scene will come in and show the grey material but loading is still slow. Maybe if this option is enabled we can wait to load the textures.

@santosd santosd added the reproduced Issue has been reproduced label Jul 28, 2020
@Xaia
Copy link
Author

Xaia commented Jul 28, 2020

Pixar is already on to something. Not sure how this translates to Maya plugin though.
https://groups.google.com/d/msg/usd-interest/8Ml3CTrwcsc/d0j54SapAAAJ

@kxl-adsk kxl-adsk changed the title Nvidia attic stage - super slow to load [MAYA-105968] Nvidia attic stage - super slow to load Nov 6, 2020
@huidong-chen huidong-chen linked a pull request Nov 30, 2020 that will close this issue
@huidong-chen
Copy link

huidong-chen commented Dec 10, 2020

Two notes for this issue:

  • VP2RenderDelegate stills takes longer than HdSt when loading the scene (70s vs. 30s). The material networks are not exactly duplicate because of different node names, but it looks like most of them have the same node types and connections. I think we will need a further optimization in VP2RenderDelegate to reuse the same shader for this type of similar material networks.
  • The scene also has quite an amount of textures but I guess the texture loading performance will be most likely similar between VP2RenderDelegate and HdSt. I will profile and confirm after the above bottleneck is fixed.

@huidong-chen
Copy link

Just to post my recent progress on this issue:

As I mentioned above VP2RenderDelegate originally took 70s to load the NV attic scene, which presented two major overheads:

  • 54s on texture loading
  • 16s on material realization

With #1043 the material realization overhead (16s) is gone, so it takes 54s now.

It is still slower than HdSt (30s). I was expecting VP2RenderDelegate texture loading performance could be equivalent to HdSt because we share the same code path to read images from disk, so we will need to understand the reason of the gap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reproduced Issue has been reproduced
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants