Fix Cataclysm scenario iridium mine production to reflect tooltip value

[Raragyay]

global.civic.coal_miner.impact is always 0.2 and is affecting iridium mine production of coal even though there are no coal miners in the scenario.

Resolves #1138

[CondoSlime]

Are you sure that this is an issue? Looking at the code, it seems like the coal mine correctly displays in the tooltip how much coal it gives.
It does seem like coal mines are the only non-job income source that is affected by a job's impact, but I don't think removing it, and 5-folding coal production is a good way to go about it.

[Raragyay]

Hey, great question! Could you clarify what you mean by "the coal mine correctly displays in the tooltip how much coal it gives"? From my understanding, each iridium mine is only giving 0.11 coal production instead of the 0.55 on the tooltip. In my attached screenshot (and I apologize for this being a bit misleading), I have 5 iridium mines active, but I'm only producing 0.55 coal base, meaning I'm only getting 0.11 / iridium mine.

I think there could be two ways to approach this:

• Given that the tooltip for iridium mines in cataclysm currently say +0.55 coal production, we should adjust coal production to reflect this tooltip
• Given that the actual production per iridium mine in cataclysm is +0.11 / iridium mine, we should adjust the tooltip to reflect this scaling.

In this case I'm not confident on which one would be the better balance decision (and am happy to adjust to whatever is most appropriate) - I'm more so trying to point out that the tooltip and the actual production are not aligned.

Happy to provide more evidence on the difference in production / tooltip if desired.

I should also note:

It does seem like coal mines are the only non-job income source that is affected by a job's impact, but I don't think removing it, and 5-folding coal production is a good way to go about it.

In this case base coal mine production (i.e. the one on the home planet) would not be affected by this change, since that's scaled here instead of at the line change that I've proposed. But maybe this is just a difference in our terminology.

[CondoSlime]

Aha, I understand it now. I did not catch the fact that you had 5 iridium mines. Looking at the Production function, makes it seem like iridium mines are expected to produce 0.55 coal/s each, so that makes this a more sensible change to me.

[ActualDio]

Ive also encountered this issue so Im bumping it so it hopefully gets merged

[yarukishi]

The coal miner "impact" rating was not applied to iridium mines in Cataclysm until the patch to implement Psychic Powers for the Eldritch genus. See commit: d37ac5f

I think that the fix made in this pull request is correct and should be applied.

[SkyeAmphi]

+1 bump good fix