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

[Port] More cool visuals for ballistics #520

Merged
merged 9 commits into from
Jan 21, 2024

Conversation

QuacksQ
Copy link
Contributor

@QuacksQ QuacksQ commented Jan 8, 2024

About The Pull Request

Holy shit this is cool as fuck please merge this 🙏
builds on #455 and #460.

This PR adds a buncha cool particle effects for ballistic guns and new explosion visuals because the old ones suck.
Code and everything is originally from TGMC so this is a port of the following PR's:

Code was borrowed from a Mojave Sun PR because I dont know how to code lmao.

This looks so fucking cool combined with the movable physics on the shell casings dude. imagine how cool gunfights on the station are gonna look after this.

How Does This Help Gameplay?

It helps gameplay because it looks cool as fuck just look at it dude.

How Does This Help Roleplay?

More Immersion because it looks alot more realistic? Idk dude sue me at this point I just want cooler gun visuals and this is gonna help my immersion alot :)

Proof of Testing

Screenshots/Videos

New explosions

2024-01-08.00-24-39.mp4

New gun visuals

2024-01-08.00-27-38.mp4

Changelog

🆑
qol: Ballistics now emit smoke when fired and bullet impacts also have proper smoke added to them.
qol: Explosions look much cooler and have many more particles to them!
imageadd: Added a newer and much cooler explosion sprite (grabbed from Mojave Sun)
/:cl:

@github-actions github-actions bot added Administration Someone messed with admin tools. Balance Also known as "I died to this, let's make it bad". Config Update Oh boy, Rimi has to touch the server config when this gets merged. Feature How I broke the server in one easy step! Fix Cleans up someone else's code from 5 years ago. Grammar and Formatting Not all of us have formal education in english. Quality of Life Also known as "I deleted a ton of other things and made one particular thing the only way". Refactor Also known as "I added a ton of new code that no one will ever use, while failing CI randomly". Removal "What is soul? Are you a cult?" Sounds Also known as "I added a background rumble to lavaland and gave everyone headaches". Sprites I like pretty things. labels Jan 8, 2024
@QuacksQ
Copy link
Contributor Author

QuacksQ commented Jan 8, 2024

Don't merge this yet, explosions are scuffed rn and I need to figure out why

@RimiNosha RimiNosha marked this pull request as draft January 8, 2024 11:57
@QuacksQ QuacksQ marked this pull request as ready for review January 8, 2024 23:08
@QuacksQ
Copy link
Contributor Author

QuacksQ commented Jan 8, 2024

Ready for review now

Copy link
Contributor

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 "Anyone home?" label Jan 16, 2024
Copy link
Contributor

@RimiNosha RimiNosha left a comment

Choose a reason for hiding this comment

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

Stylistic stuff, the code itself looks perfectly fine.

code/datums/elements/debris.dm Outdated Show resolved Hide resolved
code/game/objects/effects/temporary_visuals/explosions.dm Outdated Show resolved Hide resolved
code/datums/elements/debris.dm Outdated Show resolved Hide resolved
code/__DEFINES/particles.dm Show resolved Hide resolved
@RimiNosha RimiNosha removed Stale "Anyone home?" Administration Someone messed with admin tools. Config Update Oh boy, Rimi has to touch the server config when this gets merged. Balance Also known as "I died to this, let's make it bad". labels Jan 16, 2024
@RimiNosha RimiNosha removed Fix Cleans up someone else's code from 5 years ago. Grammar and Formatting Not all of us have formal education in english. Removal "What is soul? Are you a cult?" Sounds Also known as "I added a background rumble to lavaland and gave everyone headaches". Refactor Also known as "I added a ton of new code that no one will ever use, while failing CI randomly". labels Jan 16, 2024
@RimiNosha RimiNosha merged commit 6183232 into Artea-Station:master Jan 21, 2024
16 of 17 checks passed
Artea-Station-Bot added a commit that referenced this pull request Jan 21, 2024
@QuacksQ QuacksQ deleted the cool-shit branch January 24, 2024 13:52
@QuacksQ QuacksQ mentioned this pull request Jun 1, 2024
QuacksQ added a commit that referenced this pull request Oct 5, 2024
## About The Pull Request
This pr is a port of: 
- Mojave-Sun/mojave-sun-13#430

This PR significantly improves recoil (screenshake when shooting guns)
because our current recoil sucks ass. Instead of a simple screenshake
its now based on the direction you're shooting in. I have also gone
ahead and added recoil values to the guns we have in-game right now.

Ballistic guns have a recoil value of `1`
Energy guns have a recoil value of `0.2`
Disablers and the Miniature energy gun have a recoil value of because
they're tiny `0.1`

This is yet another expansion of the guns and other cool shit pr's like:
#520 #455 and #460.

## How Does This Help ***Gameplay***?
Feedback to in game actions is always good and guns not having recoil
when being shot is just weird.

## How Does This Help ***Roleplay***?
Immersion probably

## Proof of Testing
<details>
<summary>Screenshots/Videos</summary> <!-- Leave the line after this one
empty. Embeds like breaking if you don't -->

## Old Recoil


https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/3a483a6d-a98f-4ae7-a34e-f4b7750ac7c1

## New Recoil


https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/6bcefbc4-3070-44f6-90ba-45638c99b553

## New recoil on a disabler


https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/7ac9e71a-eebb-4be8-afc3-a971c3d2d013



</details>

## Changelog
:cl:
qol: Recoil is alot less awful now and is based on the direction you're
shooting in.
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature How I broke the server in one easy step! Quality of Life Also known as "I deleted a ton of other things and made one particular thing the only way". Sprites I like pretty things.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants