[boskid][2.0.13] Rotating splitter adds items after filter

This subforum contains all the issues which we already resolved.
rvb30
Burner Inserter
Burner Inserter
Posts: 13
Joined: Mon Feb 19, 2024 9:24 pm
Contact:

[boskid][2.0.13] Rotating splitter adds items after filter

Post by rvb30 »

IDK if this is a bug or not, but the behaviour has changed from 1.1 and I personally prefer the old version.

If you have a splitter with a filter and you rotate it (R key by default) then items pass through from the input and appear on the filtered side defying the filter

Here I have a splitter:
Before.png
Before.png (238.84 KiB) Viewed 885 times
And then after rotating it in place:
After.png
After.png (222.63 KiB) Viewed 885 times
In 1.1 the extra items on the bottom right of the splitter wouldn't be there
User avatar
boskid
Factorio Staff
Factorio Staff
Posts: 3645
Joined: Thu Dec 14, 2017 6:56 pm
Contact:

Re: [2.0.13] Rotating splitter adds items after filter

Post by boskid »

I traced back this change and it is related to adding support to flip entities. There is some attempt at swapping line contents when splitter is rotated which seems suspicious as lines should be only swapped when entity gets sides flipped. This has quite low priority right now so i will reconsider how to handle this at a later point in time. One of the options is to move line content swapping to only happen when flipping sides but not when rotating.
rvb30
Burner Inserter
Burner Inserter
Posts: 13
Joined: Mon Feb 19, 2024 9:24 pm
Contact:

Re: [2.0.13] Rotating splitter adds items after filter

Post by rvb30 »

Hey, thanks for the reply,

I know it's not a priority, but wanted to flag it up.

p.s. I notice you've officially got a blue belt of posts, well done!
User avatar
boskid
Factorio Staff
Factorio Staff
Posts: 3645
Joined: Thu Dec 14, 2017 6:56 pm
Contact:

Re: [2.0.13] Rotating splitter adds items after filter

Post by boskid »

Thanks for the report. I looked at this feature of rotating splitters and decided to get rid of it which fixes the issue. This feature was trying to swap content of splitter lines in a way that would keep items "static" (remain on the same relative screen position after rotation was done) but it actually failed to do so because that would require swapping input lanes with output lanes and this operation would require more tricks as those lines are of different length. So this lines swapping is now removed for 2.0.42.
Post Reply

Return to “Resolved Problems and Bugs”