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

feat(balance): Steel bowstring, repeating crossbow debuff #4242

Merged
merged 4 commits into from
Feb 25, 2024

Conversation

yay855
Copy link
Contributor

@yay855 yay855 commented Feb 21, 2024

Made the damage boost 5 instead of 15 and added a reload modifier of 15. Also changed the description to reflect this.

Purpose of change

The steel crossbow string was too powerful when it was made, allowing for crossbows, especially repeating crossbows, to easily outcompete many guns. The repeating crossbow's buff in the same PR that added the steel bowstring similarly made the default crossbow useless, which was poor for balancing purposes.

Describe the solution

The steel bowstring now only increases damage by 5 and has a reload penalty. The repeating crossbow now does slightly less damage and has significantly less range than a standard crossbow to make it more balanced, though still more than it had before my crossbow rebalance PR.

Describe alternatives you've considered

Testing

Additional context

Checklist

Made the damage boost 5 instead of 15 and added a reload modifier of 15. Also changed the description to reflect this.
Made the repeating crossbow have a slight damage reduction and range reduction to reflect its greater fire rate, making the standard crossbow still have a utility.
@github-actions github-actions bot added the JSON related to game datas in JSON format. label Feb 21, 2024
chaosvolt
chaosvolt previously approved these changes Feb 21, 2024
@scarf005
Copy link
Member

related: #4066

@scarf005
Copy link
Member

also cc @NappingOcean

@NappingOcean
Copy link
Contributor

The crossbow issue seems to be mostly with repeating crossbows. The bowstring nerf is devastating for those, but it seems to be even more so for other crossbows.
The bowstring is obviously very strong, so it needs to be nerfed appropriately, but I think it could be nerfed a bit less to prevent repeating crossbows from using it.

@yay855
Copy link
Contributor Author

yay855 commented Feb 22, 2024

The crossbow issue seems to be mostly with repeating crossbows. The bowstring nerf is devastating for those, but it seems to be even more so for other crossbows. The bowstring is obviously very strong, so it needs to be nerfed appropriately, but I think it could be nerfed a bit less to prevent repeating crossbows from using it.

Honestly, it might be a good idea to just buff other crossbows' damage some instead. The steel bowstring is still useful, an extra 5 damage is nothing to sneeze at, but the main issue is that non-repeating crossbows kind of suck! This game very rarely has you face up against only one enemy at a time, having the ability to just fire multiple crossbow bolts is far and above more useful than extra damage on a single-shot weapon that's nearly impossible to reload mid-battle.

@scarf005
Copy link
Member

kind of tangential, but it might make sense to change it from steel bowstring to steel prod (the bow part of crossbow), as the power of (cross)bow is determined not by bowstring but prod

On the suggestion from Scarf, the steel bowstring is now the steel prod, a steel reinforcement for the prod (or bowlimbs) of a crossbow.
@chaosvolt
Copy link
Member

Weird-ass check fail, seems unrelated?

Run echo =================================================
=================================================
Get-ChildItem: D:\a\_temp\1bba1bb3-e510-442b-91df-16d7712d8d99.ps1:3
Line |
   3 |  Get-ChildItem "D:\a\Cataclysm-BN/Cataclysm-BN/out/build/$env:CMAKE_PR …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path 'D:\a\Cataclysm-BN\Cataclysm-BN\out\build\' because it does not exist.
Error: Process completed with exit code 1.

@scarf005
Copy link
Member

image
image
image

@chaosvolt you can rerun failed workflows like this

@chaosvolt chaosvolt merged commit 929e250 into cataclysmbnteam:main Feb 25, 2024
10 checks passed
@yay855 yay855 deleted the steel-bowstring-rebalance branch February 25, 2024 17:07
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.

4 participants