pyanodon / pybugreports

Central bug-report repository for pymods
The Unlicense
5 stars 1 forks source link

Science multiplier shouldn't apply to all techs prior to wood processing #467

Closed chunkybanana closed 2 months ago

chunkybanana commented 3 months ago

Mod source

Factorio Mod Portal

Which mod are you having an issue with?

Operating system

=Windows 10

What kind of issue is this?

What is the problem?

Technologies can have a flag set to have any science multiplier not apply to them, which is often used for technologies that are required to fully automate the science that unlocks them. For example, the vanilla assembling machine technology has this to avoid handcrafting large amounts of science, and Krastorio 2 has this on its greenhouse tech and all predecessors, as the first science pack requires wood to craft.

While Py has this flag set on the vanilla automation technology, it doesn't have it set on wood processing and all its prerequisites, which are required to fully automate automation science. Currently, these techs are Wood processing, Moss - Stage 1, Botany - Stage 1, Soil washing, Glass technologies, Mining with fluid, Steel processing, Coal processing 1, Soil washing. To avoid players on science multipliers being forced to gather large amounts of wood (see below) all of these should be set to be unaffected by science multipliers.

image (screenshot courtesy of Simon Weinberger on the py discord, who's currently playing with a 100x science multiplier)

Steps to reproduce

No response

Additional context

No response

Log file

No response

notnotmelon commented 2 months ago

This is a bit like putting on a blindfold and then complaining you can't see 🤭

Thanks for the report, fixed here: https://github.com/pyanodon/pypostprocessing/commit/b51ab5024def41e915aba74c47a83b87cd913113