-
-
Notifications
You must be signed in to change notification settings - Fork 539
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
Adds a crashed pubby as a sand world ruin #2363
Conversation
Fixed all conflicts awaiting review by a maptainer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
light on locker
unrotated button. mabye make things a bit more thrown about in the ship, instead of being on tables? would give it a more unkept feel
light on apc
light on locker, light on newscaster
light on airalarm
ruin has 6 apc's, yet one area! this will cause bugs. (remove all but one apc)
Remove the autosurgeon. while it's a cool item, its a tad too good for the challenge this ruin poses. (shouldn't really be used at all, anywhere)
good work!
Changes made awaiting review again |
_maps/RandomRuins/SandRuins/whitesands_surface_pubbyslopcrash.dmm
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please beef up these rocks around the corners a little. one-wide mountains will look odd -- and reveal structures behind them, as they do not blend with walls.
replace these sand turfs with /lit variants -- or replace them with rock turfs, if you want to make this look like a cave.
good job! we'll get this test merged soon.
Requested changes made this pull request is ready to be merged once the tests are run then correct? |
essentially, yes. |
Checks are all green time to test! |
There are still podlocks and blast doors under the rocks in the pod bay area. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what Zevo said. i'll Merge once you fix it
Please add a screenshot from SDMM with the areas on. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
fix the conflict, and i'll see about merging this! |
Screen shot added to PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
take a look at the rest of our crash maps and see how they handle crashes. Usually you can see that they landed, pushing up a crater, as opposed to being entombed in rock.
Once you do that, you can scatter some more debris around and create more of an "impact" in your story.
The base idea is sound, if not exciting. Not every ruin needs to be some world shattering revelation though, so I happen to like this.
I have a few more nitpicks on this, but seeing as this has been open since september. marged. |
Adds a crashed pubby as a sand world ruin (shiptest-ss13#2363)
## About The Pull Request As per the request of @Latentish This PR tweaks the recently merged (#2363) Crashed Pubby Ruin. The shape of the crash area should now be more realistic. Several issues have been fixed, as well. ![image](https://github.com/shiptest-ss13/Shiptest/assets/118859017/4dc076fe-8dce-4ab5-a37e-c201ee5e77d2) ![9cbdcebcc92d3e5d3a96e44677e129bf](https://github.com/shiptest-ss13/Shiptest/assets/118859017/9b6d7ad7-bf15-44a1-9008-4ce6ce08a3bc) ![image](https://github.com/shiptest-ss13/Shiptest/assets/118859017/d5732f90-f242-41f4-b5e7-e58b10dc5e08) ## Why It's Good For The Game <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> ## Changelog ๐ tweak: The Crashed Pubby Ruin has been tweaked slightly. fix: Fixed a floating wallmount in the Crashed Pubby Ruin. /๐
## About The Pull Request As per the request of @Latentish This PR tweaks the recently merged (shiptest-ss13#2363) Crashed Pubby Ruin. The shape of the crash area should now be more realistic. Several issues have been fixed, as well. ![image](https://github.com/shiptest-ss13/Shiptest/assets/118859017/4dc076fe-8dce-4ab5-a37e-c201ee5e77d2) ![9cbdcebcc92d3e5d3a96e44677e129bf](https://github.com/shiptest-ss13/Shiptest/assets/118859017/9b6d7ad7-bf15-44a1-9008-4ce6ce08a3bc) ![image](https://github.com/shiptest-ss13/Shiptest/assets/118859017/d5732f90-f242-41f4-b5e7-e58b10dc5e08) ## Why It's Good For The Game <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> ## Changelog ๐ tweak: The Crashed Pubby Ruin has been tweaked slightly. fix: Fixed a floating wallmount in the Crashed Pubby Ruin. /๐
About The Pull Request
Adds a new pubby crash map along with the requisite area and ruin codes
Why It's Good For The Game
It adds another ruin... this is a good thing
Changelog
๐ Immortalartisan
add: A new sandworld ruin based on deprecated ship pubby
/:cl:
SDMM view with areas