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

Fireplaces are warm, books are burnable, burn papers with fireplaces #32087

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Plykiya
Copy link
Contributor

@Plykiya Plykiya commented Sep 11, 2024

About the PR

  • Fireplaces and bonfires will now passively heat the air around them to 305K (31.5C). Feeling cold? Come stop by the library.

  • Fireplaces and bonfires are now treated as constantly ignited ignition sources. Plasma leak in the library? Oh... unfortunate.

  • Books are now burnable in the same way that paper is. This does not apply to guidebooks.

  • Flammable objects now have an AfterInteract event. When using a flammable object on something else that is hot, like a lighter or fireplace, it will now ignite.

Previously, you were only capable of lighting something by using the lit object on the paper. This didn't work with fireplaces, because you don't hold a fireplace in your hand. So this behavior was added to work in reverse. Clicking on a fireplace does not ignite yourself with this added behavior.

Why / Balance

I want my fireplace to warm me up. I want to roleplay burning my books in my fireplace for warmth when I'm dying of cold.

Technical details

  • Added the RadiantTemperatureComponent/System. This will actively heat or cool down the air on the tile the entity is until it reaches the specified goal temperature. Whether it heats or cools is determined by the set rate of change.

  • Bonfires and fireplaces use the RadiantTemperature component and heat air to a comfortable 305K (heat damage is taken at 325K).

  • FlammableSystem's AfterInteractEvent is more or less just the copy/paste of what happens when you use the lit object on the flammable one, but in reverse with the targets.

  • Books more or less received a copy and paste of whatever made paper flammable.

Media

2024-09-11.13-14-52.mp4
2024-09-11.21-22-56.mp4

Atmos Device Update

2024-09-16.23-37-53.mp4

Requirements

Breaking changes

Changelog

🆑

  • add: Fireplaces and bonfires will now keep you warm.
  • add: Flammable objects can now be ignited by using them to interact with an ignition source, for example burning a piece of paper on a welder, lighter, or fire place.
  • add: Fireplaces and bonfires are now active ignition sources and can ignite gasses.

@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged. label Sep 11, 2024
@Plykiya
Copy link
Contributor Author

Plykiya commented Sep 11, 2024

it takes forever for the fireplace to actually warm a large area up (which is fine), that's why I'm in a two tile room for testing

@Everturning
Copy link

finally, bloodred scare in ss14...

@Plykiya
Copy link
Contributor Author

Plykiya commented Sep 11, 2024

test fail is real and related to the appearance component on the books, I'll deal with it later

@deltanedas
Copy link
Contributor

bonfire teg real

@Plykiya
Copy link
Contributor Author

Plykiya commented Sep 12, 2024

bonfire teg real

oh shit you just made me realize I forgot to log a change

@slarticodefast slarticodefast added the Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged. label Sep 13, 2024
@lzk228
Copy link
Contributor

lzk228 commented Sep 13, 2024

does fireplace put you on fire? can you cook on fireplace like on a grill?

@Plykiya
Copy link
Contributor Author

Plykiya commented Sep 13, 2024

does fireplace put you on fire? can you cook on fireplace like on a grill?

No, it just works as if it's a lit welder or lighter. Interacting with it doesn't light yourself on fire either in my testing.

Not sure about the cooking.

@github-actions github-actions bot removed the Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged. label Sep 17, 2024
@slarticodefast slarticodefast self-assigned this Oct 4, 2024
@superjj18
Copy link
Contributor

Consume oxygen, produce co2?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR requires new reviews before it can be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants