-
-
Notifications
You must be signed in to change notification settings - Fork 153
Fix warning message for transparent slices #35
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
Fix warning message for transparent slices #35
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #35 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 20 20
Lines 840 840
=========================================
Hits 840 840
☔ View full report in Codecov by Sentry. |
|
Hey, Thanks for the PR! Can you please specify Unity version which generates the warnings (or repro steps)? I'm using 2019.4 and never seen them. |
|
Great question! Yes, it also happens on Unity 2019, and above. I've included a Scene with the transparent image that existed before from another PR, and a newly generated one. You can toggle the GameObjects and you'll see the logs showing this warning. Nothing else happens other than the warning, but it gets in the way of legitimate logs.
|
|
I've checked your scene and pre-generated asset indeed produce warnings, but after re-generating it in 2019.4 (w/o the proposed changes) there are no warnings. I assume you've generated it with a later Unity version? Can you please specify which exactly? |
|
Actually, it's because you've regenerated in this branch, which contains the fix. If you regenerate in master, you'll see the warning message again. |
|
Weird, I don't see these artifacts in the image, but I did see something like that when I was coding it initially and I messed up the vertices, uvs, and triangles. I don't remember the combination, though. I'll try to reproduce that. |
My bad, when testing I didn't comment out the |
|
Ah, cool. Do you want me to remove the extra scene and all the stuff I added in the last commit? I can regenerate the Flat one that existed before. |
|
It's ok I'll brush it up myself and merge the PR. Thanks again for the contribution! |
Transparent sprites, when sliced generate this very annoying message from Unity. This PR attempts to fix them by setting the missing metadata.