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

[WIP] Fire nerf #9016

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
6 participants
@kevingranade
Copy link
Member

commented Sep 15, 2014

The basic idea here is to add a small, non-hazardous fire to model the very first stages of lighting a fire.
These fields might even be relatively long-lived if the fuel being consumed isn't particularly volatile.

Things like molotovs and flamethrowers would still create full-size fire fields immediately, mut matches and lighters would now make these little flames, which would need to be nurtured into a full-sized fire.

I started on this a while back and got distracted from it, posting in case someone wants to follow up.

  • Add growth stages for small fires, including promotion to full-size fires.
  • Add effects for small fires? (just a teensy bit of hotfoot?)
  • Switch firestarters over to making these fields instead of full-sized fires.
  • Make fire spread using these instead of spawning full-sized fire fields.

Possibly related, possibly belongs in its own PR:

  • Add a "hot air" field produced by fires and other heat sources, these would meander around much like smoke, but would be harmless except for elevating the temperature of the area. These would model simple building heating.
@Lain-

This comment has been minimized.

Copy link
Contributor

commented Sep 15, 2014

Nice!

@narc0tiq

This comment has been minimized.

Copy link
Contributor

commented Sep 15, 2014

Add effects for small fires? (just a teensy bit of hotfoot?)

Also, stepping onto a small fire should probably have a relatively high chance of putting it out.

@@ -285,6 +285,12 @@ void game::init_fields()
{_("hazy cloud"),_("fungal haze"),_("thick fungal haze")}, '.', 8,
{ c_white, c_cyan, c_cyan }, { true, true, false }, { true, true, true }, 40,
{0,0,0}
},

{ // A small, incipient fire.

This comment has been minimized.

Copy link
@narc0tiq

narc0tiq Sep 15, 2014

Contributor

Looks like a line's missing here:

+            "fd_flame",

This comment has been minimized.

Copy link
@kevingranade

kevingranade Sep 16, 2014

Author Member

Gah, did a rebase just before pushing, thanks for catching that.

@jcd000

This comment has been minimized.

Copy link
Contributor

commented Sep 15, 2014

yup nice.
i assume one will not be able to cook with the small fire field, correct?
if so, it fixes the exploit of putting a flammable material somewhere, lighting it, then immediately removing it and having enough fire to do some cooking (and with batch crafting, one can do lots of cooking with just one lighter charge)

also, yes to the "hot air" mechanic, either here or in another PR.
this mechanic can set the groundwork for more advanced base heating in the future.

@Shoes01

This comment has been minimized.

Copy link
Contributor

commented Sep 15, 2014

Oh my god, that last point about having an invisible field be a source of heat is genius. I had something similar to that in my ice content PR with "ice fog", so I will look at that and see if I can't make that PR happen!

Also, this invisible field could also directly add to temperature (instead of increasing body heat) and reduce humidity in the room!

@Shoes01 Shoes01 referenced this pull request Sep 15, 2014

Merged

Fires generate hot air #9030

@Shoes01

This comment has been minimized.

Copy link
Contributor

commented Sep 24, 2014

You can check off the last point, making this 1/5 complete!

@KA101 KA101 force-pushed the CleverRaven:master branch from a342a64 to 0da0595 Oct 30, 2014

@OzoneH3

This comment has been minimized.

Copy link
Member

commented Dec 3, 2015

No progress for over a year, closing.

@OzoneH3 OzoneH3 closed this Dec 3, 2015

@kevingranade kevingranade deleted the kevingranade:fire-nerf branch Jan 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.