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

Voltmeter & pH meter rebalancing #3336

Merged
merged 3 commits into from
Sep 30, 2023

Conversation

RoyalFox2140
Copy link
Collaborator

@RoyalFox2140 RoyalFox2140 commented Sep 30, 2023

Summary

SUMMARY: [Balance] "Voltmeter & pH rebalancing"

Purpose of change

Voltmeters are much smaller IRL and I think it belongs on a toolbelt. I added the "BELT_CLIP" flag so it can be put on a toolbelt so it won't be lost easily. I also toned down its size and weight to suggestions on the discord. pH meters are also much smaller IRL, and it was suggested by Chaosvolt to change that in the same PR as well.

Describe the solution

I added the "BELT_CLIP" flag. It can now be on a toolbelt. Also, it is 500 grams 750mL as real life examples follow this size.
Chaosvolt changed the pH meter stats to also be more reflective of the real world.

Describe alternatives you've considered

Losing my voltmeter in a lake.

Testing

Loaded game, game worked. Voltmeter can be placed in toolbelt. It works.

Additional context

image
image

You can put the voltmeter on your toolbelt
@github-actions github-actions bot added the JSON related to game datas in JSON format. label Sep 30, 2023
scarf005
scarf005 previously approved these changes Sep 30, 2023
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interesting, both tool belt and webbing belt cannot store voltmeter because for some reason voltmeter is 1.5L and 3kg.

@RoyalFox2140
Copy link
Collaborator Author

interesting, both tool belt and webbing belt cannot store voltmeter because for some reason voltmeter is 1.5L and 3kg.

Whoops I should've tried it with the toolbelt and not my modded holsters. We should add XL toolbelts in another PR with greater encumbrance and storage.

See discussion in discord about why this size is reasonable.
@RoyalFox2140
Copy link
Collaborator Author

@scarf005 As per @chaosvolt suggestions, the voltmeter is more realistically sized at 500 grams 750 mL I have made said change and I will update PR description.

@RoyalFox2140 RoyalFox2140 changed the title Voltmeters can be put on a toolbelt Voltmeter rebalancing Sep 30, 2023
@RoyalFox2140 RoyalFox2140 changed the title Voltmeter rebalancing Voltmeter & pH meter rebalancing Sep 30, 2023
Copy link
Member

@chaosvolt chaosvolt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Ported changes over to test build.
  2. Spawned in a voltmeter and tool belt.
  3. Can put it in the belt.

image

@chaosvolt chaosvolt added this pull request to the merge queue Sep 30, 2023
Merged via the queue into cataclysmbnteam:upload with commit 2202dc2 Sep 30, 2023
9 of 12 checks passed
olanti-p pushed a commit to olanti-p/Cataclysm-BN that referenced this pull request Oct 6, 2023
* voltmeter on toolbelt

You can put the voltmeter on your toolbelt

* realistic voltmeter size

See discussion in discord about why this size is reasonable.

* Update science.json

---------

Co-authored-by: Chaosvolt <chaosvolt@users.noreply.github.com>
@RoyalFox2140 RoyalFox2140 deleted the toolbelt-voltmeter branch December 10, 2023 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants