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

Add pipe destruction behavior #20928

Merged
merged 3 commits into from
Dec 11, 2023
Merged

Conversation

Partmedia
Copy link
Contributor

@Partmedia Partmedia commented Oct 11, 2023

About the PR

This adds a destruction behavior to pipes, namely, converting them to a broken pipe that can be unwrenched and welded.

Why / Balance

It makes a bit more sense for pipes to break when damaged, like disposal pipes, instead of disappearing immediately. This is also a quality of life prerequisite for future pressure-based pipe damage, because having pipes evaporate into thin air without any indication of what went wrong is not good.

Media

out-2023-10-11_15.00.52.mp4
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

🆑 notafet

  • tweak: Pipes now leak their contents when destroyed. Make sure to drain pipes before removing them.

@github-actions github-actions bot added Changes: Sprites Changes: Might require knowledge of spriting or visual design. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. labels Oct 11, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 11, 2023

RSI Diff Bot; head commit af0be59 merging into 0afc035
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Structures/Piping/Atmospherics/pipe.rsi

State Old New Status
pipeBroken Added

Edit: diff updated after af0be59

@Partmedia Partmedia mentioned this pull request Oct 12, 2023
1 task
@Partmedia Partmedia added the S: Awaiting Changes Status: Changes are required before another review can happen label Oct 12, 2023
@github-actions github-actions bot removed the Changes: No C# Changes: Requires no C# knowledge to review or fix this item. label Oct 13, 2023
@Partmedia
Copy link
Contributor Author

Partmedia commented Oct 13, 2023

Pipes now leak contents when destroyed. Needed to prevent cascading failures in the linked PR.

burst-leak.mp4

@Partmedia Partmedia removed the S: Awaiting Changes Status: Changes are required before another review can happen label Oct 13, 2023
Copy link
Contributor

@keronshb keronshb left a comment

Choose a reason for hiding this comment

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

I'd like to see the following:

  • A screenshot/video of the current damaging system of what it would look like when pipes are damaged under tiles.

  • Pipe damage should leave more hints on what broke

    • The current way its done makes it exceptionally hard to tell what kind of pipe broke
      • I wouldn't know if a T, + or L pipe broke since it's just 1/2 of a damaged pipe
    • Another screenshot/video of what these broken pipes would look like if there are tiles
  • If broken somewhere in the station, what devices can an atmos tech use to find where it is?

    • Please try loading a station, having a few pipes break, and then try to find them with the current devices to see what I mean

@keronshb keronshb added the S: Awaiting Changes Status: Changes are required before another review can happen label Oct 14, 2023
@Partmedia
Copy link
Contributor Author

I'd like to see the following:

* A screenshot/video of the current damaging system of what it would look like when pipes are damaged under tiles.

* Pipe damage should leave more hints on what broke
  
  * The current way its done makes it exceptionally hard to tell what kind of pipe broke
    
    * I wouldn't know if a T, + or L pipe broke since it's just 1/2 of a damaged pipe
  * Another screenshot/video of what these broken pipes would look like if there are tiles

* If broken somewhere in the station, what devices can an atmos tech use to find where it is?
  
  * Please try loading a station, having a few pipes break, and then try to find them with the current devices to see what I mean

This PR only adds destruction behavior. Pipes breaking on pressure damage is in the other PR.

In this PR, there is no way to destroy pipes beyond hitting it with a crowbar.

@Partmedia
Copy link
Contributor Author

* A screenshot/video of the current damaging system of what it would look like when pipes are damaged under tiles.

* Pipe damage should leave more hints on what broke
  
  * The current way its done makes it exceptionally hard to tell what kind of pipe broke
    
    * I wouldn't know if a T, + or L pipe broke since it's just 1/2 of a damaged pipe
  * Another screenshot/video of what these broken pipes would look like if there are tiles

This is how disposal junctions break. Players seem to be able to repair disposal junctions fine. I don't think this is a concern.

If you insist, then individual sprites for these pipes can be added.

Note that without this PR, pipes simply disappear. So this PR represents an improvement from the status quo.

@keronshb
Copy link
Contributor

keronshb commented Oct 14, 2023

This PR only adds destruction behavior. Pipes breaking on pressure damage is in the other PR.

That's fine. What you can do is pull from your pressure damage branch to this one temporarily and then cause some damage in the pipenet & report back with vids. Then just discard changes when you're done.

If you insist, then individual sprites for these pipes can be added.

100% otherwise it's going to be exceptionally hard to figure out what was damaged

I'd also say maybe make em a bit messy too. When they break in the media it's just neatly in the middle of the tile.

@Partmedia
Copy link
Contributor Author

This PR only adds destruction behavior. Pipes breaking on pressure damage is in the other PR.

That's fine. What you can do is pull from your pressure damage branch to this one temporarily and then cause some damage in the pipenet & report back with vids. Then just discard changes when you're done.

IMO this change is not related to pressure damage. Even if pressure damage ultimately is not merged, then this change still makes sense because pipes currently disappear when destroyed.

If you insist, then individual sprites for these pipes can be added.

100% otherwise it's going to be exceptionally hard to figure out what was damaged

I'd also say maybe make em a bit messy too. When they break in the media it's just neatly in the middle of the tile.

Sure, this can be done. Still, is the change here not an improvement over the current behavior, which is that it disappears entirely when you hit it with a crowbar?

@Partmedia
Copy link
Contributor Author

It sounds like there may be some misunderstanding about what this actually changes.

Current behaivor: When you hit a pipe with a crowbar enough times, it simply disappears.

New behavior: When you hit a pipe with a crowbar enough times, it turns into a broken pipe and leaks gas, as if you unanchored it.

@LankLTE
Copy link
Contributor

LankLTE commented Oct 14, 2023

New behavior: When you hit a pipe with a crowbar enough times, it turns into a broken pipe and leaks gas, as if you unanchored it.

Does this mean there’s no way to remove a pipe without it flooding the room? I’d honestly assumed the pipe not filling the room if broken was intentional and the point of atmos having an axe

@keronshb
Copy link
Contributor

IMO this change is not related to pressure damage. Even if pressure damage ultimately is not merged, then this change still makes sense because pipes currently disappear when destroyed.

I was suggesting it as a way to try and blow up a pipe without needing to damage it yourself as a way to test it getting damaged underneath a tile.

New behavior: When you hit a pipe with a crowbar enough times, it turns into a broken pipe and leaks gas, as if you unanchored it.

I saw in the other PR that all of the damaged pipes all became the same sprite, which makes it harder to discern what kind of pipe was broken.

@Partmedia
Copy link
Contributor Author

Here is a recording of the old behavior:

pipe-old.mp4

Note that when the pipe is destroyed, it simply disappears which makes it very hard to tell that something has broken.

This is a recording of the new behavior:

pipe-new.mp4

At the very least, it leaves evidence that there was a pipe there. This is not perfect, but I believe this represents an improvement over what currently happens, so I'm opting to merge this.

@Partmedia Partmedia merged commit 6444f7b into space-wizards:master Dec 11, 2023
@Partmedia Partmedia deleted the notafet/pipe branch December 11, 2023 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Sprites Changes: Might require knowledge of spriting or visual design. S: Awaiting Changes Status: Changes are required before another review can happen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants