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

Amber Station Changes #34656

Merged
merged 4 commits into from
Jan 27, 2025

Conversation

southbridge-fur
Copy link
Contributor

@southbridge-fur southbridge-fur commented Jan 26, 2025

About the PR

  • Added some insuls spawners
  • Replaced most of the clothing vendors with the clothing vendor spawner
  • Added a couple lights to areas I felt were too dark
  • Moved a timed mouse spawner that was spawning mice which were getting stuck
  • Increased maxPlayers to 60 from 50
  • Increased some of the starting roles to make the map more friendly to newer players

Why / Balance

Implementing new features and also improving the map

When it comes to increasing the max player limit, I wanted to increase it largely because some of the other new maps that have come out have found that they work well at higher player limits, so I wanted to test that too. Also when I was looking into why the render was too large for github I overlayed other maps and realized Amber is actually pretty large and could totally support more players.

By my 6am calculator math there are a total of 63 job slots. So even with a full server not every job is going to get filled. This is also ignoring passengers, and I know some folks main passenger.

Technical details

Media

Map render is too big to post however there isn't much visually different

Requirements

Breaking changes

Changelog

@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.
01/26/25 11:09

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/L Denotes a PR that changes 1000-4999 lines. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Map Changes: Might require knowledge of mapping. S: Needs Review Status: Requires additional reviews before being fully accepted labels Jan 26, 2025
@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.
01/26/25 11:09

@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.
01/27/25 00:26

@ArtisticRoomba ArtisticRoomba added P3: Standard Priority: Default priority for repository items. T: New Feature Type: New feature or content, or extending existing content D3: Low Difficulty: Some 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 Jan 27, 2025
@Emisse Emisse merged commit cc4f9dd into space-wizards:master Jan 27, 2025
12 checks passed
@southbridge-fur southbridge-fur deleted the amber-station-changes branch January 27, 2025 22:56
widgetbeck pushed a commit to widgetbeck/imp-station-14 that referenced this pull request Jan 29, 2025
* various changes

* Increased player limit

* slight modification

* Followed proper yaml formatting, increased lawyers
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. D3: Low Difficulty: Some codebase knowledge required. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted size/L Denotes a PR that changes 1000-4999 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.

3 participants