Smart Splitter Revived


Dynamic size belt splitters with item filtering for Factorio 2.0. Based on SmartSplitters by ThaPear, ported to 0.13 by EurypteriD.

Content
18 days ago
2.0
63
Logistics

b Crash when loading existing save

19 days ago
(updated 19 days ago)

When you load the mod into an existing non-SA save which didn't previously have the mod, you get this crash:

The mod Smart Splitter Revived (1.0.0) caused a non-recoverable error.
Please report this error to the mod author.

Error while running event SmartSplitterRevived::on_configuration_changed
Unknown entity name: smartsplitter-turbo
stack traceback:
    [C]: in function 'find_entities_filtered'
    __SmartSplitterRevived__/control.lua:360: in function 'BootstrapExisting'
    __SmartSplitterRevived__/control.lua:376: in function <__SmartSplitterRevived__/control.lua:376>

Two other bugs:

You should add an optional dependency on "? TurboBelt" to avoid a crash when loading with that mod.

Also, the localised description for the Smart Splitter items has an unresolved "[mod-setting=SmartSplitterRevived-max-group-size-express"] component in it. You might want to look into adding a "Group size" CustomTooltipField for the items instead.


As a finer point of feedback, I'm not sure why the "Unlock by research" setting exists, when the recipe for a Smart Splitter always requires the corresponding tier of splitter. Perhaps you could just enable the recipe when you fail to find the correct technology to attach it to.

18 days ago

Thanks so much for the bug reports and feedback. This is my first work on a proper Factorio mod, so I appreciate the guidance! I've resolved the issues in 1.0.1 and included your recommendations.

For the research, I absolutely see your point - I've instead tweaked this setting to either unlock the smart splitters alongside their tiers smart splitters OR further gate it behind its own research so it takes a bit more effort to get them unlocked.

18 days ago
(updated 18 days ago)

Feedback for version 1.0.2:

  • "Max group size" is listed twice for items. You can delete custom_tooltip_fields for the item prototype, it will inherit them from the entity.
  • Turbo Smart Splitter requires Space science packs even when the Turbo Belt mod is used (that mod uses Utility science packs). When playing in Space Age, Turbo Smart Splitters do not require Metallurgical science packs. (I'm not saying you necessarily need to change either case, just pointing it out - it would be fine to leave it like this.)
  • Is there a reason you're not writing your changes into a standard changelog.txt?
  • It might be nice to show which smart splitters are part of the group whenever you hover over one of them.

One way to do this could be creating a set of highlight-box entities (scroll down a bit) when selecting a smart splitter, and deleting them when un-hovering.

New response