Possible solution for #21

[Sunconure11]

https://github.com/TonimatasDEV/PacketFixer/issues/21#issuecomment-1716807706

You should probably check this out.

[qyl27]

It seems that it will cause this mod required in both client and server?

[TonimatasDEV]

It seems that it will cause this mod required in both client and server?

It is simply pinned, it will not be done for the time being.

[Sunconure11]

Do you have any idea as to when you'd be free to do this? I know I normally shouldn't ask this, but given the nature of this solution, it would probably help quite a few people.

[Sunconure11]

@Linguardium Maybe you can help with getting this on various versions of the mod?

[Sunconure11]

I discovered this in 1.19.2 though, so I'd say 1.19.2 and up should receive this.

[TonimatasDEV]

Do you have any idea as to when you'd be free to do this? I know I normally shouldn't ask this, but given the nature of this solution, it would probably help quite a few people.

The creators of Forge and Fabric said they were not going to do it because they don't think it is the most optimal way. If a case will create a separate mod.

[TonimatasDEV]

@Sunconure11 Test this version: PacketFixer-fabric-1.1.8-1.19.2.test20.zip

[Sunconure11]

I will shortly. It looks like a mod had a bad update or something for me currently.

[Sunconure11]

I'll get back with you in a little while. I also woke up a bit... too early, and I also still haven't found why I'm having an unrelated issue.

[Sunconure11]

Alright, it looks like I can begin bug hunting now

[TonimatasDEV]

Alright, it looks like I can begin bug hunting now

Let me know if I have managed to solve the problem you had last time.

[Sunconure11]

https://gist.github.com/Sunconure11/fbd57261851d30552685e64655cdfa7e

[TonimatasDEV]

https://gist.github.com/Sunconure11/fbd57261851d30552685e64655cdfa7e

What is the problem? 5 thousand lines are difficult to read.

[Sunconure11]

Packet splitter mixin seems to be failing.

https://pastebin.com/g8hhAEzX

[Sunconure11]

If you find a possible cause, I'll test it again, but I need to lay down for a bit.

[TonimatasDEV]

Packet splitter mixin seems to be failing.

https://pastebin.com/g8hhAEzX

Test this version: PacketFixer-fabric-1.1.8-1.19.2.test21.zip

[Sunconure11]

Looks like the server is now hitting something that is more likely related to the host themselves. I'll see what can be done there.

[Sunconure11]

16.09 19:53:37 [Server] Netty Epoll Server IO #5/WARN java.lang.OutOfMemoryError: Requested array size exceeds VM limit
16.09 19:53:37 [Server] Netty Epoll Server IO #5/WARN Network error for:packet_handler
16.09 19:53:37 [Server] INFO java.lang.OutOfMemoryError: Requested array size exceeds VM limit
16.09 19:53:37 [Server] Netty Epoll Server IO #5/ERROR Twilight Forest: Exception occurred in netty pipeline
16.09 19:53:37 [Server] INFO java.lang.OutOfMemoryError: Requested array size exceeds VM limit
16.09 19:53:37 [Server] Netty Epoll Server IO #5/ERROR Exception occurred in netty pipeline
16.09 19:53:37 [Server] INFO java.lang.OutOfMemoryError: Requested array size exceeds VM limit
16.09 19:53:37 [Server] Netty Epoll Server IO #5/WARN java.lang.OutOfMemoryError: Requested array size exceeds VM limit
16.09 19:53:37 [Server] Netty Epoll Server IO #5/WARN Network error for:packet_handler
16.09 19:53:37 [Server] INFO java.lang.OutOfMemoryError: Requested array size exceeds VM limit

[Sunconure11]

Server uses 16 gigs, but it is apparently running into this when it is not even using that much RAM.

[Sunconure11]

I have posted it here just in case it is being a bit deceptive and that is not host related.

[Sunconure11]

I decided to do some digging, by running the server and client without Packet Fixer.

https://pastebin.com/nGyHBF3i

https://pastebin.com/TtZNF6F5

[Sunconure11]

I am also using these mods, in case they may have recipe conflicts:

https://www.curseforge.com/minecraft/mc-mods/block-swap

https://www.curseforge.com/minecraft/mc-mods/almost-unified

https://www.curseforge.com/minecraft/mc-mods/recipe-essentials-forge-fabric

https://www.curseforge.com/minecraft/mc-mods/polymorph

[TonimatasDEV]

Server uses 16 gigs, but it is apparently running into this when it is not even using that much RAM.

Better not to fix this problem, as it causes the servers to consume all your ram. That's why it has these limits. Thank you very much for reporting this bug.

[Sunconure11]

Just curious, but do alternate solutions exist that don't eat up so much memory?

[Sunconure11]

Or at least, are physically possible?

[TonimatasDEV]

Or at least, are physically possible?

They may be possible, but I have not seen them exist.

[TonimatasDEV]

@Sunconure11 I have a plan for fix this

[Sunconure11]

Of note is that this mod still has some oddities with Krypton installed, and with all of these tests, I had to uninstall Krypton. I suggest adding some compat there too.

[Linguardium]

Just curious, but do alternate solutions exist that don't eat up so much memory?

Forge already has the fix for it built into forge itself. this would really only be needed for Fabric. As far as Quilt goes, i am not sure if the mod is being developed for Quilt or if quilt compatibility is assumed, but i would suggest making sure that whatever base it is being developed on is the base it is being tested on. just to make sure the quilt team isnt doing something funky (they have been known to break things with their QSL and such)

[Sunconure11]

I'm mostly on Quilt because there are a few things exclusive to it that I wanted to play with.

[TonimatasDEV]

Just curious, but do alternate solutions exist that don't eat up so much memory?

Test this version: PacketFixer-fabric-1.1.8-1.19.2.test3246178234.zip

[Sunconure11]

Also, regardless of my results, could you check these two mods for any possible incompat issues for the possible solution?

https://www.curseforge.com/minecraft/mc-mods/connectivity

https://www.curseforge.com/minecraft/mc-mods/krypton

[Sunconure11]

Hm, we may be close. I can enter, but the TPS seems to be very low, so stuff takes forever to load up on my end. It is possible pregeneration may be needed at this point, but I'll see how a restart of the server handles things first.

[Sunconure11]

No visible errors in the log, aside from can't keep up ones. Gonna try some ideas in the mean time to clean those up.

[Sunconure11]

Tried to test with Very Many Players to see if I could load in fully, but I just got stuck at the loading screen for the server. Gonna see if Krypton has issues with these changes next. Will take pictures and logs of both issues. But it seems we are getting close.

[Sunconure11]

https://pastebin.com/qd5Rc745

Looks like the packet splitter mixin fails with Krypton.

[Sunconure11]

VMP gets stuck on loading terrain. There was one last issue, lemme recreate it.

[Sunconure11]

https://legacy.curseforge.com/members/someaddon/projects

Some of the chunk-related mods may need a look over in regards to compat with VMP and this.

[Sunconure11]

Okay, so it seems the VMP issue occurs not because of VMP, but due to the map itself. It does not want to load in. Although VMP seemed to get me close, as I could hear the sound playing from the world.

[Sunconure11]

latest.log Server log

[Sunconure11]

latest.log Client log

[Sunconure11]

I'm seeing a mixin failure in the log with this...

https://www.curseforge.com/minecraft/mc-mods/better-chunk-loading-forge-fabric

[Sunconure11]

If I find any more info, I will post it. But I do not seem to know of anything, aside from maybe some kind of issue with the chunk performance mods

[Sunconure11]

@Linguardium @TonimatasDEV Maybe compare the Forge solution to the one being attempted? Maybe they solved weird issues like this.

[TonimatasDEV]

Also, regardless of my results, could you check these two mods for any possible incompat issues for the possible solution?

Connectivity is full compatible

[TonimatasDEV]

https://pastebin.com/qd5Rc745

Looks like the packet splitter mixin fails with Krypton.

Delete Krypton

[TonimatasDEV]

@Sunconure11 Does it let you login to the server? Is the TPS problem caused by Packet Fixer or by the number of mods?

[Sunconure11]

It shows me logged in on the log... but I am not physically in game.

TPS could be mods, yes, but I want to see what is going on a bit more. I am planning on further cleanup of this pack, but I've been unable to find someone willing to help me, or has the time to.

[Sunconure11]

Cleanup is a bit hard here too, given a lot of the config stuff I did, and what not.

[TonimatasDEV]

It shows me logged in on the log... but I am not physically in game.

TPS could be mods, yes, but I want to see what is going on a bit more. I am planning on further cleanup of this pack, but I've been unable to find someone willing to help me, or has the time to.

A modpack of 831 mods is too big, you would need a very very powerful server, how many cores and ram do you have in the server?