Smart Pipes TM

For feedback on the game code and design. Feedback on server rules and playstyle belong in Policy Discussion.
Post Reply
User avatar
Ghilker
Joined: Mon Apr 15, 2019 9:44 am
Byond Username: Ghilker

Smart Pipes TM

Post by Ghilker » #597183

https://github.com/tgstation/tgstation/pull/58038

Leave feedbacks, ideas, comments, questions here on the new pipes (yes, they'll be the only pipes)
User avatar
Saege Tilth
Joined: Mon Dec 21, 2020 4:40 am
Byond Username: Ulrichenstein

Re: Smart Pipes TM

Post by Saege Tilth » #597191

A lot better than I thought it was going to be. For a second, I thought you did to the pipes what they did to the electric circuits. As for the layer adapters themselves, it's good they still have a purpose. This has a lot of potential for adding a shit load of layers and also potential for people to really mess things up. Hopefully it works out well for maps when the new pipes are put in.

EDIT: Some places like the super matter might have issues on certain maps with the conversion due to pipes in the cooling loop being too close, but I don't think it'll completely ruin them. The color coding thing is everything.
User avatar
oranges
Code Maintainer
Joined: Tue Apr 15, 2014 9:16 pm
Byond Username: Optimumtact
Github Username: optimumtact
Location: #CHATSHITGETBANGED

Re: Smart Pipes TM

Post by oranges » #597209

>For a second, I thought you did to the pipes what they did to the electric circuits.
I did
User avatar
Scott17
Joined: Fri Feb 26, 2021 8:58 pm
Byond Username: Scott17

Re: Smart Pipes TM

Post by Scott17 » #597222

I don't know if it had anything to do with this, but there were no pipes on the map, PERIOD.

I have nothing else to add.
User avatar
TheFinalPotato
Code Maintainer
Joined: Sun Oct 13, 2019 7:58 am
Byond Username: LemonInTheDark

Re: Smart Pipes TM

Post by TheFinalPotato » #597224

What round id?
My ancestors are smiling at me, Imperials. Can you say the same?
Image
Image
Image
Image
NotVerySmart
Joined: Sun Apr 04, 2021 1:06 am
Byond Username: NotVerySmart

Re: Smart Pipes TM

Post by NotVerySmart » #597235

Its a good idea, but maybe colors that are similar (Violet and Amethyst) should be removed, as its hard to tell the difference between the two at a glance

--

After having played with it more, I feel as if it should be optional because a primary issue is pipes autosnapping together when you really dont want it to. It has good intentions but it just fails to deliver on the simplicity atmos already has with pipes.
Last edited by NotVerySmart on Sun Apr 04, 2021 5:16 am, edited 1 time in total.
User avatar
Kassori
Joined: Sat Apr 04, 2020 3:17 am
Byond Username: Kassori

Re: Smart Pipes TM

Post by Kassori » #597243

Tried to put 3 scrubbers in toxins, "smart pipes" did not link
Image
pfp by AspEv
Narskop
Joined: Sun Apr 04, 2021 4:39 pm
Byond Username: Narskop

Re: Smart Pipes TM

Post by Narskop » #597284

- Layer adapter not working as a pipe bridge ruin a ton of setup
- Pipe Bridge being a special pipe and not automatically added is a pain. Also Pipe bridge prevent from linking any other pipe to it
- Color separation of pipenet overcomplicate pipes for no reasons. We already have 5 layers, and in it's current state the PR does not permit to have several pipenet of various colors on one layer, so the color separation serve no purpose aside from bullying you to use grey everywhere. This also mess up very badly with any modification of roundstart piping
- Device being colored, if color separation stay, will just make you waste more space on most setup. All devices should be grey, like they always have been.

Overall, for a "Smart" system, it requires a LOT more thoughts from the user than the previous one. Making the "pipenet=powernet" is missing about 90% of the Atmos system subtilities and it shows in usage. If color separation of pipenets goes away and some previous functionalities are restored (most especially making all devices act as a pipe bridge and pipe bridge being automated like it was before), then I can see it being a net benefit, after all of the bugs are fixed.

In it's current state, please never merge this thing.
User avatar
Ghilker
Joined: Mon Apr 15, 2019 9:44 am
Byond Username: Ghilker

Re: Smart Pipes TM

Post by Ghilker » #597293

A lot has changed now, please read the PR body to learn how to use the new pipes
https://github.com/tgstation/tgstation/pull/58038
User avatar
mindstormy
In-Game Admin
Joined: Thu Jun 28, 2018 1:59 pm
Byond Username: Mindstormy

Re: Smart Pipes TM

Post by mindstormy » #597495

I only played a little bit so far, but I think my biggest complaint is I can't weave pipes in weird directions that I want without it snapping them to what direction it THINKS I should go which reduces the fun of making my pipes look pretty, or hiding pipe stuff from the AI or folks who don't understand the pipes. Its seems much harder now to do basic things like getting pure o2 in the mix line for turbine work. I think I need to test it more to get a feel for how this works and time to see other folks examples of proper pipe shenanigans. But given I was not a fan of how wires changed and this seems like kind of the same thing, I think I will just have to live with this feature and not moan about it.
User avatar
Capsandi
Joined: Sat Jan 26, 2019 10:59 pm
Byond Username: Capsandi

Re: Smart Pipes TM

Post by Capsandi » #597498

I still don't see the value in color separation, 5 layers is all you need and the separation of colors is really annoying to work around. The rest of the changes are great for usability, but are nullified by the constant need to go back into the RPD ui to change the color, and if you cant find an RPD then your SOL. Its for no benefit whatsoever because you can only have 1 pipe on a tile, I really see no point in this. I see potential in smart pipes, but for now they aren't an improvement over the old system because of this aimless complication.
Timonk wrote:
Wesoda25 wrote:Genuinely think they should be blacklisted.
You have clearly never seen his dick
Lower your tone with me if your tracked play time doesn't look like this:
Image
Flatulent wrote:of course you can change religion doing it while islamic however makes you lose your head from happiness
User avatar
toadtheloser
Joined: Mon Sep 14, 2020 4:19 am
Byond Username: Toad_the_loser

Re: Smart Pipes TM

Post by toadtheloser » #597523

Needing to use a pipe bridge is the bane of my existence.
User avatar
oranges
Code Maintainer
Joined: Tue Apr 15, 2014 9:16 pm
Byond Username: Optimumtact
Github Username: optimumtact
Location: #CHATSHITGETBANGED

Re: Smart Pipes TM

Post by oranges » #597531

Capsandi wrote:I still don't see the value in color separation, 5 layers is all you need and the separation of colors is really annoying to work around. The rest of the changes are great for usability, but are nullified by the constant need to go back into the RPD ui to change the color, and if you cant find an RPD then your SOL. Its for no benefit whatsoever because you can only have 1 pipe on a tile, I really see no point in this. I see potential in smart pipes, but for now they aren't an improvement over the old system because of this aimless complication.
why do you constantly need to change colours? you should only be laying one network at a time?
User avatar
oranges
Code Maintainer
Joined: Tue Apr 15, 2014 9:16 pm
Byond Username: Optimumtact
Github Username: optimumtact
Location: #CHATSHITGETBANGED

Re: Smart Pipes TM

Post by oranges » #597532

toadtheloser wrote:Needing to use a pipe bridge is the bane of my existence.
it'd be nicer to just be able to click drag one pipe colour to another on the same tile with a wrench in hand and have them bridge automatically tbh.
User avatar
mindstormy
In-Game Admin
Joined: Thu Jun 28, 2018 1:59 pm
Byond Username: Mindstormy

Re: Smart Pipes TM

Post by mindstormy » #597536

So after a bit more testing I think there are two big things this feature needs.
  • Just get rid of the whole colors being their own pipe network thing. It makes changing layers and colors at the same time take way to much space and is just generally frustrating to deal with. If you get rid of this, then you can toss color adapters too as they are just unneeded
  • Make pipe snapping toggleable. I personally HATE them snapping together as pipe work is always super finicky and any mistake is quite unforgiving to fix. A single mistake should not ruin 30-45 minutes of work.
User avatar
Ghilker
Joined: Mon Apr 15, 2019 9:44 am
Byond Username: Ghilker

Re: Smart Pipes TM

Post by Ghilker » #597538

oranges wrote:
toadtheloser wrote:Needing to use a pipe bridge is the bane of my existence.
it'd be nicer to just be able to click drag one pipe colour to another on the same tile with a wrench in hand and have them bridge automatically tbh.
it already places a bridge pipe automatically when placing a pipe on top of another one on the same layer and with different colors.
mindstormy wrote:
  • Just get rid of the whole colors being their own pipe network thing. It makes changing layers and colors at the same time take way to much space and is just generally frustrating to deal with. If you get rid of this, then you can toss color adapters too as they are just unneeded
that's impossible, the pipes would just snap to each other at will without any control.
mindstormy wrote:
  • Make pipe snapping toggleable. I personally HATE them snapping together as pipe work is always super finicky and any mistake is quite unforgiving to fix. A single mistake should not ruin 30-45 minutes of work.
can't make them toggleable (it would be like allowing to choose between old pipes and new pipes). You can make them not snap toghether by using the different colors available and the 5 layers the pipes can be placed to.

As a side note, i'm still working to improve the pipes, it has just been just shy of 2 days now and many are still with old pipes mindset. Try them more and don't use meta setups anymore, have more creativity and use colors and layers, remember that all components can be colored, layer manifolds too.
User avatar
mindstormy
In-Game Admin
Joined: Thu Jun 28, 2018 1:59 pm
Byond Username: Mindstormy

Re: Smart Pipes TM

Post by mindstormy » #597542

Ghilker wrote:As a side note, i'm still working to improve the pipes, it has just been just shy of 2 days now and many are still with old pipes mindset. Try them more and don't use meta setups anymore, have more creativity and use colors and layers, remember that all components can be colored, layer manifolds too.
Maybe you or someone else could update the wiki with how we should do things in the new pipe mindset? I watched the video that was posted on the pr but it still doesn't click with me. Even something as basic as getting the o2/plasma line setup for turbine is stupid complicated now. Some gifs would really really help here. Should I just stop using colored pipes at all if I want to kind of ignore that part of it?

I think I fundamentally disagree that pipes should `snap` to other pipes. It is just far to unforgiving to mistakes without a easy way to de-gas a pipe section, which could be a neat way to resolve this but doesn't exist yet.
User avatar
Ghilker
Joined: Mon Apr 15, 2019 9:44 am
Byond Username: Ghilker

Re: Smart Pipes TM

Post by Ghilker » #597544

I will update the wiki and try and make a few gifs on how to work with the new pipes
User avatar
oranges
Code Maintainer
Joined: Tue Apr 15, 2014 9:16 pm
Byond Username: Optimumtact
Github Username: optimumtact
Location: #CHATSHITGETBANGED

Re: Smart Pipes TM

Post by oranges » #597559

Ghilker wrote:
oranges wrote:
toadtheloser wrote:Needing to use a pipe bridge is the bane of my existence.
it'd be nicer to just be able to click drag one pipe colour to another on the same tile with a wrench in hand and have them bridge automatically tbh.
it already places a bridge pipe automatically when placing a pipe on top of another one on the same layer and with different colors.
oh that seems fine then, people are just going through an adjustment period.

I'd love to know why that one person is constantly switching colours though
User avatar
Capsandi
Joined: Sat Jan 26, 2019 10:59 pm
Byond Username: Capsandi

Re: Smart Pipes TM

Post by Capsandi » #597562

oranges wrote: I'd love to know why that one person is constantly switching colours though
Switching colours isn't needed too often, your right, unless your fixing issues with the SM. It can get a bit unwieldy when working around the gas filter array during a delam, and placing a grey pipe which the RPD defaults to moves N2 into the waste disposal injector. Though I've only attempted to fix 2 delams while this has been merged, and it may be a matter of getting used to the system, I still don't see a reason for having more than 5 pipenet separators when 2 of the 5 layers are never used.
Actually, in the time since my first post, I've realized that layers may be the less effective option due to the inability of switching layers on the same tile in which a component is placed, so I suppose now the question is why convolute the process with layers when coloured piping is more compact?
Timonk wrote:
Wesoda25 wrote:Genuinely think they should be blacklisted.
You have clearly never seen his dick
Lower your tone with me if your tracked play time doesn't look like this:
Image
Flatulent wrote:of course you can change religion doing it while islamic however makes you lose your head from happiness
User avatar
oranges
Code Maintainer
Joined: Tue Apr 15, 2014 9:16 pm
Byond Username: Optimumtact
Github Username: optimumtact
Location: #CHATSHITGETBANGED

Re: Smart Pipes TM

Post by oranges » #597566

personally I'd do away with layers, and just have it that if you want to join two pipes on any tile you just drag them together with a wrench or some other tool, and otherwise they're just seperated by colour

a lot of atmos mains seem to think you need layers & colours, but frankly I don't see the need, you only ever have independent networks that you want to join at carefully curated points.
User avatar
Ghilker
Joined: Mon Apr 15, 2019 9:44 am
Byond Username: Ghilker

Re: Smart Pipes TM

Post by Ghilker » #597626

https://imgur.com/a/BDueHKa

I made a few videos on how the pipes work, just a few seconds long, showcasing the basics of the pipes
User avatar
mindstormy
In-Game Admin
Joined: Thu Jun 28, 2018 1:59 pm
Byond Username: Mindstormy

Re: Smart Pipes TM

Post by mindstormy » #597669

You should add what color a pipe is to the examine text like it shows for layers so colorblind folks know what color the pipe is.
User avatar
Mothblocks
Code Maintainer
Joined: Fri Jul 27, 2018 9:33 am
Byond Username: Jaredfogle

Re: Smart Pipes TM

Post by Mothblocks » #597699

I agree, but it should also show it in the name of the pipe itself so you can just hover over.
Shaps-cloud wrote: Mon Dec 07, 2020 7:59 am May eventually become one of the illusive maintainer-headmins if they choose to pursue that path, having a coder in the senior admin leadership has usually been positive for both sides in the past.
Head Coder of /tg/station, hi!

Head Admin of /tg/station Feb 2022.

Mothblocks everywhere, >>> Say nice things about me <<<
Post Reply

Who is online

Users browsing this forum: Bing [Bot]