GimoXagros / morebobs

Extends additional add-ons for all of the Bob's mod
Other
1 stars 1 forks source link

Broken Research Chains #1

Open dnulho opened 6 years ago

dnulho commented 6 years ago

I have found 2 broken research chains in my game, that disappear when this mod is disabled. Mod list: mod-list.txt The two issues are: "sct-lab-t1" sct-lab-t1 unavailable "sct-lab-t1" has a dependency "basic-logistics" that is disabled and not used with your mod active. "bi_tech_fertiliser" bi_tech_fertiliser unavailable You change the dependencies of "bi_tech_fertiliser" to "sulfur-processing" and "bi_tech_bio_farming". I'm not sure yet what mod that I have is doing so, but "sulfur-processing" is also disabled.

OvermindDL1 commented 5 years ago

These are still issues in the current version, they are cutting off venues of research. The first was resolved by the sciencecosttweaker by working around what this mod is breaking, however the second is purely an issue with this mod setting a dependency chain that is controlled by angels improperly (perhaps built with an old angel's version in mind).

More details: https://mods.factorio.com/mod/morebobs/discussion/5b8e72cadc011e0009ab8d5e

OvermindDL1 commented 5 years ago

What's happening is that when angels calls angelspetrochem_0.7.12/prototypes/petrochem-global-override.lua: OV.global_replace_technology("sulfur-processing", "angels-sulfur-processing-1") it happens before morebobs final-fixes is run , so either morebobs need to change the dependency to run before angelspetrochem or morebobs needs to set angels-sulfur-processing-1 instead of sulfur-processing when angels-sulfur-processing-1 exists. Optimally it would test for if a research is enabled at all before setting it as a pre-requisite for a research though.