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: Combat adjustments #352

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

KRdaMystic
Copy link
Contributor

@KRdaMystic KRdaMystic commented Dec 29, 2024

This PR migrated from here EttyKitty#163.

As per original description:

Intent:

  • Create a definite Hierarchy for weapons including tiers and classes to ensure player has meaningful armament choice and progression in the Armamentarium for his marine's equipment.

  • Rebalance damage and max kill values in line with the established hierarchy and class system.

  • Balance durability for all ground units so that chapters can tackle targets as less than a doom stack.

  • Create descriptive guide documentation for each weapon class so as to guide future additions. (Half way done)

Desired End-state

  • Weapons are tangible choices with trade-offs that affect battle outcomes.

  • Squad loadouts are more indicative of intended battlefield role (Tank hunters and Horde breakers).

Noticeable changes

  • Heavy Bolter is no longer AP.
  • Missile Launcher is now -1 AP. (Low Penetration)
  • Add "martial" and "savage" weapon tags to enable custom stat scaling for melee attacks.
  • Rewrite the melee damage formula, to account for the new tags and be less extreme in general.
  • Adds functionality to boarding 1-3 tagged weapons.

Notes

A related PR: #315
A structured doc with all said rebalance changes:
https://docs.google.com/spreadsheets/d/1ZMPRXjmd3tWvFmWiIYb-nQ9qT5PYPiu_75SJNhcYqvg/edit?gid=1993273333#gid=1993273333

Summary by Sourcery

Rebalance weapon stats, introduce weapon tags for melee combat, and add boarding actions.

New Features:

  • Added "martial" and "savage" weapon tags to enable custom stat scaling for melee attacks.
  • Added boarding actions, allowing marines to board enemy ships with 1-3 tagged weapons.

Summary by Sourcery

Update weapon stats, introduce melee weapon tags, and implement boarding actions.

New Features:

  • Introduce "martial" and "savage" tags to customize melee attack scaling.
  • Add boarding actions, enabling marines to board enemy ships using weapons with the "boarding" tag.

Tests:

  • Adjust tests to reflect the updated weapon stats and boarding action mechanics.

KRdaMystic and others added 8 commits November 10, 2024 07:28
Adjusts the power of weapons according to  Weapon balance doc.
Adds functionality to boarding 1-3 tagged weapons.
Adjusts tags on some weapons and AP values on assault cannon
- Add "martial" and "savage" weapon tags to enable custom stat scaling for melee attacks.
- Rewrite the melee damage formula, to account for the new tags and be less extreme in general.
Copy link
Contributor

sourcery-ai bot commented Dec 29, 2024

Reviewer's Guide by Sourcery

This PR introduces a significant rebalance of combat mechanics, primarily focusing on weapon adjustments and the addition of a boarding action feature. Key changes include adjustments to weapon stats like attack, range, splash, and armor penetration (arp), the introduction of "martial" and "savage" tags for melee weapons, and the implementation of boarding actions with tagged weapons.

File-Level Changes

Change Details Files
Rebalanced weapon stats across various weapon types.
  • Adjusted attack values for numerous weapons, both ranged and melee.
  • Modified range and splash values for several ranged weapons.
  • Updated armor penetration (arp) values for many weapons.
scripts/scr_weapon/scr_weapon.gml
Introduced "martial" and "savage" weapon tags and reworked melee damage formula.
  • Added "martial" tag to weapons that benefit from dexterity.
  • Added "savage" tag to weapons that benefit from strength.
  • Melee damage formula now incorporates these tags, influencing damage calculation based on dexterity and strength.
scripts/scr_weapon/scr_weapon.gml
scripts/scr_marine_struct/scr_marine_struct.gml
Implemented boarding actions.
  • Added boarding action functionality, allowing marines to board enemy ships.
  • Introduced weapon tags "boarding 1", "boarding 2", and "boarding 3" to specify weapon suitability for boarding actions.
objects/obj_p_assra/Step_0.gml
scripts/scr_weapon/scr_weapon.gml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time. You can also use
    this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@github-actions github-actions bot added PR: Feature Adds something new PR: Major Coding Big enough to possibly clash with other PRs labels Dec 29, 2024
sourcery-ai[bot]
sourcery-ai bot previously approved these changes Dec 29, 2024
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @KRdaMystic - I've reviewed your changes - here's some feedback:

Overall Comments:

  • Consider implementing damage increases more gradually across multiple updates to better assess balance impact. The code improvements look great but such large jumps in damage values (e.g. Chainsword 50->130) could be disruptive.
Here's what I looked at during the review
  • 🟡 General issues: 2 issues found
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

objects/obj_p_assra/Step_0.gml Outdated Show resolved Hide resolved
refactor: Add code suggestion to boarding

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
OH296
OH296 previously approved these changes Dec 29, 2024
Copy link
Collaborator

@OH296 OH296 left a comment

Choose a reason for hiding this comment

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

Good changes all round

objects/obj_p_assra/Step_0.gml Outdated Show resolved Hide resolved
@EttyKitty EttyKitty added the PR: Balance Noticeably affects game balance label Dec 30, 2024
@EttyKitty EttyKitty added this to the v0.10.0 milestone Dec 30, 2024
@EttyKitty
Copy link
Contributor

My only note (as others may be out of the scope of this PR) is this:

  • Re some of the weapon buffs: I saw, namely, that Missile launcher is now AP, has buffed damage and 3 times more spli. My only concern, in this case and other similar cases, is how it compares to its current price that is still unchanged?

@KRdaMystic
Copy link
Contributor Author

I meant to remove the AP on it for now until I came up with a better armor pen formula

KRdaMystic and others added 4 commits December 30, 2024 15:48
adjusts weapon values so certain weapons aren't targeting tanks
adjusts weapon values so certain weapons aren't targeting tanks
@KRdaMystic
Copy link
Contributor Author

@sourcery-ai review

sourcery-ai[bot]
sourcery-ai bot previously approved these changes Dec 31, 2024
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @KRdaMystic - I've reviewed your changes and they look great!

Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@KRdaMystic KRdaMystic mentioned this pull request Dec 31, 2024
7 tasks
@OH296
Copy link
Collaborator

OH296 commented Dec 31, 2024

My only note (as others may be out of the scope of this PR) is this:

  • Re some of the weapon buffs: I saw, namely, that Missile launcher is now AP, has buffed damage and 3 times more spli. My only concern, in this case and other similar cases, is how it compares to its current price that is still unchanged?

Reading through actually i am also reading some issues.
Player weaponry in general seems to be suffering from a near continuous power creep from constant buffing without ever touching enemy weaponry, for example(taken somewhat randomly) why have meltaguns been given spli value of 6 this has essentially made them a horde killer in one fell swoop.
There have been maybe 3 or 4 big refurbs of weapons done by people that constantly are boosting weapon values i think generally as a response to the writers preferred choices and as they knit together these tweaks are becoming more and more un-sutainable.

@EttyKitty not sure what you think on this or what the position is

@KRdaMystic this is also not really about this pr as a whole but i think it needs to be considered

@OH296 OH296 modified the milestones: Release: v0.10.0, v0.10.1 Jan 1, 2025
@KRdaMystic
Copy link
Contributor Author

@sourcery-ai review

sourcery-ai[bot]
sourcery-ai bot previously approved these changes Jan 13, 2025
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @KRdaMystic - I've reviewed your changes and they look great!

Here's what I looked at during the review
  • 🟡 General issues: 3 issues found
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

scripts/scr_shoot/scr_shoot.gml Show resolved Hide resolved
objects/obj_p_assra/Step_0.gml Show resolved Hide resolved
scripts/scr_roster/scr_roster.gml Outdated Show resolved Hide resolved
@KRdaMystic KRdaMystic changed the base branch from release/0.10.0.0 to main January 13, 2025 10:37
@KRdaMystic KRdaMystic dismissed sourcery-ai[bot]’s stale review January 13, 2025 10:37

The base branch was changed.

@KRdaMystic
Copy link
Contributor Author

@sourcery-ai review

sourcery-ai[bot]
sourcery-ai bot previously approved these changes Jan 13, 2025
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey @KRdaMystic - I've reviewed your changes - here's some feedback:

Overall Comments:

  • Consider extracting weapon stats and configurations into structured data files (e.g. JSON/YAML) to reduce code duplication and make the system more maintainable.
Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Balance Noticeably affects game balance PR: Feature Adds something new PR: Major Coding Big enough to possibly clash with other PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants