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

Holoflare pistol #2338

Merged

Conversation

ErhardSteinhauer
Copy link
Contributor

@ErhardSteinhauer ErhardSteinhauer commented Oct 26, 2024

About the PR

Adds a selfrecharging energy pistol that shoots holographic light sources. Has three modes of fire: cyan, red and yellow. The projectiles, holoflares, don't deal any damage and don't set anything on fire, last for 15 minutes.

Why / Balance

I like the idea of flare guns, I believe that they can add to the atmosphere of the game, but actually using flare guns is a nightmare due to how inconvenient the whole process is. This PR tries to solve this issue.

How to test

Spawn in, spawn holoflare pistol or buy it from lesslethal vendomat.

Media

image

Requirements

Breaking changes

Changelog

🆑 erhardsteinhauer, whatston3

  • add: Adds a self recharging energy pistol that shoots holographic light sources. Available from LessLethal vendomat.

@ErhardSteinhauer ErhardSteinhauer marked this pull request as draft October 26, 2024 19:27
@ErhardSteinhauer ErhardSteinhauer marked this pull request as ready for review October 26, 2024 20:00
@ErhardSteinhauer ErhardSteinhauer changed the title Holoflarepistol Holoflare pistol Oct 26, 2024
@whatston3
Copy link
Contributor

whatston3 commented Oct 31, 2024

Pushed a suggestion onto whatston3:2024-10-31-holoflare-suggestions

  • Reduced flare length to 4 minutes from 15, consistent with the thrown emergency flares (up for debate, but I think it's more reasonable)
  • Changed the red color to #FF8080, used for emergency flares - now roughly as bright as the other flare colours
  • Used yellow/red magic ball sprites for flares where appropriate
  • Added firing mode enum values for red/cyan/yellow flares (naming might not be great, feel free to change, but the in-game sprite does update properly now)
  • Named the flares with their color (now they show up appropriately in the right click menu)
  • Minor grammar pass
  • Reduced some redundancy in the red/yellow flare definition

Really nice feature. Might even suggest removing the existing flare gun from the LessLethalVend.

A few pictures showing the new red flare colour:
image
image

@github-actions github-actions bot added the C# label Oct 31, 2024
Copy link
Contributor

@whatston3 whatston3 left a comment

Choose a reason for hiding this comment

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

Welp.

@ErhardSteinhauer
Copy link
Contributor Author

I buffed lifetime of a light source to 9 mins. Not sure how useful 4 min flare is. From experience of using emergency flares - they are not great, you have to change them pretty often. But you at least can carry them with you.

@ErhardSteinhauer
Copy link
Contributor Author

All of a sudden not sure if this thing is actually that useful: between long charge regeneration, light source you can't carry around and low lifetime of light source might end up not used at all. (Still better than a regular flare imo). Probably will have to adjust the values down the line.

@whatston3
Copy link
Contributor

My thoughts on this:
Since the flares recharge in fairly short order, a shorter lifespan isn't too penalizing. The existing flares are already pretty long lived, though you do lose the ability to carry them around (honestly, wouldn't care too much if it shot out disposable glowstick flares).

Between this and their larger light radius, those are the reasons you might want to still use flares, but they're still a pain. Even with a four minute lifespan, I imagine these are fine.

@Leander-0
Copy link
Contributor

prism
Dark souls prism stone but it is a flare i do love it, my only nitpick is that it needs another version but the color is random or RGB :^)

@ErhardSteinhauer
Copy link
Contributor Author

My thoughts on this: Since the flares recharge in fairly short order, a shorter lifespan isn't too penalizing. The existing flares are already pretty long lived, though you do lose the ability to carry them around (honestly, wouldn't care too much if it shot out disposable glowstick flares).

Between this and their larger light radius, those are the reasons you might want to still use flares, but they're still a pain. Even with a four minute lifespan, I imagine these are fine.

I recon it's mighty fine in a situation when you don't need to move too far from the light source, so good for looting wrecks and mining rocks. But it becomes less useful for illuminating abandoned installations on expeditions during the night time.

@ErhardSteinhauer
Copy link
Contributor Author

ErhardSteinhauer commented Oct 31, 2024

Dark souls prism stone but it is a flare i do love it, my only nitpick is that it needs another version but the color is random or RGB :^)

Yeah, probably could be added later. Would work fine with punk stuff I think.

@ErhardSteinhauer
Copy link
Contributor Author

Pushed a suggestion onto whatston3:2024-10-31-holoflare-suggestions

Almost forgot to say thank you for your additions, very nicely done.

@whatston3
Copy link
Contributor

I recon it's mighty fine in a situation when you don't need to move too far from the light source, so good for looting wrecks and mining rocks. But it becomes less useful for illuminating abandoned installations on expeditions during the night time.

There's likely a sweet spot in terms of number of charges, charge time, and despawn timer.
Funnily enough, DRG might have some useful lessons there.

Almost forgot to say thank you for your additions, very nicely done.

N-no you 👉👈

@ErhardSteinhauer
Copy link
Contributor Author

Funnily enough, DRG might have some useful lessons there.

Don't tell anyone, but charge replenish rate and number of charges were taken from DRG...

@dvir001
Copy link
Contributor

dvir001 commented Nov 6, 2024

Have to lower this to like 4 minutes since the upstream merge going to make it a pain with the light issues.

@dvir001 dvir001 merged commit 52208e2 into new-frontiers-14:master Nov 6, 2024
14 checks passed
FrontierATC added a commit that referenced this pull request Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants