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

arnold-usd does not respect all types of visibility #210

Closed
kreppene opened this issue Feb 11, 2020 · 0 comments
Closed

arnold-usd does not respect all types of visibility #210

kreppene opened this issue Feb 11, 2020 · 0 comments
Labels
bug Something isn't working procedural Related to the Arnold Procedural user Reported by a user
Projects

Comments

@kreppene
Copy link
Contributor

Describe the bug
it looks like not all kinds of visibility is supported in arnold-usd
i run this check before create any arnold geo in my dev branch, and that seams to do the what we want (stolen from the Rodeofx project)

bool getPrimVisibility(const UsdPrim& prim, float frame)
{
	// We should also consider standard USD visibility flag.
    UsdGeomImageable imageable = UsdGeomImageable(prim);
    bool invisibleFromUSD = imageable &&
        imageable.ComputeVisibility(frame) == UsdGeomTokens->invisible;

    return invisibleFromUSD;
}

To Reproduce
render the .ass file provided

Expected behavior
We expect only 1 sphere to be visible, but 2 is visible when rendering,
theirs a total of 3 spheres inn the scene,
so it looks like it respect some kind visibility but not all

Screenshots
visibility

Used Software Versions

  • Arnold 6.0.1.0
  • USD 0.19.11
  • visual studio 2017
  • Windows 10

visibility_test.zip

@kreppene kreppene added the bug Something isn't working label Feb 11, 2020
@sirpalee sirpalee added this to Needs triage in Ticket Triage via automation Feb 12, 2020
@sirpalee sirpalee moved this from Needs triage to Reproed in Ticket Triage Feb 12, 2020
@sirpalee sirpalee moved this from Reproed to Needs triage in Ticket Triage Feb 12, 2020
@sirpalee sirpalee added the procedural Related to the Arnold Procedural label Feb 12, 2020
@sirpalee sirpalee moved this from Needs triage to High priority in Ticket Triage Feb 12, 2020
@compso compso removed this from High priority in Ticket Triage Feb 21, 2020
@compso compso added this to To do in Sprint 19 via automation Feb 21, 2020
Sprint 19 automation moved this from To do to Done Mar 6, 2020
@sirpalee sirpalee added the user Reported by a user label May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working procedural Related to the Arnold Procedural user Reported by a user
Projects
No open projects
Sprint 19
  
Done
Development

No branches or pull requests

2 participants