DRY411S / Recycling-Machines

A factorio mod that adds Recycling Machines to the game. These disassemble products back into their original ingredients.
GNU General Public License v3.0
10 stars 2 forks source link

Bob's Mods support #72

Closed embermctillhawk closed 4 years ago

embermctillhawk commented 4 years ago

Yes, it's me again. Shouldn't be so much of a hassle this time though, this is just a few missing subgroups/crafts from assorted Bob's mods that ought be added to modplugins.lua, some for support, some to ignore. This will fix, among other things, being unable to recycle Copper Wire and Electronic Circuit with certain Bob's mods installed under the new code.

Regarding bobmodules; putting aside the questionable ethics/physics of disassembling items then using productivity effects to create more of the input item than you put in, there's several missing entries for bobmodules. This will fix being unable to recycle modules, or the Beacon, because bobmodules changes the recipe type.

Now, a point about bobmodules which requires a judgement call. There are three other missing item-subgroup's, but it might make more sense to add them to ignore.

The materials required to manufacture the modules, then combine them, is greater than the materials to directly craft the combined type. Thus one can make a stable infinite loop under 1:1 recycling by disassembling the combined module via specifically the combination recipe, disassembling the resulting modules, then assembling the materials directly into the combined module, pocketing the difference.

This may be a balance issue inherent to the concept of being able to recycle in Factorio and thus a non-issue, and it becomes far less prominent or simply irrelevant if one uses a harder recycling ratio anyways, but it is something to be noted.

EDIT EDIT: Nevermind that last edit, I'm investigating that further.

DRY411S commented 4 years ago

What version are you using? v0.18.3 supports bobplates only. v0.18.4 introduced support for all Bob's mods and seems to address all your comments (apart from the 'judgement' paragraphs).

I haven't flagged it as a 'release' here in GitHub yet, but v0.18.4 is the version at the factorio official mod portal.

Edit: Actually v0.18.3 also ignores the air-pump craft (as you suggest it should). Are you trying to help by telling me how I might implement support for Bob's Mods (that as it happens I have already implemented), or reporting bugs?

embermctillhawk commented 4 years ago

Right, I'm sorry. I thought I was on the latest version already. I was attempting to help with support for Bob's Mods but you are right. The only useful information here now is the note about combining bob's modules; but you have read it so I think I'll go ahead and close this, sorry.