Equipment Grid Logistic Module (logiquipment Revived)


This mod takes 'dorfl' https://mods.factorio.com/mod/logiquipment mod and extends it to include the cargo wagon entity, for those of us who want construction trains.

Content
3 years ago
0.17 - 1.1
4.78K
Logistics

b [fixed 1.0.2]Wagon deconstruction with bots has high time usage.

3 years ago

I think this has started happening since the last mod update.

When bots are deconstructing cargo wagons and their contents, time usage by the mod goes very high for a split second every time a bot picks something up.
If the wagon is full, then this can cause a constant slowdown until the wagon is empty.

3 years ago

Are you marking the cargo wagon for deconstruction or waiting for the train to empty all the un-filtered slots ?

3 years ago

I should have clarified:
I'm not using the Equipment Grid Logistic Module on the trains at all. (Just on spidertron and aircraft)
The cargo wagon and locomotive should therefore act like vanilla as bots will not collect from the un-filtered slots.

The cargo wagon is filtered.
I am using a standard deconstruction planner.
The high time usage happens everytime items are picked up from the wagon, and when the wagon itself is deconstructed.
The high time usage also happens when manually picking up the wagon.
There is no slowdown when deconstructing/picking up the locomotive.

3 years ago

I use the events 'on_pre_player_mined_item' / 'on_robot_pre_mined' to capture when something is trying to decom one of the entities I am monitoring. Its been in the code for 7+ months. With these events I get given the entity being mined so I can make sure to disable the logistics module, or you could end up in a loop with robots bringing new items, while others are trying to take them away. However when you decom a train every time a robot takes an item from it, the event gets called.

I will see what I can do, maybe there is a way to exit out of the function early. Or I can investigate a recently added feature register_on_entity_destroyed and capture that event instead.