PepperCode1 / Continuity

A Fabric mod that allows for efficient connected textures
GNU Lesser General Public License v3.0
269 stars 70 forks source link

Compatibility Issue Between gtceu and Continuity Mods #303

Closed Start-Buling closed 11 months ago

Start-Buling commented 1 year ago

GregTech CEu Version

gtceu-fabric-1.20.1-1.0.11

Addons Installed

the log of the continuity-3.0.0-beta.2+1.20: 2023-08-15-1.log.gz the log of the continuity-3.0.0-beta.3+1.20.1: 2023-08-15-2.log.gz

PepperCode1 commented 1 year ago

LDLib makes it so all states of all blocks that implement a certain interface use the same ModelIdentifier. Continuity assumes each state has a unique ModelIdentifier, but LDLib violates this assumption. I do not think LDLib should making multiple states use the same ModelIdentifier, so I will ask them to change this. I can still work around this issue in Continuity if it is not fixed in LDLib, but I would prefer not to.

Start-Buling commented 1 year ago

Thank you very much for your response. In fact, I had already raised this issue with them before submitting it to you. However, their response was to have me report it to you first. In reality, I very much agree with your point that they should prioritize solving this issue, rather than you. I believe both of these mods are excellent, and it would be unfortunate if they are unwilling to fix it. Also, I'd like to ask, do I need to report this issue to them again?

PepperCode1 commented 1 year ago

Also, I'd like to ask, do I need to report this issue to them again?

No, I opened a new issue with technical details: Low-Drag-MC/LDLib-Architectury#9.

Start-Buling commented 1 year ago

I understand,thank you for your dedicated attention to my issue..have a good day!

PepperCode1 commented 1 year ago

This should have been fixed. Try updating your mods and check if it works.

PepperCode1 commented 11 months ago

I will assume this is resolved. If not, open a new issue.