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

Fixes space ruin generation #47

Closed

Conversation

NanoCats
Copy link
Contributor

@NanoCats NanoCats commented May 7, 2023

About The Pull Request

I have a friend who knows a lot about the inner machinations of the game, I invited him to hang out on Monkestation and we decided to pop out to space for fun.

Turns out he's figured out that we're generating 10x less space ruins than we should be, there probably was a issue in the ruin distribution formula. I've fixed it by basically copying TGstation pull #74841

Why It's Good For The Game

Bug fixes are good, and space exploration was weirdly anemic so we'd basically only get an extremely low number of ruins.

Furthermore it will cut down on spaghetti code, preventing unnecessary generation of empty z-levels, and allow for future mappers to customize how many space z-levels should be generated.

Changelog

🆑

fix: Space ruins are now spawning at their intended rate.

/:cl:

@NanoCats
Copy link
Contributor Author

NanoCats commented May 7, 2023

ok turns out this isn't working for some reason I hate coding

Aeri added 5 commits May 7, 2023 14:35
…SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTITHE SPAGHETTI
@NanoCats NanoCats marked this pull request as draft May 7, 2023 19:01
@NanoCats
Copy link
Contributor Author

NanoCats commented May 7, 2023

continuing to work on this, tougher to crack than I first presumed

@Zonespace27
Copy link
Collaborator

Request a re-open when this PR is ready

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants