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

Cardinal Station (old Bunched station) #12030

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

Conversation

BarteG44
Copy link
Contributor

@BarteG44 BarteG44 commented Dec 14, 2024

About The Pull Request

Card Station is a station based around 4 central corridors running in cardinal directions. Despite being about as large as Fland station in footprint, it's departments are placed and laid out is such a way as to ensure player interaction.

Design principles:
-High traffic areas such as security, bridge, and medbay are all located at the ends of the corridors, forcing traffic to go trough the service-oriented center.
-Most rooms directly connected to the corridors also have plentiful windows facing them with optional shutters or curtains. This is done to allow anyone from the corridor to see and talk directly to people who may be busy in their workplace, who would usually be tucked away somewhere far from sight. The shutters are there to permit some privacy to the people working, while their prevalence will make it less likely for anyone to suspect them of shady business if they do choose to close their department off. It's not suspicious if everyone's doing it.
-Greenery is present in most places. Just like in real life, having some plants around makes the whole space more pleasant to look at.
-More gimmick areas such as an actual Theater, Barbershop, decently sized office and a maintenance night club
-to allow for more or less discrete travel between parts of maintenance, there are 6 rooms linked in pairs with dispsal pipes

Why It's Good For The Game

Box, Meta, and delta are pretty much just different flavors of the same thing, people hate kilo for some reason and people hate corg for good reason. That leaves 3 maps that at least I personally would consider good, 2 of which are exclusive to diametrically different player counts. We could really use something more unique.

Testing Photographs and Procedure

Screenshots&Videos

image

StrongDMM-2025-01-02 16 03 10

image

image

image

image

image

image

image

image

image

image

Changelog

🆑
add: Added Card Station and all of it's shuttles
add: Added big potted plants
add: Added sub divided big engines for shuttles
fix: fixed decals not working on circuit and grass floor tiles
/:cl:

i'm making these commits way too often
I pretty much brute forced this, because i don't have an idea how to edit the decal code to allow for nudging them
…have them be automatically loaded on mapload
last one for the night
@BarteG44
Copy link
Contributor Author

BarteG44 commented Jan 2, 2025

You could call it Cardinal Station! Or just Card Station

Some other options: Cross Station Causeway Station Devil station (It's at the crossroads) Chatter Station Walk Station

Bunched station seems very hard to say and even just thinking it it doesn't flow great.

The deed has been done. Cardinal Station it is

Copy link
Contributor

@ClownMoff ClownMoff left a comment

Choose a reason for hiding this comment

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

My thoughts on some of the code

_maps/bunchedstation.json Outdated Show resolved Hide resolved
code/datums/shuttles.dm Outdated Show resolved Hide resolved
@@ -56,6 +56,11 @@ map echostation
votable
endmap

map bunchedstation
minplayers 20
Copy link
Contributor

@ClownMoff ClownMoff Jan 2, 2025

Choose a reason for hiding this comment

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

Suggested change
minplayers 20
minplayers 40

Bump this to at least the same amount of players required for Fland. This is a huge station and it is unrealistic to think that this map will play well with 20 people. I don't think it will play well with 30 either but 20 is just straight wrong. If the map gets voted with 25 people, 5 leave, 20 come into the round, 2 people go mining and 2 exploring you end up with 16 people in a station that should be for +40 people on station

Copy link
Contributor Author

Choose a reason for hiding this comment

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

again, as bacon said, it's not just the footprint of the map that matters. We'll have to see how it actually plays in game. If it does feel too empty then i'm fine with increasing it

Copy link
Contributor

@ClownMoff ClownMoff Jan 2, 2025

Choose a reason for hiding this comment

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

Yeah, both matter, that's why this map can feel amazing but if I'm alone wandering in hallways 6 tiles wide by myself and I see some assistant from time to time the map will fail as a lowpop map. Echo can get away with this because everything is cramped together, Delta puts 40 people minimun, same with Fland. Box can get away with being a big map for lowpop since the majority of the action happens on the same hallway (this map has those points of interest/actions either in smaller size or more separated between them so it wont get the same Box effect). 20 pop is a bad pop to put on a map this big and I think the grand majority of mappers would agree and suggest, at minimun, 25-30.

@ClownMoff
Copy link
Contributor

ClownMoff commented Jan 2, 2025

Onto the map itself.

Map review

image

This engi tools room looks a lot like the one in DeltaStation. That one is not avaiable to everyone. Is this one public? It certainly gives engi-only access vibes but there's an assistant spawn on it

image

Too cramped

image

Shock those windows if they aren't already please, this is just an outpost that can easily have no one inside and it just screams free chemistry stuff

image

The doble airlock needs some resprite, it looks too out of touch with the rest of the artstyle to use it this much on a map

image

Put windows here or see-through doors. Cryo pods should never be hidden anywhere and easily accessible for everyone without having to open tons of doors to find them. Ideally, they wouldn't even have their own dedicated room in the prison imo

image

Is this the security medbay outpost?

image

A second bar/stripclub is a nice touch but having plasma tables unguarded means people can just go there and get free plasma roundstart

image

Decor-wise doesn't make much sense to have this part open to go through, why would it be a tree in the way of the space dedicated to transit? I'd do a full circle of raillings around

image

Double doors look pretty bad, either put a window or make a new blue double airlock for command.

I do think this is a pretty station with tons of good ideas and well executed, but what I really fear is how insanely big is for a server that hits 60 pop a couple times a year

@BarteG44
Copy link
Contributor Author

BarteG44 commented Jan 2, 2025

Onto the map itself.

  1. yeah, it's restricted. good catch on the trapped assistant spawn.
  2. it's intentional
  3. it being a place to yoink a chem dispenser is also intentional but making it more secure sounds fine
  4. I really don't see this. They're as outdated as all the other airlocks
  5. I don't think this is that big of an issue. I can sprite a cryopod sign maybe though. Also most of the time when an antag gets caught and ragequits, they don't bother going into cryo anyways.
  6. yes.
  7. Plasma isn't that secure anyways. You can find it in maintenance and probably get it from cargo if you just asked. I haven't seen anyone do a plasma bombing either in my long time on this server so even if they do start stealing it, i see it as a positive.
  8. it's not meant to be traversed, more like to be approachable. Similar idea to like a park. you can walk trough it, but it's not as easy as the road next to it.
  9. double doors get used a lot on maps. I honestly think they look fine

@BarteG44 BarteG44 marked this pull request as ready for review January 2, 2025 15:00
@BarteG44 BarteG44 requested a review from Crossedfall as a code owner January 2, 2025 15:00
@BarteG44 BarteG44 changed the title Bunched Station Cardinal Station (old Bunched station) Jan 2, 2025
@ClownMoff
Copy link
Contributor

ClownMoff commented Jan 2, 2025

  • I really don't see this. They're as outdated as all the other airlocks
Comparison

image

image

Different shading, doble one looks definitely older and worse. A resprite should be easy enough

8. it's not meant to be traversed, more like to be approachable. Similar idea to like a park. you can walk trough it, but it's not as easy as the road next to it.

I think it looks pretty odd visually tbh

5. I don't think this is that big of an issue. I can sprite a cryopod sign maybe though. Also most of the time when an antag gets caught and ragequits, they don't bother going into cryo anyways.

You should not do design choices around people breaking the rules, that's what admins are for. You should always have the option to leave the game easily located

9. double doors get used a lot on maps. I honestly think they look fine

Wouldn't a command double door look better? ;))

Also, your changelog is wrong. it should have your name on it, like this

🆑 BarteG44
rest of the changelog
/:cl:

@BarteG44
Copy link
Contributor Author

BarteG44 commented Jan 2, 2025

the changelog doesn't need that... it gets credited automatically to the author.

@spockye
Copy link
Member

spockye commented Jan 6, 2025

a lot of creative and good mapping here,
amazing work!
would like to see this tm'd at a later date to see how it plays out.

@BarteG44
Copy link
Contributor Author

BarteG44 commented Jan 7, 2025

i'm working on a feature to make the SM ejection a bit less of a "get out of jail free card" but it requires me to learn TGUI so it's gonna be a while

@CRISOOOOO
Copy link

Hey, do you have a discord? I would like to talk with you about something

@BarteG44
Copy link
Contributor Author

Hey, do you have a discord? I would like to talk with you about something

@barteg_

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Config Update Feature Fix Frozen Frozen until a larger project is finished Mapping DMM Change Sprites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants