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

Armor trim incompatibility #5303

Closed
UWUsraka132 opened this issue Jan 11, 2025 · 4 comments
Closed

Armor trim incompatibility #5303

UWUsraka132 opened this issue Jan 11, 2025 · 4 comments
Labels
1.20 Issue affects 1.20 Bug Issue describes unintended or broken behavior Mod interaction Issue only happens when another mod is loaded Needs Crosspost Issue needs to also be reported to the other tracker or the issue needs to be linked here Other mod Issue is caused by another mod, and it is best fixed in the other mod Resolved Issue is fixed in code, but there is not a release with that fix yet.

Comments

@UWUsraka132
Copy link

Minecraft Version

1.20.1

Modloader

Forge

Modloader Version

47.3.0

Mantle Version

1.11.28

Tinkers' Construct Version

3.9.0.6

Describe your issue

Tinkers' armors are incompatible with some armor trims and trim materials.

When applying a Heart trim from Geode+ using any trim material other than vanilla and Geode+, the 3D model of the armor breaks.
And the other way around, vanilla trims with Geode+ materials show a bugged 3D model.
The item models are also broken in all cases.

image

Crash Report

No response

Other mods

Geode+ version 1.2.5
Alex's Caves version 2.0.2
L_Ender's Cataclysm version 2.35
AllTheModium version 47.1.25-2.5.1
Biomes O' Plenty version 18.0.0.592

Tried reproducing with just Tinkers?

No

Performance Enchancers

Rubidium/Embeddium

Searched for known issues?

Searched open issues, Searched closed issues, Checked the FAQ

@UWUsraka132 UWUsraka132 added 1.20 Issue affects 1.20 Bug Issue describes unintended or broken behavior Unreviewed Issue is new and is awaiting the team to review it labels Jan 11, 2025
@KnightMiner
Copy link
Member

KnightMiner commented Jan 11, 2025

That's not a tinkers issue, that's just a trim issue in general. You will see installing a geode+ trim with a non geode+ material will break on vanilla armor too. Also just checked their mod, they don't even try to make their trim work, I see nothing making textures for their trim on items not vanilla trims.

Try using the mod trimmed, it makes trim work more. And tell geode + to adds trimmed support. That will fix this

@KnightMiner KnightMiner added Not enough information Insufficient information is provided in the issue to accurately respond Mod interaction Issue only happens when another mod is loaded Other mod Issue is caused by another mod, and it is best fixed in the other mod Needs Crosspost Issue needs to also be reported to the other tracker or the issue needs to be linked here and removed Unreviewed Issue is new and is awaiting the team to review it labels Jan 11, 2025
@KnightMiner
Copy link
Member

KnightMiner commented Jan 11, 2025

Had our tester debug this a bit more, Geode+ just does armor trims entirely wrong. Their armor trims only work on vanilla armor for their materials on their trims. Their materials on even vanilla trim, and their trim with vanilla material are both broken on vanilla armor.

If its broken on vanilla armor, you can't expect it to work on Tinkers' Armor. Report this issue to Geode+, tell them that the mod Trimmed solves all their compatibility problems including with vanilla if they follow the steps on that mods wiki, then we can close this.

KnightMiner added a commit that referenced this issue Jan 12, 2025
If a trim pattern does not have a texture for the specific material variant, we now default to using the base trim texture tinted for the variant. Tint is guessed from the display name
Its not the best approach, but its legions better than a missing texture
@KnightMiner
Copy link
Member

Did some work on the modifier and the armor renderer, armor trim will no longer show missing textures if the trim has no variant for the material, will instead recolor the base trim texture using the font color of the trim material. Color might be a little off as its not the intended palette, but its better than missing.

Geode+ still needs to fix their trims to actually provide at minimum vanilla trim textures in their materials and vanilla materials textures in their patterns, if not just fully using Trimmed to add all combos automatically.

@KnightMiner KnightMiner added Resolved Issue is fixed in code, but there is not a release with that fix yet. and removed Not enough information Insufficient information is provided in the issue to accurately respond labels Jan 12, 2025
@KnightMiner
Copy link
Member

I've added a workaround for armor trim/armor material missing compatibility in Tinkers' Construct 3.9.1.17 for 1.20.1, which includes the missing item icons in many mods adding trim.

Note that Geode+'s trim is still very broken as it doesn't even work on vanilla armor trim patterns, so you still should report this issue to them if you want that mod fixed. I've just done everything reasonable to do on my side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.20 Issue affects 1.20 Bug Issue describes unintended or broken behavior Mod interaction Issue only happens when another mod is loaded Needs Crosspost Issue needs to also be reported to the other tracker or the issue needs to be linked here Other mod Issue is caused by another mod, and it is best fixed in the other mod Resolved Issue is fixed in code, but there is not a release with that fix yet.
Projects
None yet
Development

No branches or pull requests

2 participants