यह साइट एक आधुनिक ब्राउज़र में सबसे अच्छी तरह से देखी जा सकती है जिसमें जावास्क्रिप्ट सक्षम है।

GetRepackedSkin Error with 3.7 beta

ara

I just update my spine to 3.7.32 to use the miltiple-skins new feature.
http://zh.esotericsoftware.com/forum/Multiple-skins-in-editor-ETA-10749
After exporting my spine Json file.The GetRepackedSkin function in AttachmentTools run into an error.
While using spine-unity 3.6 the error is:

Texture rectangle is out of bounds (-55 < 0)
UnityEngine.Texture2D:GetPixels(Int32, Int32, Int32, Int32)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:ToTexture(AtlasRegion, Boolean, TextureFormat, Boolean) (at Assets/Spine/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:600)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:GetRepackedSkin(Skin, String, Shader, Material&, Texture2D&, Int32, Int32, TextureFormat, Boolean, Material, Boolean, Boolean) (at Assets/Spine/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:517)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:GetRepackedSkin(Skin, String, Material, Material&, Texture2D&, Int32, Int32, TextureFormat, Boolean, Boolean) (at Assets/Spine/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:484)

Then I switch spine-unity runtime to 3.7 the error becomes:

Graphics.CopyTexture called with negative region coordinate or size (srcX 894, srcY -55, srcWidth 106, srcHeight 62, dstX 0, dstY 0)
UnityEngine.Graphics:CopyTexture(Texture, Int32, Int32, Int32, Int32, Int32, Int32, Texture, Int32, Int32, Int32, Int32)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:CopyTexture(Texture2D, Rect, Texture2D) (at Assets/Spine/Runtime/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:628)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:ToTexture(AtlasRegion, TextureFormat, Boolean) (at Assets/Spine/Runtime/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:599)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:GetRepackedSkin(Skin, String, Shader, Material&, Texture2D&, Int32, Int32, TextureFormat, Boolean, Material, Boolean, Boolean) (at Assets/Spine/Runtime/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:517)
Spine.Unity.Modules.AttachmentTools.AtlasUtilities:GetRepackedSkin(Skin, String, Material, Material&, Texture2D&, Int32, Int32, TextureFormat, Boolean, Boolean) (at Assets/Spine/Runtime/spine-unity/Modules/AttachmentTools/AttachmentTools.cs:484)

Maybe the Json output file doesn't fit both 2 version runtimes, just because the project generated with 3.6 then updated to 3.7?

Pharan

I tried your json export. It seems to work fine when used with repacking.
Do you have a combination we can test that causes this problem?

ara

Pharan लिखा
I tried your json export. It seems to work fine when used with repacking.
Do you have a combination we can test that causes this problem?

Thanks Pharan Please take a look at this.

Pharan

You had your texture's "Non Power Of Two" setting to "ToNearest".

This chopped off a part of the texture in Unity because it was nearest to 1024x512.
When it tried to get the pixels that were chopped off, it correctly says the coordinates are outside of the texture.

You need to keep the original texture size and proportions so that you don't get this error.

If you need power-of-two textures, check the "Power of Two" checkbox in the Texture Packer settings in Spine.

ara

Pharan लिखा
You had your texture's "Non Power Of Two" setting to "ToNearest".

This chopped off a part of the texture in Unity because it was nearest to 1024x512.
When it tried to get the pixels that were chopped off, it correctly says the coordinates are outside of the texture.

You need to keep the original texture size and proportions so that you don't get this error.

If you need power-of-two textures, check the "Power of Two" checkbox in the Texture Packer settings in Spine.

That's it, Thanks Pharan.

EDIT:
Well I could't find where to set Repacked Texture's size. I tried to set spine export setting to Power of Two, Square or other, and change texture setting in unity but every time the runtime repack result were the same.

Pharan

Unity's PackTextures method which this functionality relies on only receives a single maximumAtlasSize parameter.

Likewise, we have maxAtlasSize as an optional parameter for our GetRepackedSkin and GetRepackedAttachments methods to match it.

ara

Pharan लिखा
Unity's PackTextures method which this functionality relies on only receives a single maximumAtlasSize parameter.

Likewise, we have maxAtlasSize as an optional parameter for our GetRepackedSkin and GetRepackedAttachments methods to match it.

Sorry still can't make it right to avoid repack error.
I check the "Power of Two" checkbox in Spine, to keep the original proportion.
Is there anything else should be done?

Pharan

Just leave Unity's "Non Power of 2" setting to None. And make sure no other setting is changing your image's dimensions.

ara

Pharan लिखा
Just leave Unity's "Non Power of 2" setting to None. And make sure no other setting is changing your image's dimensions.

Thanks for the quick reply Pharan.That's what I did.
And now I find that the error occurs only when I have some sprites to GetRemappedClone.
After 5 times GetRemappedClone 3 sprites were repacked into new texture the other 2 were chopped off.

EDIT:
The point is that I forgot to set sprite to FULL RECT . 😃

Wecica

ara लिखा
Pharan लिखा
Just leave Unity's "Non Power of 2" setting to None. And make sure no other setting is changing your image's dimensions.

Thanks for the quick reply Pharan.That's what I did.
And now I find that the error occurs only when I have some sprites to GetRemappedClone.
After 5 times GetRemappedClone 3 sprites were repacked into new texture the other 2 were chopped off.

EDIT:
The point is that I forgot to set sprite to FULL RECT . 😃

Thank you very much for light my way in a cold dry dim winter. Nobody answers me on Christmas day....

Mario

Sorry, we have to take breaks every now and then as well 🙂

ara

Wecica लिखा
Thank you very much for light my way in a cold dry dim winter. Nobody answers me on Christmas day....

Glad to help! :grinteeth:

Wecica

badlogic लिखा
Sorry, we have to take breaks every now and then as well 🙂

Don't mind it. I understand you. Thanks for all of your efforts!