Skip to content

Skill rebalance from kro#3200

Draft
gbasso666 wants to merge 31 commits intoHerculesWS:masterfrom
gbasso666:kro-mass-skill-rebalance
Draft

Skill rebalance from kro#3200
gbasso666 wants to merge 31 commits intoHerculesWS:masterfrom
gbasso666:kro-mass-skill-rebalance

Conversation

@gbasso666
Copy link
Copy Markdown

  • I have followed [proper Hercules code styling][code].
  • I have read and understood the [contribution guidelines][cont] before making this PR.
  • I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

Implemented the skill changes from this patch: https://www.divine-pride.net/forum/index.php?/topic/3453-kro-mass-skills-balance-1st-2nd-and-transcendent-classes-skills/

All skills that were reworked were tested and seem to be working correctly. If I miss anything let me know

NOT IMPLEMENTED:
-Basilica (completely new skill);
-AMP (completely new skill);
-Harmonic Lick (completely new skill);
-Classical Pluck (todo: add CONFUSION to the aoe effect);
-New Falcon Assault formula (IRO formula seems weird/wrong and I couldnt find another source).

Also, Bowling Bash should only have 4 hits on 4 or more targets with 2h sword. Atm I only implemented 4 hits with 2h sword at all instances (which I think it's fair).

Additional content: Some skill were later (2022) adjusted again in kro, I might have already updated them (like songs lasting 180sec).

PS: btw, Hindsight skill from Professor seems very convoluted on skill.c, large portions of code doesnt seem to do anything, I recommend someone clean it up.

From this patch: https://www.divine-pride.net/forum/index.php?/topic/3453-kro-mass-skills-balance-1st-2nd-and-transcendent-classes-skills/

All skills that were reworked were tested and seem to be working correctly.
If I miss anything let me know

NOT IMPLEMENTED:
-Basilica (completely new skill)
-AMP (completely new skill)
-Harmonic Lick (completely new skill)
-Classical Pluck (todo: add CONFUSION to the aoe effect)

Also, Bowling Bash should only have 4 hits on 4 or more targets with 2h sword. Atm I only implemented 4 hits with 2h sword at all instances (which I think it's fair).

Additional content: Some skill were later (2022) adjusted again in kro, I might have already updated them (like songs lasting 180sec).
@gbasso666
Copy link
Copy Markdown
Author

Thanks Mishima, I've been terribly busy with work (and sick with covid also), also I'm not particularly good at github, so for now that's all I can contribute.

@MishimaHaruna MishimaHaruna self-assigned this Jun 14, 2023
@MishimaHaruna MishimaHaruna marked this pull request as draft June 14, 2023 22:25
@MishimaHaruna MishimaHaruna removed this from the Release v2024.06 milestone Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:inprogress Issue is being worked on / the pull request is still a WIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants