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.

Sign up for GitHub

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

Jump to bottom

--Set default fallback material handling #2265

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

--Set default fallback material handling #2265

wants to merge 3 commits into from
+54 −4

Conversation

jturner65
Copy link
Contributor

Motivation and Context

This PR provides some extensions to the fall back material handling.

  • Make the fallback material magenta color
  • Add a boolean to drawables to denote they are being shaded via the fallback color. This will signify that either the materials were not loaded or they do not exist for the source asset
  • Add a reporting tool to provide information to the user about all assets/objects that ended up being configured to consume the fallback material instead of an appropriate asset-related material.

Still a WIP

How Has This Been Tested

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Nov 14, 2023
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 2ac3229 to 41b6fcc Compare November 29, 2023 14:33
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 4 times, most recently from 579d931 to 4f114ba Compare December 12, 2023 17:24
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 5a87217 to 419dd20 Compare January 4, 2024 13:52
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from dfe520d to 92ac4da Compare January 9, 2024 21:12
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 38996c7 to 15dbe47 Compare January 29, 2024 19:22
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 3 times, most recently from df87790 to a25b8c6 Compare February 7, 2024 16:00
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 1a31f92 to 169946d Compare February 15, 2024 14:08
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 8f74add to d06df58 Compare February 26, 2024 17:29
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 4 times, most recently from b2d5dc8 to 37554a4 Compare March 5, 2024 19:04
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 4 times, most recently from 3e348b4 to f545426 Compare March 20, 2024 16:15
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 2 times, most recently from 1e3d4dc to 03866b8 Compare April 22, 2024 13:33
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 4 times, most recently from 0929bdf to bdc9439 Compare May 2, 2024 13:20
@jturner65 jturner65 force-pushed the DfltFallbackMaterials branch 3 times, most recently from e5456b9 to 725338e Compare May 9, 2024 13:43
@jturner65
--initial commit; Make fallback material noticeable
98c8f70 
--make material color magenta
--add flag that this material is a fallback for an object lacking any other materials
--remove default phong material function; specifying phong shader to use should be sufficient for depth sensor.
@jturner65
--set flag when a drawable is built using fallback material
bb99513 
--this could indicate that the source asset lacks a material, or else that no materials were loaded by design (i.e. for depth sensor).
@jturner65
--fallback should be flat
6fe5da6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jturner65 @facebook-github-bot