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

Hollow Knight: Add additional DeathLink option and add ExtraPlatforms option. #2545

Merged
merged 2 commits into from
Dec 17, 2023

Conversation

ThePhar
Copy link
Member

@ThePhar ThePhar commented Dec 1, 2023

What is this fixing or adding?

ArchipelagoMW-HollowKnight/Archipelago.HollowKnight#157 - Splits DeathLink options up to allow more configuration of behavior on death without needing to program all possible combinations in existing DeathLink option.

ArchipelagoMW-HollowKnight/Archipelago.HollowKnight#58 - Also adds an "ExtraPlatforms" option to create additional platforms in Hallownest to make moving around easier.

@BadMagic100 - Leaving in Draft until client-side can be updated and verified they work with these changes.

How was this tested?

No logic changes were made, but verified output on WebHost. Awaiting client-side tests before pulling out of Draft state.

If this makes graphical changes, please attach screenshots.

N/A

@ThePhar ThePhar added the is: enhancement Issues requesting new features or pull requests implementing new features. label Dec 1, 2023
@ThePhar ThePhar changed the title Hollow Knight: Break DeathLink options up and add ExtraPlatforms option. Hollow Knight: Add additional DeathLink option and add ExtraPlatforms option. Dec 1, 2023
Copy link
Collaborator

@BadMagic100 BadMagic100 left a comment

Choose a reason for hiding this comment

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

Will work on the client side changes this weekend and update here once they are merged



class DeathLinkBreaksFragileCharms(Toggle):
"""Sets if fragile charms break when you are killed by a DeathLink. All other deaths still break fragile charms.
Copy link
Collaborator

Choose a reason for hiding this comment

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

As noted by folks on discord it's probably slightly more accurate to say something like "fragile charms will still be broken by your own deaths as usual" to cover for cases where your own deaths actually will not break the charm

Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually on second thought, maybe removing the "all other deaths still break fragile charms" is better for consistency, or should be implied by the "when you are killed by a deathlink." That would be consistent with the wording on the shade option as well



class DeathLinkShade(Choice):
"""Sets whether to create a shade when you are killed by a DeathLink and how to handle your existing shade, if any.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Similar to the fragile charms setting a caveat that your own deaths will create a shade and remove any existing shade as normal would be good

@BadMagic100
Copy link
Collaborator

Client side changes are good to go as of ArchipelagoMW-HollowKnight/Archipelago.HollowKnight#203

@ThePhar
Copy link
Member Author

ThePhar commented Dec 5, 2023

Personally, I don't count those as actual deaths anyway if it doesn't drop a shade. E.g. dream fights.

@ThePhar ThePhar marked this pull request as ready for review December 5, 2023 17:35
@BadMagic100
Copy link
Collaborator

BadMagic100 commented Dec 6, 2023

A deathlink is still sent even in cases you normally wouldn't leave a shade or end steel soul. In any case I think the comment regarding shades probably still stands.

Maybe the better way to phrase it is to put somewhere that "the penalties of your own deaths inflicted on you are unmodified," either on the deathlink toggle or with respect to each option (e.g. "whether or not a shade spawns from your own deaths is unmodified")

@BadMagic100 BadMagic100 closed this Dec 6, 2023
@BadMagic100 BadMagic100 reopened this Dec 6, 2023
@BadMagic100
Copy link
Collaborator

(closing this was unintentional, the mobile UI had it out for me)

@ThePhar
Copy link
Member Author

ThePhar commented Dec 12, 2023

Made some more verbiage tweaks.

@ThePhar ThePhar merged commit a549af8 into ArchipelagoMW:main Dec 17, 2023
12 checks passed
@ThePhar ThePhar deleted the new_hk_options branch December 17, 2023 16:11
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement Issues requesting new features or pull requests implementing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants