Skip to content

Handle ignored_by_productivity for productivity techs#93

Open
JFletcher94 wants to merge 1 commit intodanielmartin0:mainfrom
JFletcher94:main
Open

Handle ignored_by_productivity for productivity techs#93
JFletcher94 wants to merge 1 commit intodanielmartin0:mainfrom
JFletcher94:main

Conversation

@JFletcher94
Copy link

@JFletcher94 JFletcher94 commented Mar 3, 2026

Description

  • Update process_technology_recipe_productivity_effects to take into account ignored_by_productivity
    • If a product has a value of ignored_by_productivity >= amount (or amount_max), that product will be ignored for evaluating the recipe
    • e.g.: for a productivity technology for uranium-238, uranium processing will always be excluded because increased productivity will never result in the recipe outputting more U238
    • and for a productivity technology for uranium-235, uranium processing will be included even if allow_multiple_results is false, because there's only one "productivity-capable" product (i.e. only one product will actually be affected by the productivity change)
      • It might be better to put this part behind a setting, though I haven't done that yet
  • Update process_technology_recipe_productivity_effects to not count multiple products with the same name as "different", for the purposes of allow_multiple_results
    • e.g.: the golden science pack from secretas has 2 products, but they're both golden-science-pack (one has a probability)
  • Fixed that type was documented as only applying if using name but not if using category
    • type was being evaluated even if using category

Checklist

  • Changes have been tested
  • Changes do not break backwards compatibility
  • changelog.txt has been updated
  • README.md has been updated (documentation section and/or Contributors list)

Testing Notes

I ran a diff test with the "Enable All Planets" mod enabled, and the only diffs (aside from what I was trying to get working for Crucible) were a few recipes from Cubium that have non-prod-capable byproducts were now included in productivity techs:

  • dream-processing-unit
  • cube-jelly-plastic
  • rocket-fuel-dreams
  • dream-concrete
  • refined-dream-concrete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant