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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

The house edge now has a new secondary attack. #82946

Closed
wants to merge 1 commit into from

Conversation

ArcaneMusic
Copy link
Contributor

About The Pull Request

2024-04-28.19-13-23.mp4

The house edge weapon, a weapon made from the v8 engine from the black market, now has a slightly less awful secondary attack. For recap on how it already works, the house edge, while wielded, gives itself a charge for every successful melee hit that it lands. Un-wielding or dropping the house edge will reset it's charges. Each charge is visibly seen on the icon_state as a plume of fire coming out of the exhaust.

The old system had the house edge effectively launch the wielder as a dangerous tackle at it's target. This had a higher likelihood of just killing the wielder than it did to really provide any real advantage in a fight, against one or multiple targets, either way. At best, it stunned both targets, and dropped the house edge to the ground.

The new special attack instead launches the weapon as a throw from the wielder's hand, and then after 0.5-1.5 seconds, teleports the wielder to the sword, and puts it back in hand. It's range, as well as it's teleport delay is based on the number of fire charges, making it a particularly tricky weapon to use correctly. That said, it can allow for the wielder to effectively teleport out of a bad situation and buy some time for either an escape for an equipment swap as necessary. Using it this way also poses the potential risk that someone could catch the house edge, in which case the teleport will be cancelled. Plus it leaves behind a little fire teleport effect in the wielder's old location.

Why It's Good For The Game

The original secondary effect was... godawful. I was made aware of it in the original PR, I've known it's basically a worse to use the secondary effect than to just use it as a discount fire axe. This helps to provide a useful buff to a rare weapon that provides some cool utility.

Having weapons that provide any kind of mobility is pretty rare, or blatantly comes with a higher bracket of power (ninja blade for example, mobility wands, etc). Having something like this that can live station-side, even if gated by both economy and rarity, would make this very cool and live in it's own unique niche.

Changelog

馃啈
balance: The house edge, a rare blade made from an ancient V8 engine, now has a new special attack, activated by right clicking while wielded after landing one ore more successful hits.
/:cl:

@tgstation-server tgstation-server added Sprites A bikeshed full of soulless bikes. Balance Changes to functionality that modifies how effective certain methods are at powergaming labels Apr 28, 2024
@Jacquerel
Copy link
Contributor

Maybe asking a lot but given that the throw essentially drops it instantly on the destination tile it'd be nice to have some kind of trail and landing effect, maybe for the "following" teleport too.

@ArcaneMusic ArcaneMusic added the Feature Exposes new bugs in interesting ways label Apr 28, 2024
@carlarctg
Copy link
Contributor

I like the idea of a House Edge buff, but I don't like this implementation. The original is grounded in (cartoon) physics and fits a theme, it looks like an engine, you charge with it like an engine. This one.... you throw it? For some reason? And then you somehow teleport to it? Huh??
There are ways to buff the item that don't turn it into magic. It's built from a V8 engine, not magic unobtanium phlebutonium bluespace plasma or whatever.

@Jacquerel
Copy link
Contributor

Yeah I guess that is also a reasonably good point, this just doesn't look like a weapon which would teleport you
Maybe a charge which lets you phase through players without knocking either over and leaves some fire behind you would be more fitting 馃

@necromanceranne
Copy link
Contributor

necromanceranne commented Apr 29, 2024

I mean, not for nothing, but I think the charge is the much cooler ability if only it didn't punish you for collision. I think it should do the opposite of what it does currently. You get a cool big hit if you land a charge on someone, kinda like the grappling hooks kick. That's...what I thought the charge would do and was really surprised when it was decided otherwise. It just felt appropriate.

And, frankly, if you want to increase the possible design space for this, you could probably make it only lose charges after a delay from unequip/unwield, if only because I think it will at least alleviate how severely punishing the weapon can be at times, and lets you do more interesting things with unwielding the weapon on purpose.

...

If it did work like that...Boomerang component at max charges?

Edit: Hey, have you considered adding an anomaly core to the recipe to give it as much power as you think is appropriate?

@YakumoChen
Copy link
Contributor

YakumoChen commented Apr 29, 2024

Change the charge attack to make them move like a tram, knocking away anyone hit

@Admiral-Gigaton
Copy link

Skipping over other comments for now, what happens if the sword embeds on an enemy? (This could be used for funny/cool interactions, not unlike using recall spell when an item embeds)

And to address any extra things, such as comments about this not making sense - similar to another comment, you could add a trail from where the player is to where the item is, like the chef meat hook, and that trail being a... chain? Cable? (Likely this would change recipe for the item)

@Livrah
Copy link

Livrah commented Apr 29, 2024

Without any animation this looks weird and doesn't fit the theme if you ask me.

Copy link
Contributor

@MrMelbert MrMelbert left a comment

Choose a reason for hiding this comment

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

Should def. make it a "charge" instead of a teleport

reset_charges(on_dash = TRUE)
return FALSE
new /obj/effect/temp_visual/fire_fizzle(get_turf(user))
user.forceMove(drop_location())
Copy link
Contributor

Choose a reason for hiding this comment

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

Also you're using forcemove rather than doteleport which means this can bypass noteleport stuff

@MrMelbert MrMelbert marked this pull request as draft April 29, 2024 20:24
@YuliceTheDruid
Copy link

The sword's a DMC reference, so a fiery charge would be more fitting thematically, yeah. Surely there'd be a way to give them some sort of unstoppable or unstunnable status while they're doing it, along with the fire trail

@Loafin34
Copy link

a cool idea for the charge is you wind it up after getting full charge, and get rush gland ability but can't stop moving until you hit something. once somethin is hit it takes 50 damage or you charge into a wall and take 10 damage and the sword is stuck in the wall, requiring you to grab at it to pull it out

Copy link
Contributor

github-actions bot commented May 8, 2024

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Even the uncaring universe rejects you, why even go on label May 8, 2024
@ArcaneMusic
Copy link
Contributor Author

I'll come back to this some other time I'm currently overloaded with RL work to give this the creative energy it needs for this 5 seconds

@ArcaneMusic ArcaneMusic closed this May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways Sprites A bikeshed full of soulless bikes. Stale Even the uncaring universe rejects you, why even go on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet