[boskid][2.1.8] Legendary Inserter has inconsistent rotation times

[AkaraVortex]

Latest 2.1.8 changes has affected Legendary Inserter's rotation times from consistent 8 ticks per cycle into inconsistent 8/9 ticks depending on swing direction.

I will explain this on maxed out Legendary Stack Inserter (4 stacks of size 4).

For chest-to-chest movement:
If it is horizontal, then all is as intended - 8 ticks per cycle (4 ticks for forward movement, and 4 ticks backward). But vertical one alternates between 8 ticks for left-sided rotation (4+4) and 9 for right-sided one (5+4 if looking up, 4+5 if looking down), meaning it takes 1 tick longer to travel every second cycle when swinging from right side upwards (counterclockwise).
You can easily test if for yourself by pasting in editor chest-to-chest inserters for all 4 rotations side by side, and seeing them desync almost instantly. Example below:

inserter_inconsistency.mp4

(14.44 MiB) Downloaded 9 times

inserter_inconsistency.gif (15.59 MiB) Viewed 7090 times

Exactly same for chest-to-belt and belt-to-chest movement, just with some extra amount of ticks in the middle depending on stack size and pick conditions.

Lower quality inserters do not see such issue, they have consistent swing cycle times.
I do not need to explain how devastating it is - it drops throughput from 120 to ~113 for chest-to-chest and from 80 to ~76 for chest-to-belt, which ruins a lot of builds and ratios. Secondary it breaks all clocked builds which do tick-perfect output, not only by changing the active window duration, but making it alternate uncontrollably.
And finally it is exact opposite of what 2.1.8 fix wanted to do - consistent swing times regardless of rotation or mirroring, which is right call, but current version is anything but for legendary inserter, admittedly the only quality inserter for all megabasers.

I really do hope that will be fixed to what it was before, please, Wube.

[eugenekay]

134293?

Can you share a testbed save please? I am putting one together myself to measure the tick times across versions, because this sounds interesting.

[AkaraVortex]

Yes, but since it was rejected for not being technical enough, i decided to make a post with exact details on the issue.

Here is the test map with display clock, switch and 3 cases with 4 directions to see it:

Test.zip

(1.06 MiB) Downloaded 10 times

Reply to previous post was indicating that this is insignificant, but for my base it literally breaks all builds, because they are all clocked.

[vmainen]

Definitely seems like a problem that needs to be fixed. Veteran players remember when direction of the rails decided how many inserters would fit per train cart, but a great effort has been taken by the devs to remove these kinds of quirks. I think the inserter swinging has been tweaked too over the years.

[Techjar]

I can also add to this issue, it's messing up any setup I have that uses 2 legendary stack inserters to fill a belt lane. It seems like the timings are a bit off so the inserter ends up leaving gaps where it doesn't swing over the belt in time. They do seem ever so slightly slower visually as well, but not sure if that's just a placebo.

I made a simple blueprint to test it:

Code: Select all

0eNrVmNGOmzAQRf/Fz7ACYwPJr1SrCMgktQomtc2qacS/d4BtUrUo3bvdPDRvweM5zI3nMuQi6nagkzM2iO1FmKa3Xmw/XYQ3R1u10zVbdSS2Igyu7uPgKutPvQtxTW0QYySM3dM3sU3H50iQDSYYWjLMX847O3Q1OQ6IfmYy9mAsL8XNZ/JBROLUe97W24nGqWKZb550JM68SaZPeoYsW3aeQjD26KdQR13/QruB19pAjvY7E6jjpUPVeorEcnm5mVd0NYS+qyZW7BtDtqH4VDVf+B6afpgkkEkSia7fz8Ehbqma7/BW5DhGf5Qmr6X5wNliYz05Rv+lNDmX9nWoWk7Gm1s6kt1X7ixWGFl094e4T8pm0t44apYQtUJQVwK1HOdME5MldzxzPVzMoWpoHbNAsglRD4cDuZ033zlPmlw/KziNnIfyvzoPOXIeyvedhwI/DyV2HkqcUGCEDU7IMUKa4AgNIlIcoUCExBEZiHiHv0gQoXBECiI0jkhARA4jNGjE6a23/VCzhcyhq4lfrVe+3TdSvK01aBwp3tcadA6J97UGrUPifa1B65B4X2vQOiTe1xq0Don3tQatQ+J9rUHrkO/oa9A6ZAGMPOrfRuDgBlodVUtgNlG/z6q/FMh5OuNc7xbWm/1H4uagQP/MEkTn8iE6Zymic/nxOme4tyjQ57MM0bl4jM4K0bl4gM64NSnwYZfliM75Y3QuEJ3zB+iMzy3q7hP/maXhmjnf7R+QSLzwC+IcoHO50Yp7YpPKclOM4w8rm6vU

Here's how it looked on 2.1.7 and earlier:


And here's what happens on 2.1.8:


This does in fact only occur with vertical inserters. If I paste the blueprint rotated so the inserters are horizontal, it works fine and has no gaps like the 2.1.7 image.