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

We're fetching textures from SPM but they never come out right #637

Closed
carmenfan opened this issue Aug 30, 2023 · 2 comments · Fixed by #641
Closed

We're fetching textures from SPM but they never come out right #637

carmenfan opened this issue Aug 30, 2023 · 2 comments · Fixed by #641
Labels

Comments

@carmenfan
Copy link
Member

carmenfan commented Aug 30, 2023

Description

SPM API are providing us with textures which we are recording, but it never comes out correctly. We need to figure out this is an issue on our side interpreting their data, or their API is broken.

Steps to replicate

  • import an SPM with texture (example here)

Current Behaviour

  • model imports as a black box

Expected Behaviour

  • Textures should show
@carmenfan carmenfan added the bug label Aug 30, 2023
@carmenfan carmenfan self-assigned this Sep 1, 2023
@carmenfan
Copy link
Member Author

2 issues here

  1. We're not passing the UVs to the meshNode, so naturally it couldn't map the textures. This is an easy fix.
  2. The texture image is corrupted (Seems like we can't open it on image editing tools, but can view them on the web/image gallery). If manually configured through the Unity editor, we can generate the texture correctly, however through scripting it is not available and result in a red question mark.

@carmenfan
Copy link
Member Author

Waiting on another example to determine if Synchro is passing the image through incorrectly, or the image was originally corrupted

carmenfan added a commit that referenced this issue Sep 4, 2023
carmenfan added a commit that referenced this issue Sep 5, 2023
@carmenfan carmenfan removed their assignment Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant