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

Harpy shoes 1984 #2581

Conversation

Monotheonist
Copy link
Contributor

@Monotheonist Monotheonist commented Jan 2, 2025

About the PR

This simple two line change nukes any semblance of there having been shoes on harpies, by giving them the diona inventory template. Before you ask, no, this wasn't my idea, it was originally like this where they're being maintained currently (EE,) it just got changed because of a technical issue when it came here.

Why / Balance

One thing I've always noticed about Harpies here is that they were never intended to wear shoes. There are 2 to three pixels sticking out of the sprite when you look at it on the server, and I've been unable to play harpy after noticing this. It pisses me off so much. Of course, this is the nuclear option; there's another solution, like making a displacement map, which might make more sense, but I feel like this also adds a unique and kind of evil downside of having to make them be like dionae and suffer with glass shards and other things, so I am doing this first to provoke conversation of if this is too much.

Technical details

Two line yaml change, gives them the template id of Diona, which is shoeless. This is to prevent the extra bloat of bringing back the digitigrade template yaml, but as a side effect, anything that happens to that template will reflect on harpies, too. Just something for Deltanedas to note.

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Changelog

🆑

  • tweak: Harpies can no longer wear shoes. The moths ate them all.

@Monotheonist Monotheonist requested a review from a team as a code owner January 2, 2025 08:14
@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files size/XS Under 16 lines labels Jan 2, 2025
@gluesniffler
Copy link

If you're gonna take away their ability to wear shoes, you need to also give them a buff in return. The reason the shoes thing got reverted by Milon was twofold:

  1. It broke a few things because of how Adei ported it.
  2. There were a lot of complaints from what little harpy players we have since its a nerf to an already weak species.

Additionally, not being able to wear shoes entails a hard weakness to anything involving slips, with not much to show for that drawback.

@Monotheonist
Copy link
Contributor Author

Monotheonist commented Jan 2, 2025

I'm fairly sure this need for a buff was negated by EE by making them immune to steptrigger things like glass (thereby giving them noslips,) which would suddenly make them absurdly OP for security players.
I'm willing to give them a buff, but I don't have any ideas, so if anyone could suggest something to go along with this that would actually improve the harpy experience (cold resistance, if they don't have that, or like, faster punch speed) I will gladly buff them (or, once again get the displacement map thing done.)

@gluesniffler
Copy link

The EE steptrigger "buff" was accompanied with a selective refactor for the system, which has them set to be immune only to landmines, glass, and mice traps.

So you have a couple routes from here, either:

@Lyndomen
Copy link
Contributor

Lyndomen commented Jan 2, 2025

I haven't consulted Direction, but my personal opinion is that a displacement map would be best for this situation.

I am not the biggest fan of steptrigger immune, it doesn't make the most in game sense.

@Monotheonist
Copy link
Contributor Author

This is GREAT feedback, thank you guys for contributing to it (as well as people I asked on discord about this!!) From my understanding displacement maps does seem to be the way to go, so I'm gonna close this and open up a seperate and more comprehensive PR at some point.

@Monotheonist Monotheonist deleted the remove-harpys-shoes-like-a-maniac branch January 3, 2025 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files S: Needs Review size/XS Under 16 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants