Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework Mega Alloy Blast Smelter bonuses #2673

Merged
merged 2 commits into from
Jul 13, 2024
Merged

Conversation

GDCloudstrike
Copy link
Member

The mabs' speed bonus currently is miserable, there is a very minor difference between using cupronickel and eternal coils, resulting in there being basically no incentive at all to upgrade them.
This PR aims to fix that by redoing the speed bonus and adding an energy discount as well.
The new speed bonus formula scales off the last 10 coil and glass tiers and works as follows:
The lower number between glass and coil tier determines the strength of this bonus, namely an additive 5% recipe time reduction per tier.
The new energy discount scales off the difference between the recipe's voltage tier and the mabs' coil tier.
This bonus grants a 5% energy discount per tier (multiplicatively)
Combined, these two bonuses should make upgrading both the glass and coils of the mabs more rewarding and more worth it.

@GDCloudstrike GDCloudstrike added the Affects Balance Change affecting balance. Requires admin approval before merging. label Jun 19, 2024
@Dream-Master Dream-Master requested a review from a team June 19, 2024 22:05
@Nxer
Copy link
Contributor

Nxer commented Jun 20, 2024

Parameters that are completely determined by the structure are best calculated in checkMachine. (I think)

@chochem
Copy link
Member

chochem commented Jun 22, 2024

will you change the quest accordingly as well?

@GDCloudstrike
Copy link
Member Author

will you change the quest accordingly as well?

sure thing

@Dream-Master Dream-Master merged commit 933650d into master Jul 13, 2024
2 checks passed
@Dream-Master Dream-Master deleted the mabs_bonus_rework branch July 13, 2024 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects Balance Change affecting balance. Requires admin approval before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants