[kovarex] [1.1.19] Autodrag underground belt removes belt from unconnected beltnetwork

[SteveMacGyver]

Dear Factorio Staff
Thank you very much for the great work that went into getting the game out.
Recent addition of the auto underground built functionality is extremely useful.
This, combined with the information on the "belt system" which allows for multithreading, should enable the engine to know that when dragging a belt underground from point A to point B, with belt between A and B being in the same direction, but not connected to it, should NOT replace the belt if it is not connected to the belt it is replacing. In all other instances, it should replace this belt as of current behavior.
Say you have a 2 belt bus, with an underground split-off on the outside of the belt, so you need to go underneath.
(blueprint for the first part)

Code: Select all

0eNqd1OtugyAUAOB3Ob9p4/32KsuyqCUNCQIBXGYM7z7QrO0irCu/FIIf5xyOrDDQGQtJmIZuBTJypqB7W0GRK+upm9OLwNAB0XgCBKyf3EjLninBpT4NmGowCAi74C/oUvOOADNNNMG7tA2WDzZPA5Z2QchAILiyn3HmdnVUmZxLBAt0RX0ujUEHK3vBSp9Y+f+tovmx7ItN/UIkHvcFmUcuYuTWH2X5gtU+Wr+irDxydZNne5LyKrl9PrePFUC3jmFidq1x2KqOaYHGX5AmpgUCVhtjBQ4qTWKSDGH3n0ZNPaUnTG25JRlPglP81/m4OL3i/ddRghKt7aQvsB1JAkgekWMwoiKi+kGsjMG2NO31tV113cPNiOATS7W3d5MWdZvVWZvlaV4Z8w3Qf7UR

When dragging an underground belt either from left to right or right to left, directly below the existing underground belt, the design become fluked.
(blueprint for the result)

Code: Select all

0eNqdle9qwyAUxd/lfral/muavMoYI22lCEZFzVgoefeZhnaD5m6Nn4Kiv3vu8RivcDS98kHbBM0V9MnZCM3bFaK+2NZMc2nwChrQSXVAwLbdNEqhtdG7kDZHZRKMBLQ9qy9o6PhOQNmkk1Yz6TYYPmzfHVXICzAGAe9i3ubsVHVC0XorCQzQSL6VucRZB3WaF7CRPJHZg9xnLeESXP5ibLbD2eTRs/X91NxTKf56E4zeC7FcaIElSlh8mSVXWcBfsMD1CfFgv0I3+0d3VcJC/Dys8YAe7jRREIO6JMtiWTbdPWDRG51SnlxKwAyRCISWhAlTxMrSJErSRHlJBDAbRAkMs0GugEnchCX0mjsk1qGrkiRUiAU/dyp2rTEbZXLtoE8b74z68++6R4h1iTwExnYlx430ymgJ7KYsv3y3V7L59agS+FQhzsd0oKKqWcVqxinfj+M326ZuJQ==

In case this isn't clear, I can provide screenshots as well.
Above blueprint is the result of dragging from right to left.
Kind regards
SteveMacGyver

[Loewchen]

[ptx0]

Dear Factorio Staff
Thank you very much for the great work that went into getting the game out.
Recent addition of the auto underground built functionality is extremely useful.
This, combined with the information on the "belt system" which allows for multithreading, should enable the engine to know that when dragging a belt underground from point A to point B, with belt between A and B being in the same direction, but not connected to it, should NOT replace the belt if it is not connected to the belt it is replacing. In all other instances, it should replace this belt as of current behavior.
Say you have a 2 belt bus, with an underground split-off on the outside of the belt, so you need to go underneath.
(blueprint for the first part)

kovarex mentioned lately the feature isnt meant to be intelligent in any way and these requests are out of scope.

[SteveMacGyver]

Thanks for the tip, ptx0.
It is unfortunate, but understandable after final release, even though I think that after all the logic and intelligence that went into coding this in, it is a shame that the functionality would be classed as unimportant, out of scope or otherwise ignored, as this is in my humble opinion, not the way this feature is supposed to work.

[ptx0]

Thanks for the tip, ptx0.
It is unfortunate, but understandable after final release, even though I think that after all the logic and intelligence that went into coding this in, it is a shame that the functionality would be classed as unimportant, out of scope or otherwise ignored, as this is in my humble opinion, not the way this feature is supposed to work.

i could be wrong. i re-read your request and maybe it is simple enough that it makes sense for inclusion. ignore my nay-saying.

[thuejk]

out.mp4

(58.71 KiB) Downloaded 176 times

Simpler test case. It seems clear that it just considers all belts going the other way as replaceable. IMO this is a real bug.

[Xoriun]

When placing an underground over other belts without using the smart drag feature, it only replaces straight belts going in the same direction which are not connected to anything else.
I would expect the smart belt dragging to behave the same .

[kovarex]

The logic would be far from simple to not break any other expectations.
The logic would probably be something like "disable fast replace when smart belt building and obstacle was encountered".
The personal conclusion is, that I will wait for my next playthrough, to see how important this corner case feels, and if it feels worth doing, to test what kind of other problems will this logic bring.
Until then, it is considered to be minor issue.