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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

PTL Income Adjustment #3933

Merged
merged 2 commits into from Mar 26, 2021
Merged

PTL Income Adjustment #3933

merged 2 commits into from Mar 26, 2021

Conversation

Azrun
Copy link
Contributor

@Azrun Azrun commented Mar 7, 2021

[bug][balance][input wanted]

About the PR

Adjusted Credit scaling:

  • https://www.desmos.com/calculator/sgpk5aj6hy
  • Lower MW values are scaled to provide a behavior that feels more like diminishing returns a function that is intended to scaling infinitely. Result is "similar" to if output to be +600MW.

Keep track of power not sold, and add it to the next attempt, until it can eventually be distributed to station/crew.

Why's this needed?

Give reason to chamber burns when not attempting to "Hellburn".
Give reason to turn on PTL with Char burns.
Resolves bug when outputting in "bursts". 馃槺

Changelog

(u)Azrun
(+)Nanotrasen no longer pockets money from rounding down when paying out for PTL.  Lower end MW drastically improved.

@keywordlabeler keywordlabeler bot added C-Balance Balance changes, buffs and nerfs C-Bug A bug that impacts usage of a feature E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. labels Mar 7, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2021

Created discussion thread: https://forum.ss13.co/showthread.php?tid=15976

@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Mar 25, 2021
code/modules/power/pt_laser.dm Outdated Show resolved Hide resolved
code/modules/power/pt_laser.dm Outdated Show resolved Hide resolved
code/modules/power/pt_laser.dm Outdated Show resolved Hide resolved
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 26, 2021
@ZeWaka ZeWaka merged commit 3bc1fc4 into goonstation:master Mar 26, 2021
github-actions bot pushed a commit that referenced this pull request Mar 26, 2021
@Azrun Azrun deleted the ptl_rebalance_001 branch March 27, 2021 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Balance Balance changes, buffs and nerfs C-Bug A bug that impacts usage of a feature E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Stale An inactive PR that has had no updates in the past two weeks size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants