Skip to content

Conversation

@Levi-Armstrong
Copy link
Contributor

These are taken from Ignition repository.

Remaining Tasks:

  • Port unit tests
  • Port PBR material
  • Finish out material class

@Levi-Armstrong Levi-Armstrong force-pushed the feat/AddColorImageAndMateriaClass branch from 415127d to 8c99c88 Compare November 19, 2022 23:04
@Levi-Armstrong Levi-Armstrong force-pushed the feat/AddColorImageAndMateriaClass branch from 8c99c88 to 11cd2e6 Compare November 29, 2022 15:48
@Levi-Armstrong Levi-Armstrong force-pushed the feat/AddColorImageAndMateriaClass branch from 11cd2e6 to d16d103 Compare November 29, 2022 15:53
@Levi-Armstrong
Copy link
Contributor Author

@johnwason Do you think the color and material structures are useful to get integrated?

@johnwason
Copy link
Contributor

We already have a lot of these capabilities implemented. What do you think is missing?

@johnwason
Copy link
Contributor

If we are going to add rendering camera sensors as discussed in #1164 we may want to revisit this. I didn't think it was necessary for the viewer.

@Levi-Armstrong
Copy link
Contributor Author

I agree.

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.

2 participants