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

New Low-Mid Pop Station - Amber Station #33441

Merged
merged 29 commits into from
Dec 3, 2024

Conversation

southbridge-fur
Copy link
Contributor

@southbridge-fur southbridge-fur commented Nov 21, 2024

About the PR

Amber station, an ex-independent amalgamation of various ships cobbled together around some asteroids which has recently come under the ownership of Nanotransen.

This station is intended for low and medium populations. A lot of things are designed to be easy to set up for skeleton crews but still offer depth for higher amounts of players.

There is

  • No containment area, the station can be run mostly off of solars + the AME however setting up the TEG is the best source of power.
  • Sec is a carved-out nukie ship
  • Cargo is a carved-out ATS
  • Several other departments feature elements from various ships.
  • Atmosia takes the center of the map
  • Plenty of fun secrets to find!

Why / Balance

When playing on Wizden servers during low-pop hours it can get a bit boring rotating between only a couple maps. This map is intended to add some more variety to the selection.

Technical details

I've been keeping this branch up to date with the main branch so I have all the new features like dim lights added as well.

Media

Map Render as of bc18367

Amber-0

Requirements

Breaking changes

Changelog

🆑 Southbridge, Flipsie27

  • add: Added a new low-mid pop map: Amber Station

@ss14-map-server
Copy link

This PR contains new or changed maps:

  • amber.yml

Maps are being rendered. This comment will be updated if rendering them is done.
11/21/24 01:13

@github-actions github-actions bot added size/XL Denotes a PR that changes 5000+ lines. S: Needs Review Status: Requires additional reviews before being fully accepted Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Map Changes: Might require knowledge of mapping. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. and removed size/XL Denotes a PR that changes 5000+ lines. labels Nov 21, 2024
@southbridge-fur
Copy link
Contributor Author

I need to do a bit more work but I figured I'd open my PR now so I can collect comments

@github-actions github-actions bot added the size/XL Denotes a PR that changes 5000+ lines. label Nov 21, 2024
@southbridge-fur
Copy link
Contributor Author

All current issues have been resolved. I'm ready for this to move forward into more formal reviews!

@ScarKy0 ScarKy0 added P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. T: New Feature Type: New feature or content, or extending existing content D1: High Difficulty: Extensive codebase knowledge required. A: Mapping Area: Maps, or tools used for mapping and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Nov 21, 2024
@ScarKy0
Copy link
Contributor

ScarKy0 commented Nov 21, 2024

I love the gimmick of this map so much

@southbridge-fur
Copy link
Contributor Author

I've gotten a lot of feedback from Emisse which I'll implement later today.

@Thatonestomf
Copy link
Contributor

What's the main gimmick? To me it looks like a normal station with core's tiling.
Not bashing on the station or anything, just asking

@ScarKy0
Copy link
Contributor

ScarKy0 commented Nov 21, 2024

What's the main gimmick? To me it looks like a normal station with core's tiling.
Not bashing on the station or anything, just asking

From what ive seen and read on the PR, the gimmick is the fact the station is built from different parts of many shuttles (cargo is partially the ATS, sec is the nukie shuttle, etc)

@SpaceLizardSky
Copy link
Contributor

the station looks nice

@cohanna
Copy link
Contributor

cohanna commented Nov 21, 2024

is it possible to throw in a few thrusters and shuttle parts for salv, since the station is a bunch of reclaimed shuttles i feel there would be some parts left over that cargo might be able to use.
edit: nvm i found them

@GoodWheatley
Copy link
Contributor

What's the main gimmick? To me it looks like a normal station with core's tiling.
Not bashing on the station or anything, just asking

From what ive seen and read on the PR, the gimmick is the fact the station is built from different parts of many shuttles (cargo is partially the ATS, sec is the nukie shuttle, etc)

I feel like this would be more obvious if those departments had different types of flooring that were taken from the shuttle

@southbridge-fur
Copy link
Contributor Author

What's the main gimmick? To me it looks like a normal station with core's tiling.
Not bashing on the station or anything, just asking

From what ive seen and read on the PR, the gimmick is the fact the station is built from different parts of many shuttles (cargo is partially the ATS, sec is the nukie shuttle, etc)

I feel like this would be more obvious if those departments had different types of flooring that were taken from the shuttle

This was something I tried to do early on but it made the station a confusing mess to look at. Plus most shuttles have really intricate flooring designs which don't work when recontextualized into a station.

@saga3152
Copy link
Contributor

The wastenet looks weird, why does the waste go to tye burn chamber? Also there's something wront with the water puddle in maints under the TEG.

@K3-1R
Copy link

K3-1R commented Nov 26, 2024

Map looks great!

@Emisse Emisse merged commit e4e84ae into space-wizards:master Dec 3, 2024
12 checks passed
@OnsenCapy
Copy link
Contributor

Could use small chance of mob spawns on the really defunct areas of the ship to the right but otherwise seems interesting. Worried about the AI though, 13 rods to be cut and two RCDs and it is gone. I personally think that if you can disconnect the AI with just a singular RCD (6 charges of decon) then it is not secure enough.

Doctor-Cpu pushed a commit to Doctor-Cpu/space-station-14 that referenced this pull request Jan 24, 2025
* Initial Commit with two departments done

* Checkpointing work

* Added most of service and science

* Started work on medbay

* Modified TEG setup and set up atmos pipes for engineering

* Re-added medbay, added chapel and janitorial, started routing disposals and generally the final layout of the map is coming together

* Plenty of additions, too many to list in a commit message

* Major update

* Nearly done with the map, just have to do decals and emergency lights

* Added all decals, only thing missing is department signs

* Toned back the dirt decals a bunch, added hallway signs and other decorations

* Finishing touches on the map before testing.

* Fixed invalids

* Renamed the station to Amber, and made a couple adjustments

* Completely redid cargo, added maints around sec

* Added Amber to the map post init integration test

* Many small updates addressing issues.

* Ran a script to update all the camera names, also ran fixgridatmos, fixrotations, tilewalls, and variantize

* Started addressing some of the issues brough up by Emisse and others

* Addressed all issues mentioned by reviewers. Added mail system.

* Wrapped up meeting all the issues raised by reviewers, also did a bunch of testing and resolved issues found during those tests

* Addressed additional requested changes, and nabbed some ship designs from Frontier. Redesigned south east maints and surrounding meteor nets based off of those designs

* Making a couple more requested changes

* Reduced the amount of Nitrogen Closets, made several small adjustments. Ready for review again!

* Removed invalids!

* web edit lmao

---------

Co-authored-by: Emisse <99158783+Emisse@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Mapping Area: Maps, or tools used for mapping Changes: Map Changes: Might require knowledge of mapping. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. D1: High Difficulty: Extensive codebase knowledge required. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Review Status: Requires additional reviews before being fully accepted size/XL Denotes a PR that changes 5000+ lines. T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants