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

Model exporter gives wrong vertex colors #411

Closed
ppartous opened this issue Apr 1, 2024 · 4 comments · Fixed by #415
Closed

Model exporter gives wrong vertex colors #411

ppartous opened this issue Apr 1, 2024 · 4 comments · Fixed by #415

Comments

@ppartous
Copy link

ppartous commented Apr 1, 2024

The mod Neo-Ishgardian_Top_of_Fending_TGT_1.2.ttmp2.zip normally looks like this

But if I export it with penumbra and then import it again without making any modifications to it, it now looks like this

If I export with textools then use blender to convert to gltf and import with penumbra it looks normal so it breaks during exporting.
Looking at the vertex colors in blender, the model exported from penumbra has the color #FF0101 on the areas that are broken

Whereas the model exported from textools has #FFC864

I'm using penumbra version 1.0.1.0

@ackwell
Copy link
Contributor

ackwell commented Apr 2, 2024

hm, thanks for the report. i'll need to check this.

@ackwell
Copy link
Contributor

ackwell commented Apr 19, 2024

Ah, I wasn't normalising the color values. Will be fixed with next batch of fixes.
image

@sprocketaudio
Copy link

Hi, has this fix been pushed to live yet? I want to test a similar issue i reported a while back on discord

https://discord.com/channels/884363610640498698/1221513268879818752

@ackwell
Copy link
Contributor

ackwell commented Apr 28, 2024

No. This issue will automatically be closed once the fix is merged.
If you want to check, you can run a dev build on this branch: master...ackwell:Penumbra:mdl-io-triage-6

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 a pull request may close this issue.

3 participants