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

Look first assignment ordering #30

Closed
wants to merge 6 commits into from
Closed

Conversation

kwokcb
Copy link
Owner

@kwokcb kwokcb commented Oct 20, 2022

Look vs material first assignments

This needs a re-work to be look first vs material first.
Just leaving this as a WIP branch for others if interested.

Refined test:

<?xml version="1.0"?>
<materialx version="1.38">

  <standard_surface name="SR_red" type="surfaceshader">
    <input name="base_color" type="color3" value="1, 0, 0" />
  </standard_surface>
  <surfacematerial name="Red" type="material">
    <input name="surfaceshader" type="surfaceshader" nodename="SR_red" />
  </surfacematerial>

  <standard_surface name="SR_blue" type="surfaceshader">
    <input name="base_color" type="color3" value="0, 0, 1" />
  </standard_surface>
  <surfacematerial name="Blue" type="material">
    <input name="surfaceshader" type="surfaceshader" nodename="SR_blue" />
  </surfacematerial>

  <look name="Look">
    <materialassign name="Red_P" geom="/Preview_Mesh" material="Red" />
    <materialassign name="Blue_P" geom="/Preview_Mesh" material="Blue" />
    <materialassign name="Blue_C" geom="/Calibration_Mesh" material="Blue" />
    <materialassign name="Red_C" geom="/Calibration_Mesh" material="Red" />
  </look>

</materialx>

Desktop

assignment_order_desktop

Web

assignment_order_web

@kwokcb kwokcb changed the title Parent child + ordered assign Look first assignment ordering Oct 20, 2022
@kwokcb kwokcb closed this Oct 23, 2022
@kwokcb kwokcb deleted the parent_child_oredered_assign branch October 23, 2022 18:28
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.

1 participant