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

ABILITY_SYMBIOSIS #2117

Merged
merged 11 commits into from
Sep 24, 2022
Merged

ABILITY_SYMBIOSIS #2117

merged 11 commits into from
Sep 24, 2022

Conversation

AgustinGDLV
Copy link
Collaborator

@AgustinGDLV AgustinGDLV commented May 6, 2022

Addresses issue #346. Lots of testing done, some of it below, a lot of it recorded and some of those recordings below, too. All major issues so far resolved! I may go back and try to combine trysymbiosis with the new MOVEEND_EFFECT which was implemented after to fix some other issues, but I'm not sure how simple that would be. If it would be best to merge the two, let me know.

Symbiosis Showcase

Issues:

  • Gem interactions unknown, but presumably they function like Focus Sash and the like.
  • Bestow gets cut off early when Symbiosis procs.
  • If an ally eats a foe's berry with Bug Bite but it has no effect (e.g. Sitrus Berry on max HP), Symbiosis does not proc.
  • Fling is completely busted.
  • Eject Button and Eject Pack incorrectly trigger Symbiosis. Need to test in IE, wasn't working because of debug mode.
  • Message bug with damage-reducing berries (new item is received before message appears).

Tested Interactions:

  • Red Card
  • Focus Sash
  • Bug Bite (all cases!)
  • Knock Off
  • Incinerate
  • Air Balloon
  • Trick
  • Eject Button / Pack
  • Gems
  • Unexchangeable Items
  • Fling
  • Natural Gift
  • Damage-reducing Berries
  • Multiple Symbiosis abilities on side
  • Gem behavior differences in Gen 6 and Gen 7+

Extra Demos:

show:

Bug Bite:
Symbiosis Bug Bite (No Effect Item)
Symbiosis Bug Bite

Focus Sash:
Symbiosis Focus Sash
Incinerate:
Symbiosis Incinerate
Knock Off:
Symbiosis Knock Off
Red Card:
Symbiosis Red Card
Gen 6 Gems:
Symbiosis Gem + Life Orb (Gen 6)
Gen 7 Gems:
Symbios Gem + Life Orb (Gen 7)
Eject Pack:
Symbiosis Eject Pack

Discord Contact Info

Agustin#1522

@AgustinGDLV AgustinGDLV changed the title functional but not perfect implementation ABILITY_SYMBIOSIS May 6, 2022
@AgustinGDLV
Copy link
Collaborator Author

AgustinGDLV commented May 7, 2022

New commit. Other than some major issues with Fling, Symbiosis is accurate and functional in an isolated BE branch. More testing required with items from IE.

@AgustinGDLV AgustinGDLV marked this pull request as ready for review May 7, 2022 05:31
@AgustinGDLV
Copy link
Collaborator Author

Final commit? Tested most interactions and whatever edge cases I could think of. Gems and damage-reducing berries now function correctly (using a new MOVEEND effect), as do Eject Pack and Eject Button. Added battle config for gem function. I think this is complete!

@ghoulslash
Copy link
Collaborator

fix conflicts

@AsparagusEduardo AsparagusEduardo changed the base branch from battle_engine to master August 24, 2022 04:41
@AsparagusEduardo AsparagusEduardo changed the base branch from master to dev September 10, 2022 01:34
@AsparagusEduardo AsparagusEduardo added category: battle-mechanic Pertains to battle mechanics and removed category: battle-mechanic Pertains to battle mechanics labels Sep 16, 2022
@AsparagusEduardo AsparagusEduardo changed the base branch from dev to dev_1.x September 16, 2022 17:32
src/battle_script_commands.c Outdated Show resolved Hide resolved
src/battle_script_commands.c Outdated Show resolved Hide resolved
src/battle_script_commands.c Outdated Show resolved Hide resolved
@AsparagusEduardo AsparagusEduardo merged commit 0f743c4 into rh-hideout:dev Sep 24, 2022
@AsparagusEduardo AsparagusEduardo mentioned this pull request Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: battle-mechanic Pertains to battle mechanics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants