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

[Combat] Adjustments to Crippling Blows/Slay Undead and Confirmed Critical Code #4354

Merged
merged 2 commits into from
May 27, 2024

Conversation

fryguy503
Copy link
Contributor

Description

After moving to master, we had reports of Crippling Blows, Slays and criticals not always working as intended. Number of reports of slays in the double digit damage. We have utilized this reordered and I think a bit more readable logic for years. I also have increased combat logic and tuning to allow custom servers such as solo only servers to help ramp up that damage or ramp it down to better account for their content.

Rule Added:

RULE_REAL(Combat, SlayDamageAdjustment, 0.5, "Slay Damage Adjustment Default: 0.5")

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Testing

Post Adjustment showing holy blade cleansing with an appropriate value for DoN level characters.

image

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@fryguy503 fryguy503 requested a review from Kinglykrab May 27, 2024 21:52
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
common/ruletypes.h Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
zone/attack.cpp Outdated Show resolved Hide resolved
@Kinglykrab Kinglykrab changed the title Adjustments to Crippling Blows/Slay Undead and Confirmed Critical Code [Combat] Adjustments to Crippling Blows/Slay Undead and Confirmed Critical Code May 27, 2024
@Kinglykrab Kinglykrab merged commit 0d88826 into master May 27, 2024
2 checks passed
@Kinglykrab Kinglykrab deleted the trust/slay_holyforge_fixes branch May 27, 2024 23:53
@Akkadius Akkadius mentioned this pull request Jun 2, 2024
catapultam-habeo pushed a commit to The-Heroes-Journey-EQEMU/Server that referenced this pull request Jul 14, 2024
…tical Code (EQEmu#4354)

* Adjustments to Crippling Blows/Slay Undead and Confirmed Critical Code

* Adjustments per comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants