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

[Balance] Ability-ignoring effects no longer ignore the source's own Abilities #3556

Merged
merged 7 commits into from
Sep 23, 2024

Conversation

innerthunder
Copy link
Collaborator

@innerthunder innerthunder commented Aug 15, 2024

What are the changes?

This change makes it so that if a Pokemon has an ability or uses a move that ignores ability effects, those ability-ignoring effects do not apply to the user's own abilities. For example, if a Pokemon has Mold Breaker as an ability and Contrary as a passive ability, it using Overheat would now raise the user's Sp. Atk instead of ignoring Contrary and lowering it.

Why am I doing these changes?

This change was requested by the Balance team, fitting in line with previous changes to Neutralizing Gas (#1662 ). Also, this interaction has been reported as a bug before in the Discord, which indicates that players may find this interaction unintuitive as it stands.

What did change?

  • field/arena: New field Arena.ignoreAbilitySource tracks the source of active ability-ignoring effects by battler index, and is set within setIgnoreAbilities.
  • field/pokemon: Pokemon.canApplyAbility now checks that arena.ignoreAbilitySource does not match the Pokemon's battler index before ignoring abilities' effects.
  • phases: Small adjustments reflecting the changes to Arena.setIgnoreAbilities()

Screenshots/Videos

Clanging Scales from a Pokemon with Mold Breaker + Contrary after the changes:

PokeRogue.-.Google.Chrome.2024-08-14.22-03-16.mp4

How to test the changes?

tested via overrides

Checklist

  • I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I considered writing automated tests for the issue?
  • Have I tested the changes (manually)?
    • Are all unit tests still passing? (npm run test)
  • Are the changes visual?
    • Have I provided screenshots/videos of the changes?

@innerthunder innerthunder added Move Affects a move Ability Affects an ability labels Aug 15, 2024
src/field/arena.ts Outdated Show resolved Hide resolved
Co-authored-by: NightKev <34855794+DayKev@users.noreply.github.com>
@Xavion3
Copy link
Collaborator

Xavion3 commented Aug 15, 2024

It's unintuitive partially because Neutralising gas is bugged to not negate abilities like it should which creates an inconsistency, we can easily fix that by just reverting the change which broke it.

Also this is not a quality of life change, this is a significant change in how an ability works with significant balance and gameplay ramifications. This is even a nerf under some circumstances.

The bug report was also thrown off by the text of mold breaker not lining up with the effect, but this change does not solve that problem so that's not really a good excuse either. Unless we want to make it not ignore stuff like lightning rod.

Copy link
Collaborator

@Xavion3 Xavion3 left a comment

Choose a reason for hiding this comment

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

As above, this currently fails to actually solve the problem laid out in the reasons for this PR.

src/field/arena.ts Outdated Show resolved Hide resolved
@DayKev
Copy link
Collaborator

DayKev commented Aug 16, 2024

It's unintuitive partially because Neutralising gas is bugged to not negate abilities like it should which creates an inconsistency, we can easily fix that by just reverting the change which broke it.

Also this is not a quality of life change, this is a significant change in how an ability works with significant balance and gameplay ramifications. This is even a nerf under some circumstances.

The bug report was also thrown off by the text of mold breaker not lining up with the effect, but this change does not solve that problem so that's not really a good excuse either. Unless we want to make it not ignore stuff like lightning rod.

So we need balance team comments then.

@innerthunder innerthunder added the Balance Team Specifically a balance team change label Aug 17, 2024
@innerthunder
Copy link
Collaborator Author

innerthunder commented Sep 9, 2024

Closing this as there is at least some evidence (via Sunsteel Strike/Moongeist Beam) that ability-ignoring effects apply to the user's ability in mainline; it's just that these effects are never available alongside ignorable abilities (hence Shadow Shield, Full Metal Body, and Mold Breaker itself not being ignorable).

I may reopen this if the Game Design/Balance teams decide custom implementation is needed to remove potential anti-synergy between a Pokemon's passive and ability, but as it stands I'd say the current implementation is more accurate to mainline.

EDIT: nvm I guess lol

@damocleas damocleas reopened this Sep 11, 2024
@damocleas damocleas requested a review from a team as a code owner September 11, 2024 00:27
@innerthunder innerthunder changed the title [QoL] Ability-ignoring effects no longer ignore the source's own Abilities [Balance] Ability-ignoring effects no longer ignore the source's own Abilities Sep 11, 2024
@innerthunder innerthunder dismissed Xavion3’s stale review September 12, 2024 01:39

AFAIK, this was overruled by the Balance Team. Yes, there's a good chance this diverges from mainline behavior, but it's in the interest of making Mold Breaker more flexible with passive/fusion combos

@flx-sta flx-sta merged commit f2fe430 into pagefaultgames:beta Sep 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ability Affects an ability Balance Team Specifically a balance team change Move Affects a move
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants