Support for 64x64 DLC skins

[Langtanium]

Description

Adds code to support DLC 64x64 classic and slim skins. Also adds the remaining default skins.
https://www.youtube.com/watch?v=lX_AaVbohZE

Changes

1. Added code to handle 64x64 classic and slim skins.
2. Added code to handle skin boxes connected to second layer or armor layer of skin.
3. Added missing default skins.

Previous Behavior

Skins and skin packs that were or had 64x64 dimension textures would not render properly or cause the game to crash. Also skin boxes would not be mirrored if the skin box had the mirror bit flag was set to true, were not scaled proper if the skin box had a scale value present, and wouldn't be hidden when wearing a helmet is the "hide when helmet is worn" bit flag was set to true.

Root Cause

Lack of 64x64 DLC skin support.

New Behavior

Skins and skin packs that are or have 64x64 dimension textures will render properly and mostly won't cause the game to crash. Skin boxes are properly mirrored when the mirror bit flag is present. Skin boxes with a scale value present will now be scaled properly Skin boxes with the "hide when helmet is worn" bit flag will not be rendered while wearing a helmet. Skin boxes attached to the second layer or armor layer of skin no longer cause the game to crash. Added code to handle DLC skin offset.

Fix Implementation

References to the added default skins texture were added to the files: Textures.cpp, Textures.h, Definitions.h, UIScene_SkinSelectMenu.cpp, and Player.cpp. In SkinBox.h more enums and floats were added. In ModelPart.cpp and ModelPart.h a boolean for that stores a bit flag, responsible for hiding a skin box when a helmet is worn, was added. In HumanoidModel.cpp and HumanoidModel.h code was added to handle whether the skin is 64x32 or 64x64, a Boolean was added to determine if it is slim or classic, and code was added to handle the skin box mirror bit flag and scale of skin box if attached to the second layer (jacket, sleeve0, sleeve1, pants0, pants1). In TextureAndGeometryPacket.cpp added code to handle the extra skin boxs' floats. A check was added to UIControl_PlayerSkinPreview.cpp and XUI_Ctrl_MinecraftSkinPreview.cpp to make the Alex skins use the 64x64 slim model, make the Developer Steve skin use the 64x64 classic model and any DLC skin use it declare model variant. A classic and slim model options were added to EntityRenderer.cpp and EntityRenderer.h, which will be empty if a mob is spawned instead of a player. PlayerRenderer.cpp and PlayerRenderer.h contain similar functionality as UIControl_PlayerSkinPreview.cpp, but for the first-person hand and similar features and uses the DLC skin files bit mask to determine whether to use the default model or the 64x64 variants. LivingEntityRenderer.cpp contains new functionality for rendering a player model in a third-person or second-person game and include the same functionality of using the DLC skin bit mask to determine the model type. The functionality works as follows: PlayerRenderer.cpp passes a parameter to LivingEntityRenderer.cpp which creates and renders the regular, classic and slim models, which are saved in the EntityRenderer.cpp and EntityRenderer.h scripts, these models are also used in the skin menu display script (UIControl_PlayerSkinPreview.cpp). Code was added to Console_App.cpp to make DLC skin boxes render on the 64x64 classic and slim models. Changed code in ClientConnection.cpp and PlayerConnection.cpp to always send DLC metadata.
An extra enum was added to DLCManager.cpp and DLCManager.h for DLC skin offsets. SkinOffset.h was created to store the DLC skin offset data in a class, like SkinBox.h, which includes body part, direction, and amount to move the part. Code was added to DLCSkinFile.cpp and DLCSkinFile.h to store the extra data values of skin boxes and store DLC skin offset data in a vector of SKIN_OFFSETs.

AI Use Disclosure

No AI was used in the writing of this code.
Some code was taken from ItzSonicFaner's pull request (#1183), but has more development.
Thanks to @ZomkaDEV for implementing code to fix DLC skin metadata not being sent in multiplayer.

Related Issues

• Related to [Suggestion] Add 64x64 skin compatibility #1099

Issues

1. Switching to or between 64x64 DLC skins in game can sometimes cause the game to crash during multiplayer. (Both instances were running on the same computer; The instances that had the skin change crashed, the other one continued to run fine)
2. The hat layer for 64x64 skins has its UV for the bottom face flipped, also applies to skin boxes attached to that layer. The hat layer for 64x32 skins renders correctly.
3. Other players in multiplayer don't render (need to fix).

[sylvessa]

will test when you're ready, just mark as ready for review when you want me to test

[sylvessa]

decided to try this out, it seems to work almost fully, the only issue is that capes on 64x64 skins do not work properly

with 64x64 skin patch:

without 64x64 skin patch:

normal 64x32 skins with capes still work fine:

[Langtanium]

@sylvessa I fixed the cape not rendering properly with 64x64 skins.

I want to implement skin offsets, used in the Mario skin pack, before marking this as ready for review.
Also, I believe the code for 64x64 skins could be better written and I'm exploring how to improve/optimize it.

[sylvessa]

ok! take ur time 👍

[ZomkaDEV]

Hey, I found an issue. If a player has a 64x64 slim skin, it's still treated as a 64x32 skin by other clients in multiplayer. The opposite happens with 64x64 classic skins though.

[Langtanium]

@ZomkaDEV I have looked into the issue you have reported, but cannot recreate the issue you have.

[ZomkaDEV]

There are still issues :/
Here are screenshots showing what I'm talking about.

64x32 skins don't render properly on the player's side but render fine for other players.

The opposite happens for 64x64 skins.

All three are DLC skins, I can send you the PCK if you want.

[Langtanium]

@ZomkaDEV send me your PCK to test because I also have my own custom PCK, but I find no problems with mine.

[ZomkaDEV]

@ZomkaDEV send me your PCK to test because I also have my own custom PCK, but I find no problems with mine.

Here you go. Perhaps the issue is with how I made my pck...

[TROYNELSON12]

@ZomkaDEV send me your PCK to test because I also have my own custom PCK, but I find no problems with mine.

Here you go. Perhaps the issue is with how I made my pck...

One issue is with your pck, it incorrectly has the 64x64 Classic flag on the Jeb skin while the texture is 64x32.

[ZomkaDEV]

@ZomkaDEV send me your PCK to test because I also have my own custom PCK, but I find no problems with mine.

Here you go. Perhaps the issue is with how I made my pck...

One issue is with your pck, it incorrectly has the 64x64 Classic flag on the Jeb skin while the texture is 64x32.

Oh, well, I didn't know that. I removed the ANIM flag and the jeb (64x32) skin is now rendering properly on both sides. Still experiencing issues with 64x64 skins though...

[ZomkaDEV]

Okay, I've done some testing. I was able to find that if both of the clients have the PCK file with the 64x64 skin, it is rendered correctly. If they don't, it is treated as a 64x32 skin.

[ZomkaDEV]

Aha, I figured out why! If the skin has no extra geometry, only the texture is being sent without all the other metadata like the ANIM value :)

I fixed it in my fork, feel free to implement the same change.
ZomkaDEV@de49012

[codeHusky]

Is there anything major holding this back from merging?

[Langtanium]

@codeHusky I want to implement the full functionality of DLC skins before marking this as ready. I'm working on the last feature I want to add of skin offsets used in the Toy Story skin pack and Mario skin pack. In a separate branch, I have added code to store the DLC skins' offsets, the code for the offsets modifies a significant amount of files which could be considered major.
https://github.com/Langtanium/MinecraftConsoles-Langtanium/tree/feat/skin-offsets

[Langtanium]

I got skin offsets working.
Before:

After:

But now other players are not rendering, need to fix that.