FakeFishGames / Barotrauma

A 2D online multiplayer game taking place in a submarine travelling through the icy depths of Jupiter's moon Europa.
http://www.barotraumagame.com/
1.77k stars 408 forks source link

Using cigars underwater while wearing divingsuits #12499

Closed G-M-twostay closed 1 year ago

G-M-twostay commented 1 year ago

Disclaimers

What happened?

You can smoke cigars underwater in a divingsuit. However, the cigar appears to also be underwater and outside of the diving suit. You simply put the cigar to the area near your mouth to smoke it, even though it's outside the diving suit.

This is quite weird because 1. cigars probably can't burn underwater 2. the smoke probably can't go through the divingsuit that blocks water.

I don't know if this is an intended behavior, but given that in a previous version equipping a flashlight will make it looks like that the flashlight is sticking out of player's mouth was considered a bug, I would consider underwater divingsuit-penetrating smoking to also be a bug.

Reproduction steps

  1. Get a cigar or pipe tobacco.
  2. Get a diving suit.
  3. Put on the diving suit and go out of the sub.
  4. smoke the cigar/pipe tobacco

Bug prevalence

Happens every time I play

Single player or multiplayer?

Happens in both single player and multiplayer

-

No response

Version

v1.0.21.0

-

No response

Which operating system did you encounter this bug on?

Windows

Relevant error messages and crash reports

No response

Regalis11 commented 1 year ago

Prevented smoking pipes and cigars underwater in https://github.com/Regalis11/Barotrauma-development/commit/cbfb529d62ef4c25bafa4bbb1ef6de6a8bda4e31.

While it's definitely weird you can smoke them with a diving suit on, I'd maybe not change that. If we did that, I think it'd raise the question of why you can drink, eat and use syringes while wearing a suit, and it'd either mean introducing logical inconsistencies in which items can be used, or changes that'd have enormous gameplay implications if we prevented using any of them.

ubertpendragon commented 1 year ago

While it's definitely weird you can smoke them with a diving suit on, I'd maybe not change that. If we did that, I think it'd raise the question of why you can drink, eat and use syringes while wearing a suit, and it'd either mean introducing logical inconsistencies in which items can be used, or changes that'd have enormous gameplay implications if we prevented using any of them.

Protein bars and Energy drinks can be injected in the bloodstream via medical interface and make a syringe injection sound, if it's supposed to simulate feeding someone it should at least make a gulp sound (both of them lack sounds). Currently it doesn't even sound or look like you're eating or drinking which isnt fun for RP

https://github.com/Regalis11/Barotrauma/assets/73229309/9b8f90f3-d398-425f-8b23-f1ab7d1be9d0

As for the pipe I loved the in-universe silliness of smoking underwater; replace the smoke by bubbles. After all this is what the pipe valve is for

image

Regalis11 commented 1 year ago

As for the pipe I loved the in-universe silliness of smoking underwater; replace the smoke by bubbles.

This is a good point. Decided to backtrack on this and just switched to bubble particles in https://github.com/Regalis11/Barotrauma-development/commit/48fbbc6c204d6ca0f1d64c40e466e6348d1ce840

Protein bars and Energy drinks can be injected in the bloodstream via medical interface and make a syringe injection sound, if it's supposed to simulate feeding someone it should at least make a gulp sound (both of them lack sounds).

The intention was actually that using things through the medical interface just means using the item in whichever way it's meant to be used (you don't inject bandages or antibiotic glue either). But yeah, a gulping sound would make that intention a lot clearer.

ghost commented 1 year ago

After all this is what the pipe valve is for

Exactly this. Poul Anderson described something like that in Margin of Profit.

From engineer's point of view, smoking in a diving suit is just a temporary switch from internal gas storage to external intake. If intake conditions allow smoking - why not? Same goes for drinks. Solid food is a problem - although HOPEFULLY it's solved somehow in Year 206X (or otherwise future-which-is-not-so-far). Since it's not zero-gravity environment, the hardest task is cleaning all leftovers up from the helmet. And everything else is manageable with even modern-day technology, if done right.

Jasontti commented 1 year ago

Tested. Smoked in the sub and outside the sub, normally and upside down. Pipes particles work, smoke in the air, bubbles in water. Cigar doesn't emit smoke or bubbles. It has pipetobacco as RequiredItem. Removing these lines makes particles work.

      <StatusEffect type="OnSecondaryUse" target="This">
        <Conditional InWater="false" />
        <ParticleEmitter particle="swirlysmoke" particlespersecond="1" scalemin="1" scalemax="2" anglemin="0" anglemax="360" velocitymin="0" velocitymax="10"/>
        <RequiredItem items="pipetobacco" type="Contained" />
      </StatusEffect>
      <StatusEffect type="OnSecondaryUse" target="This">
        <Conditional InWater="true" />
        <ParticleEmitter particle="bubbles" particlespersecond="3" scalemin="0.3" scalemax="0.5" anglemin="0" anglemax="360" velocitymin="0" velocitymax="10"/>
        <RequiredItem items="pipetobacco" type="Contained" /> 
      </StatusEffect>

Also upside down smoking animation is not working. Maybe needs to be separated on it's own ticket? image

Jasontti commented 1 year ago

Tested. Issue was fixed in: https://github.com/Regalis11/Barotrauma-development/commit/10ee146e054b4e68c6cd97aba68784e63c4b9a95 PR: https://github.com/Regalis11/Barotrauma-development/pull/4455 Animation work properly with pipe and cigar.

Cigar doesn't emit smoke or bubbles. It has pipetobacco as RequiredItem. Removing these lines makes particles work.

Was not yet addressed

Regalis11 commented 1 year ago

Fixed in https://github.com/Regalis11/Barotrauma-development/commit/69f45e81f209dd97c542413a8466f1a632d4b37e

Jasontti commented 1 year ago

Tested. No issues found. Closing ticket.