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

2-2 jobs skill rebalance (Part 1) (2018 patch/Renewal) and small skill fixes #3229

Merged
merged 11 commits into from
May 25, 2024

Conversation

guilherme-gm
Copy link
Member

@guilherme-gm guilherme-gm commented Aug 20, 2023

Dependency

This PR depends on #3223 , all commits from #3223 are included in this one. The first commit original to this PR is:

Rebalance of CR_GRANDCROSS (Grand Cross)

Pull Request Prelude

Draft Reason

This PR by itself is ready and I don't think there will be further changes, unless someone finds an issue or a review asks for some change.

BUT, the rebalance in main official servers came at once, so I believe the right thing to do is to merge this PR together with the rest of the rebalance (that will be in other PRs).

So I will keep this as draft for now, but feel free to give an early review :)

Changes Proposed

The main purpose of this PR is to introduce the rebalance of part of the 2-2 jobs skills. This change affects Renewal-only.

On official servers this came along with rebalances of 1st, 2nd jobs and transclass too. I am working in additional PRs for the remaining 2-2, including bard/dancer, being in separate PRs in order to keep those PRs in a reasonable size.

The implementation in this PR is based on kRO and kRO zero patch notes, iRO Wiki, rAthena and divine pride info, along with some in-game testing. I can't say everything is 100% accurate because there were discrepancies between different sources, and I could not test everything in kRO, but should be quite close.

I won't list all the rebalance changes in the PR description, but it may be checked in each commit text. Also, the commits are in the same order as they appear in the references below.

Affected jobs/skills

  • Crusader
    • CR_GRANDCROSS (Grand Cross)
    • CR_SHIELDBOOMERANG (Shield Boomerang)
    • CR_SPEARQUICKEN (Spear Quicken)
  • Alchemist
    • AM_DEMONSTRATION (Bomb)
    • AM_ACIDTERROR (Acid Terror)
  • Rogue
    • RG_RAID (Sightless Mind)
    • RG_BACKSTAP (Back Stab)
    • RG_PLAGIARISM (Intimidate)

References:


Additionally, there were 3 changes not related to the rebalance patch that is being applied in this PR (since it just appeared exactly where I was touching 😅 )

  • (Both) Moved AM_DEMONSTRATION chance of breaking weapon to skill_db.conf
    • this simplified the rebalance patch, which gives new chances for renewal
  • (Both) Fixed Rogue's Backstab bow damage
    • part of the damage was not being halved because it was in the default 100% ratio, now it properly halves the entire damage
  • (RE) Fix the damage efect of AM_ACIDTERROR
    • In renewal, it shows a bundle of 5 hits instead of only 1. (I don't have info for pre-re, so I didn't change it)

Huge thanks to Asheraf and skyleo for helping me with some tests/validations of how it should work.

Issues addressed:
Part of #2727

db/re/skill_db.conf Outdated Show resolved Hide resolved
src/map/battle.c Outdated Show resolved Hide resolved
db/pre-re/skill_db.conf Outdated Show resolved Hide resolved
db/re/skill_db.conf Outdated Show resolved Hide resolved
src/map/battle.c Outdated Show resolved Hide resolved
db/re/skill_db.conf Outdated Show resolved Hide resolved
db/pre-re/skill_db.conf Outdated Show resolved Hide resolved
db/re/skill_db.conf Outdated Show resolved Hide resolved
db/re/skill_db.conf Outdated Show resolved Hide resolved
src/map/skill.c Outdated Show resolved Hide resolved
- Fixed casting time changed: 1.5s -> 0.5s
- Variable casting time changed: 1.5s -> 1s
- Global delay changed: 1.5s -> 0.5s
- 1s cooldown added
- Self damage is removed

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
- Damage formula changed
  - Old: (100 + 30 x SkillLevel)% ATK
         Shield weight gives extra flat damage
  - Now: (80 x SkillLevel + Shield Weight + Shield Upgrade x 4)% ATK

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
- Additionally gives +10% of Equipment ASPD

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
- Chance of breaking target weapon increased
  - Old: SkillLevel %
  - New: (3 x SkillLevel)%
- It now uses basic/default meelee damage calculation (instead of
  special formula)
- Damage formula changed:
  - Old: [100% + (Skill Level x 20)]% (ATK + MATK)
  - New: [(Skill Level x 60) + (Potion_Research_Lv x 10)]% ATK

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
- Chance of breaking target armor increased
  - Old: 3/7/10/12/13%
  - New: 5/15/25/35/45%
- It now uses basic/default ranged physical damage calculation (instead
  of special formula)
- Damage formula changed:
  - Old: [200% + (Skill Level x 80)]% (ATK + MATK)
  - New: [(Skill Level x 200) + (Potion_Research_Lv x 100)]% ATK

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
For Renewal Acid Terror damage is applied in a bundle of 5 hits isntead
of a single damage
- SP cost reduced: 20 -> 15
- Damage increased
  - Old: [100 + (Skill Level x 40)]%
  - New: [50 + (Skill Level x 150)]%
- Damage debuff changed
  - Old: 20% more damage for 5 seconds or 7 hits
  - New: 30% more damage (15% on boss monsters) for 10 seconds

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
before this commit, only part of the skillratio value was being halved
when using bows, because skillratio by default is 100.

the correct behavior for bows in backstab is to halve the entire
skillratio value (including the initial 100%)
- Skill mechanics changed
  - No longer needs to be behind the target to use skill
  - No longer ignores accuracy check
  - No longer usable from Hiding state
  - Now increases skill Hit by 4 per skill level
  - Number of hits changed:
    - Old: 1 hit
    - New: 2 hits with Dagger, 1 hit with other weapons
- SP cost reduced: 16 -> 12
- Cooldown added: 0.5s
- Number of hits changed

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
- Also increases Attack Speed by 1% per Skill Level

From massive skills rebalance (1st/2nd/transclass) (2018.10.31)
@MishimaHaruna MishimaHaruna merged commit 3c4fe7b into HerculesWS:rebalance May 25, 2024
345 of 346 checks passed
MishimaHaruna added a commit that referenced this pull request May 25, 2024
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.

2 participants