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

Gens 4-9: Remove interaction between Natural Gift and Berry Juice/Berry Sweet #593

Merged
merged 4 commits into from
Feb 14, 2024

Conversation

shrianshChari
Copy link
Contributor

Currently, when a Pokemon uses Natural Gift while holding a Berry Juice, it is treated as a 1 BP Normal-type move. This will now accurately display that a Pokemon using Natural Gift while holding a Berry Juice will deal 0 damage.

Screenshot from 2023-12-26 15-57-44

@shrianshChari
Copy link
Contributor Author

Bumping.

@ShivaD173
Copy link
Contributor

This fails to take into account Berry Sweet, which also has the same wrong interaction. An easier approach than this I think would be to just change items.ts:400 from 1 to 0bp.

@shrianshChari shrianshChari changed the title Gens 4-9: Remove interaction between Natural Gift and Berry Juice Gens 4-9: Remove interaction between Natural Gift and Berry Juice/Berry Sweet Jan 12, 2024
@shrianshChari
Copy link
Contributor Author

An easier approach than this I think would be to just change items.ts:400 from 1 to 0bp.

The reason why I didn't want to do this was that the text that would display would be inconsistent with other items that don't interact with Natural Gift. Take an Abomasnow holding an Icy Rock:

Abomasnow Natural Gift vs. 168 HP Abomasnow: 0-0 (0 - 0%) -- possibly the worst move ever
image

To highlight that Berry Juice and Berry Sweet don't interact at all with Natural Gift, we want to ensure the text isn't different so that it isn't confusing to people who are familiar with how Natural Gift appears when using a berry.

This is what it would look like when changing the bp from 1 to 0:

Berry Juice Abomasnow Natural Gift (Normal) vs. 168 HP Abomasnow: 0-0 (0 - 0%) -- possibly the worst move ever
image

And this is what it would look like when explicitly telling Natural Gift to ignore Berry Juice and Berry Sweet:

Abomasnow Natural Gift vs. 168 HP Abomasnow: 0-0 (0 - 0%) -- possibly the worst move ever
image

As you can see, the second example is more in line with how Natural Gift interacts with other items, and I believe it is preferable to treat Berry Juice and Berry Sweet this way.

calc/src/mechanics/gen4.ts Outdated Show resolved Hide resolved
calc/src/mechanics/gen56.ts Outdated Show resolved Hide resolved
calc/src/mechanics/gen789.ts Outdated Show resolved Hide resolved
calc/src/mechanics/gen789.ts Outdated Show resolved Hide resolved
@thejetou thejetou merged commit c877bc7 into smogon:master Feb 14, 2024
2 checks passed
@thejetou
Copy link
Collaborator

Thanks!

@shrianshChari shrianshChari deleted the natural-gift-fix branch February 14, 2024 21:18
Celestia74 pushed a commit to Celestia74/eipp-calc that referenced this pull request Oct 17, 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.

3 participants