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

Update thermal gear power draw in tool_armor.json #78832

Closed
wants to merge 1 commit into from

Conversation

mark-ik
Copy link

@mark-ik mark-ik commented Dec 29, 2024

Applied values from here to thermal electric (TE) gear.

Changed:
TE socks power draw
TE power draw, battery flag, and default battery
TE gloves power draw
TE mask power draw
TE outfit power draw

Summary

Balance "Thermal electric gear power draw reduction + medium battery for TE suit"

Purpose of change

Currently, the thermal electric outfit eats through a medium battery in seven minutes. This PR reduces the power draw of all thermal electric gear to a total of 5 W for the TE outfit. It also changes the thermal electric suit to use a medium battery instead of a light one, along with changing the battery flags for that item. See IdleSol's comment in Issue #78798

Describe the solution

Updated the on variants to new power draws:

  • TE socks (200 mW)
  • TE suit (4400 mW)
  • TE gloves (150 mW)
  • TE mask (250 mW)
  • TE outfit (5 W)

Also changed TE suit magazine flag restriction to BATTERY_MEDIUM and default magazine to medium_battery_cell

Describe alternatives you've considered

Considered letting the TE suit also reload with light batteries, but the outfit doesn't take light batteries so I stuck with what IdleSol said. Also considered lowering the power draw to a total of 4 or even 2 W, but 5 W is manageable.

Testing

Additional context

Applied values from 

CleverRaven#78798 (comment) 

Changed:
thermal socks power draw
thermal suit power draw, battery flag, and default battery
thermal gloves power draw
thermal mask power draw
thermal outfit power draw
Copy link
Contributor

You are creating a pull request with the master branch as the head branch. This is likely a mistake unless you really know what you are doing. You may read https://docs.github.com/en/get-started/quickstart/contributing-to-projects#creating-a-branch-to-work-on for a typical workflow of contributing to a project on GitHub.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Items: Armor / Clothing Armor and clothing Game: Balance Balancing of (existing) in-game features. new contributor astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Dec 29, 2024
@mark-ik mark-ik marked this pull request as ready for review December 29, 2024 11:32
@mark-ik
Copy link
Author

mark-ik commented Dec 29, 2024

Whoops, did not know about branches

@mark-ik mark-ik closed this Dec 29, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Dec 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Game: Balance Balancing of (existing) in-game features. Items: Armor / Clothing Armor and clothing [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions new contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant