Brian-Wuest / MC-Prefab

A minecraft mod designed to provide ways of creating Pre-Fabricated(designed) buildings.
Other
26 stars 20 forks source link

[Bug]: Stuck while running with [Physics Mod] #294

Closed MTZYWX7 closed 2 weeks ago

MTZYWX7 commented 2 weeks ago

Minecraft Version

1.21.1

Forge Version

Neoforge 21.1.72

Mod Version

1.0.0

Details

Conflict with the [Physics Mod] module, right clicking on the ground or right clicking to open the GUI causes it to freeze. [Physics Mod] version is : Physics-Mod-pro-v172b-neoforge-1.21.0

Brian-Wuest commented 2 weeks ago

You're going to have to provide a link to that as it doesn't appear to exist on CurseForge or Modrinth or at least not easily findable. Without having a downloadable copy of the mod to troubleshoot with I cannot determine what is wrong with the mod compatibility.

MTZYWX7 commented 2 weeks ago

Sorry, I didn't think carefully. Here is the download link: https://www.patreon.com/posts/physics-mod-pro-105510754 or https://www.patreon.com/file?h=105510754&m=320716234

kaojinn commented 2 weeks ago

Due to this being a paid mod, and us not having a subscription to the patreon for the author, we can't just take the mod, even for diagnostic purposes. It would be ethically dicey, since your access to it is dependent upon your contributing to their work. At this point, I would suggest reaching out to the mod author about the issue, and see if they have any suggestions, or the author can reach out to us and provide a copy of the mod and permission. It's the same kind of professional courtesy we would expect from anyone else if our mod followed a pay structure.

On Fri, Nov 8, 2024 at 7:37 AM MTZYWX7 @.***> wrote:

Sorry, I didn't think carefully. Here is the download link: https://www.patreon.com/posts/physics-mod-pro-105510754 or https://www.patreon.com/file?h=105510754&m=320716234

— Reply to this email directly, view it on GitHub https://github.com/Brian-Wuest/MC-Prefab/issues/294#issuecomment-2464787492, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE5UAS4QIWPZNHWSFSM4UNLZ7S5ABAVCNFSM6AAAAABRKQSHS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRUG44DONBZGI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

MTZYWX7 commented 2 weeks ago

But this mod is currently free, and the professional version can be downloaded without the need for a membership. It is included in the MCMod website…(https://www.mcmod.cn/class/3460.html屏幕截图(1208)

MTZYWX7 commented 2 weeks ago

I uploaded two screen recording files in Gofile cloud storage, one without Physics Mod operation and the other with Physics Mod module operation. This time, the Physics Mod module is the free version available on the Modrinth website.

https://modrinth.com/mod/physicsmod

https://gofile.io/d/k4ol6R

Brian-Wuest commented 2 weeks ago

Thank you for the download links, I was able to duplicate the issue with the freely available version of the mod on CurseForge.

Unfortunately I cannot determine the cause of the issue. There is a significant lag spike in processing during a core Minecraft method call here: https://github.com/Brian-Wuest/MC-Prefab/blob/683229eb05bdb9bb1b43dead75d546705f0649fe/Shared/src/com/prefab/gui/GuiBase.java#L331

It gets past this line of code but it takes like 10 seconds to process it, which is a VERY long time and then just seems to loop forever after our code is complete.

Because Phsyics mod is closed source, I cannot look at their code to see what kind of Mixins or other type of code they might be using to change core Minecraft mechanics to change such functionality.

Please note that when I disable the Physics mod this freezing behavior goes away.

As a results I would have to put this back onto the author of the Physics mod to determine what's going on to break this functionality as this is fairly basic and standard usage of Minecraft GUI rendering.

So, closing this issue.

MTZYWX7 commented 2 weeks ago

Thank you!