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

Projectilesimprovement #782

Merged
merged 10 commits into from
Jan 31, 2024
Merged

Conversation

Kapu1178
Copy link
Member

About The Pull Request

Why It's Good For The Game

Changelog

🆑 TiviPlus, LemonInTheDark, ChungusGamer666, Watermelon914, vinylspiders, tyeagg, Kapu1178
add: Added new particle effects for being on fire or wet.
add: Added new particle effects for firing ballistic weapons.
add: Added new particle effects for bullets impacting surfaces.
add: Bullets are now illuminated.
add: Bullets leave behind impact holes on most surfaces.
/:cl:

Kapu1178 and others added 10 commits January 27, 2024 17:24
Demo: https://streamable.com/wnj3mf

Features:
- Full support for most gradients/vectors/numbers/generators/transforms
( I might have forgotten some of the more esoteric ones)
- A "tutorial" section that explains the different rand/generation types
and how physics works with pictures
- Button for viewing what each var does
- Selecting a particle type to set immediately
- The generator types use defines now

Not included:
Color matrix support for color generators (I'm sorry but hell no)

Special thanks to @jlsnow301 for explaining js things to me

Making cool stuf

:cl:
refactor: Added a particle editor to VV dropdown which can be used by
coders and admins to edit particle values on the fly easily.
/:cl:

Co-authored-by: TiviPlus <[email protected]>
It isn't really an issue now, but these will be used more in future, so
let's start off strong.

There's a lot of work going on here that doesn't really need to be
happening, mostly off not knowing a trickTM.

Biggest one is vis_locs and vis_contents are linked lists, being in one
requires being in another. Atoms clear out their vis_locs on Destroy, so
we do not need to "own" references to things that have us in their
vis_contents.

This combined with knowing our old loc's loc off Moved made the use of
weakrefs here unneeded. Similarly, atoms inside atom movables qdel on
when the upper layer is deleted, so most cases of the qdeleting signal
were unneeded.

Also, we only cared about movement if we were an item (speaking of
which, I swapped out the isitem stuff with a flag that gets passed into
the new() call)

Speed
Mobs on fire and wet mobs get cool particle effects

https://user-images.githubusercontent.com/82850673/230459326-184b5553-17db-49df-997f-8db4c87df601.mp4
fire particles (big fire)
small fire just reduces the spawn rate of particles to 20% of this

https://user-images.githubusercontent.com/82850673/230468680-b831e557-bb0a-4489-8aef-4fefdfc14e43.mp4
wet particles

sick

:cl:
add: Mobs on fire get particle effects (it looks sick)
add: Wet mobs get particle effects (it looks sick)
/:cl:
Particle holders are placed into the contents of the thing they're
showing the particles off of. This creates problems as some items can
drop all their contents to the ground or, in the case of storage items,
use their contents in some way

Fixes particle holders falling out of items.

:cl:
fix: Fixed particles sometimes being left behind when an object drops
all of its contents whilst having a particle active.
/:cl:

---------

Co-authored-by: Watermelon914 <[email protected]>
## About The Pull Request

The harddel hunt continues...


![image](https://github.com/tgstation/tgstation/assets/13398309/19fb7446-e667-4e4f-9ab5-b80e02541592)

This stupid mythril coin has come up a bunch of times in our CI recently
and it is starting to annoy me. This should fix it hopefully.

Attempts to get rid of a potential source of harddels in
particle_holder.dm

## Why It's Good For The Game


## Changelog

:cl:
fix: fixes a harddel in particle holders
/:cl:
@QuacksQ
Copy link

QuacksQ commented Jan 28, 2024

this one seems familiar

@Kapu1178 Kapu1178 merged commit 7d53106 into DaedalusDock:master Jan 31, 2024
12 checks passed
@Kapu1178 Kapu1178 deleted the projectilesimprovement branch January 31, 2024 09:17
QuacksQ added a commit to Artea-Station/Artea-Station-Server that referenced this pull request Oct 5, 2024
## About The Pull Request

This pr yoinks the bullet sprite from Mojave Sun and the idea to make
them light up from DD, credit goes to the original Pr and codebase
respectively:
- DaedalusDock/daedalusdock#782
-
https://github.com/Mojave-Sun/mojave-sun-13/blob/master/mojave/icons/objects/projectiles/projectiles.dmi

The sprite:

![image](https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/ee2122a4-936b-4b76-a250-8dd2760cf72f)


## How Does This Help ***Gameplay***?
Cooler visuals to look at are nice while gaming :)

## How Does This Help ***Roleplay***?
Minimal impact on RP

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

## OLD


https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/139032f4-0cf7-4c16-885e-0ba030120c55

## NEW


https://github.com/Artea-Station/Artea-Station-Server/assets/79924768/cd060115-e57e-4c1a-9594-ff0ef68e2315


</details>

## Changelog

:cl:
imageadd: Bullets now have a new sprite!
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants