moidu21 / minerplus

0 stars 0 forks source link

Speeds higher than FactoryTick can allow #37

Open lynkfox opened 1 month ago

lynkfox commented 1 month ago

This mod has belts that exceed the hardlimits of the Factory Tick:

Due to the complexities of how the game has implemented belts, the maximum belt transfer speed can't exceed 2000 items/min without leading to a loss of items. Splitters and mergers also have a similar 2000item/min limitation. If a mod offers faster belts than this, they will not work properly, and are likely a relic from older updates. Mods that compress items or globally reduce item costs (ex. Th3's Part/Recipe Compressor) exist as a workaround for the time being.

This hard coded limited was confirmed by CSS Devs in the modding server, and belt speeds faster than this are likely to cause game issues.

V3rald commented 1 month ago

Might i add, not just loss of items but belts not transporting items at all, corrupted save files and other various visual bugs because of the high speed

moidu21 commented 1 month ago

did you try this mod (because i play with it on my dedicated server and i don't get any issue with the belt speed) but it's clearly depending on your CPU capacity if you have a old one one yeah it might not follow Ben from CSS in U8 tell me how to bypass this and yeah in U8 it do not work very well but with this update (1.0) it's working well at least for me and my CPU (and all users that using it do not complain about the belt speed so i thnk it's ok for some prople)

V3rald commented 1 month ago

i did, thats why i commented i have a ryzen 5 5600x cpu, should be plenty for it.

V3rald commented 1 month ago

https://imgur.com/a/rlYUBDR this is just one of the issue with it that i happened to get on camera

After this, it eventually breaks the whole game, by stopping the whole conveyor belt (if you break the belt 1 ghost item at the end stays somehow but its clearly empty because machines dont get any and ofc the conveyor belt seems empty too) and if you save at this point, and try to reload it will crash the game consistently

lynkfox commented 1 month ago

It is not dependent on your CPU. It is a hard coded game limit on the factoryTick that it will only operate at 2000 or so item transfers per minute.

This information was provided directly by CSS devs.

Bypassing it means changing the timing of the FactoryTick which means changing the entire timing of the the entire game

On Wed, Oct 23, 2024, 5:47 AM Verald @.***> wrote:

https://imgur.com/a/rlYUBDR this is just one of the issue with it that i happened to get on camera

— Reply to this email directly, view it on GitHub https://github.com/moidu21/minerplus/issues/37#issuecomment-2431546662, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADLOBQ5B5523JPTGNOAMTVTZ45WEHAVCNFSM6AAAAABQLFN3J6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZRGU2DMNRWGI . You are receiving this because you authored the thread.Message ID: @.***>

Th3Fanbus commented 1 month ago

>belts2000 exists for a good reason: belt throughput can drop when the simulation struggles to keep up. I've seen people who couldn't even achieve 2000 items per minute on their computers.

I highly doubt that Ben from CSS would tell you specifically how to bypass a belt speed limit. Why would you be the only one to know?

V3rald commented 1 month ago

and clearly this bypass doesnt work anymore, or very unstable and doesnt work for everyone. i have a good cpu so that surely isnt the issue

Mastrmyrtr9 commented 1 month ago

I have performed two seperate tests on your Mk8 belt (claims to transfer 6000 items/minute). One was using SirDigby's Throughput Counter, and it reported an average of around 4800 items/min over 5 minutes (5 seperate counts).

For the other, I simply placed 6000 items in a storage container and then linked the container up to a sink for 1 minute, and then disassembling the belt right after. In that test, there was 1360 leftover, which means 4640 items were transferred in 1 minute.

I've given you the methods I used, so if you would like to replicate them yourself, then feel free. But it is certainly not related to CPU performance (I have a 12400f. Not the best in the world, but absolutely a capable CPU.)

moidu21 commented 3 weeks ago

@Th3Fanbus i'm not a liar ben

@Mastrmyrtr9

SirDigby's Throughput Counter use factory grab output to count, so while it count item one by one it's limited edit. i just try your second way to replicate with BAM FP (not the renewed version) and for me nothing left in the container after 1 min

Th3Fanbus commented 3 weeks ago

@Th3Fanbus i'm not a liar

I never said you were one. Thanks for the information, though - I found the message in https://discord.com/channels/555424930502541343/862002311579435028/1055040873479540736 (modding channel in the Satisfactory Modding Discord).

moidu21 commented 3 weeks ago

Bypassing it means changing the timing of the FactoryTick which means changing the entire timing of the the entire game

@lynkfox

Sorry i forgot to respond to you and i want to be clear i just change the number of substep on the factory tick and not the factory tick itself so their is no change on the timing on the entire game it stay the same as vanilla game

Acxd25 commented 2 weeks ago

I've replicated Mastrmyrtr9's 2nd method, and for me it was much worse.

1st test: 2354 left 2nd test: 2377 left 3rd test: 2325 left

Imo that mod should be marked as damaged

Edit: I can record a video if needed.

moidu21 commented 2 weeks ago

@Acxd25

I've replicated Mastrmyrtr9's 2nd method, and for me it was much worse.

me too and that's what happening on my side video belt speed

so no need to mark it as damaged

Borketh commented 1 week ago

The fact remains that others are not able to reproduce your success. "Works on my machine" is not grounds to declare a mod working.

moidu21 commented 1 week ago

The fact remains that others are not able to reproduce your success. "Works on my machine" is not grounds to declare a mod working.

the thing i don't understand is where are all the report about the belt speed if it work only on my machhine i should get many bug report on discord and here

moidu21 commented 1 week ago

i don't even know if you have actived the setting in config

Th3Fanbus commented 1 week ago

i don't even know if you have actived the setting in config

Which setting in which config?

moidu21 commented 1 week ago

Which setting in which config?

setting

Acxd25 commented 6 days ago

So in order for conveyors to work beyond 4k, players need to enable a config that will cause performance loss. To me, this is a damaged mod.

+The current CTD when players try to use some of your machines in BP, which you mentioned that there's no fix.

V3rald commented 6 days ago

and btw i have issue with the config enabled

moidu21 commented 5 days ago

latest version of all my mod

https://drive.google.com/drive/folders/14xBIkQjkl5rIJ7k2HxVcV5L6q8Qn1wKY?usp=sharing