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

Move boss kill logic into CanKillEnemy #5000

Merged
merged 2 commits into from
Feb 25, 2025

Conversation

serprex
Copy link
Contributor

@serprex serprex commented Feb 5, 2025

Added hammer as being able to defeat Gohma, King Dodongo, Morpha, & Twinrova

split out from #4866

Build Artifacts

@serprex serprex force-pushed the can-kill-boss branch 5 times, most recently from d0634db to 57e3139 Compare February 5, 2025 01:31
Copy link
Contributor

@aMannus aMannus left a comment

Choose a reason for hiding this comment

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

I know you're simply moving these, but this feels the opportune moment to fix the logic in these as well. All of these are confirmed by Spoon by fighting the bosses ingame.

Comment on lines +685 to +687
return HasBossSoul(RG_PHANTOM_GANON_SOUL) &&
(CanUse(RG_KOKIRI_SWORD) || CanUse(RG_MASTER_SWORD) || CanUse(RG_BIGGORON_SWORD)) &&
(CanUse(RG_HOOKSHOT) || CanUse(RG_FAIRY_BOW) || CanUse(RG_FAIRY_SLINGSHOT));
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of the separate swords, this can use CanJumpslashExceptHammer().

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't think we want logic to expect fighting phantom ganon with sticks

Comment on lines +691 to +692
return HasBossSoul(RG_MORPHA_SOUL) && CanUse(RG_HOOKSHOT) &&
(CanUse(RG_KOKIRI_SWORD) || CanUse(RG_MASTER_SWORD) || CanUse(RG_BIGGORON_SWORD) || CanUse(RG_MEGATON_HAMMER));
Copy link
Contributor

Choose a reason for hiding this comment

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

This can use CanJumpslash().

Copy link
Contributor Author

@serprex serprex Feb 13, 2025

Choose a reason for hiding this comment

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

morpha has 20 health. Seems tight with sticks

Comment on lines +696 to +697
(CanUse(RG_KOKIRI_SWORD) || CanUse(RG_MASTER_SWORD) || CanUse(RG_BIGGORON_SWORD)) &&
(CanUse(RG_HOOKSHOT) || CanUse(RG_FAIRY_BOW) || CanUse(RG_FAIRY_SLINGSHOT) || ctx->GetTrickOption(RT_SHADOW_BONGO));
Copy link
Contributor

Choose a reason for hiding this comment

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

This can use CanJumpslashExceptHammer().

(CanUse(RG_HOOKSHOT) || CanUse(RG_FAIRY_BOW) || CanUse(RG_FAIRY_SLINGSHOT) || ctx->GetTrickOption(RT_SHADOW_BONGO));
case RE_TWINROVA:
return HasBossSoul(RG_TWINROVA_SOUL) && CanUse(RG_MIRROR_SHIELD) &&
(CanUse(RG_KOKIRI_SWORD) || CanUse(RG_MASTER_SWORD) || CanUse(RG_BIGGORON_SWORD));
Copy link
Contributor

Choose a reason for hiding this comment

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

Twinrova can be beaten by hammer so can use CanJumpslash().

Copy link
Contributor Author

@serprex serprex Feb 13, 2025

Choose a reason for hiding this comment

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

twinrova has 24 health, but will add hammer

@Pepe20129
Copy link
Contributor

If we're going to change boss logic in this PR, it's worth noting that the ganon logic is wrong, it doesn't account for the first phase where you don't have the master sword.

@serprex
Copy link
Contributor Author

serprex commented Feb 13, 2025

Added hammer kills, but for sticks it may be preferable to have a concept of being able to damage boss down with weapons that can't deliver kill before dealing killing blow with stick

@serprex serprex requested a review from aMannus February 15, 2025 22:53
@aMannus aMannus merged commit db41c65 into HarbourMasters:develop Feb 25, 2025
5 checks passed
@serprex serprex mentioned this pull request Feb 25, 2025
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