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

IR-2726 Material Component Breakup #10406

Merged
merged 83 commits into from
Jun 25, 2024
Merged

IR-2726 Material Component Breakup #10406

merged 83 commits into from
Jun 25, 2024

Conversation

AidanCaruso
Copy link
Member

@AidanCaruso AidanCaruso commented Jun 18, 2024

Summary

Breaks up the material component array into explicitly defined components

Subtasks Checklist

Breaking Changes

References

closes https://tsu.atlassian.net/browse/IR-2726

QA Steps

Switch to the material-component-breakup branch on the advanced materials repo, see also
https://github.com/theinfinitereality/eepro-advanced-materials/pull/35

@AidanCaruso AidanCaruso marked this pull request as draft June 18, 2024 15:26
@AidanCaruso AidanCaruso marked this pull request as ready for review June 22, 2024 17:24
Copy link
Member

@dinomut1 dinomut1 left a comment

Choose a reason for hiding this comment

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

Screencast from 06-24-2024 10:24:31 AM.webm
This video shows two bugs I'm seeing.

  1. Material prototype selection doesn't differentiate between materials which are extensions of a certain material class. In this video, I'm trying to switch between the Lava material and the Standard material. The Lava material is an extension of the Standard material, and so the UI doesn't recognize that they're different, and the prototype selection is a no-op.
  • This can be fixed by using the userData.type field "type":
    material-types
  1. Changing a material prototype only modifies one instance of the material
  • looks like materials are not being shared across entities. When the prototype is changed from Lava to Basic, only one of the lava materials in the scene actually changes -- the other remains as a lava material.

Copy link
Member

@dinomut1 dinomut1 left a comment

Choose a reason for hiding this comment

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

I'm gonna go ahead and approve this since the bugs mentioned above don't specifically pertain to the splitting up of the Material Component. However let's get those resolved in a subsequent PR ASAP

@HexaField HexaField merged commit 7711e85 into dev Jun 25, 2024
28 checks passed
@HexaField HexaField deleted the material-component-breakup branch June 25, 2024 00:12
MbfloydIR added a commit that referenced this pull request Jun 25, 2024
* dev:
  Optimize non 3d webpages from rendering canvas (#10421)
  case insensitive full string search for file browser (#10431)
  [IR-2091] studio: update ContextMenu (#10434)
  Exported EditorRouter for MT
  chore: Update project removal logic to ignore resources JSON (#10446)
  fix File container styling (#10435)
  IR-2726 Material Component Breakup (#10406)
  Update onSet to correctly read colors (#10443)
  Fix hookstate bugs (101, 102) in SelectInput (#10442)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants