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

Allow Surgery, construction small puddles. #6340

Conversation

hislittlecuzingames
Copy link
Contributor

@hislittlecuzingames hislittlecuzingames commented May 27, 2024

About the pull request

Allows Surgery, construction in what look like small puddles

Explain why it's good for the game

Allows surgery and construction to be done in shallow water and shallow puddles. My rule was "There can only be one 'full' water tile in the shortest path across the depth of the puddle. if the shortest path is 2 or more 'full' water tiles, it's too deep to do surgery."
Select "long" puddles allow it if they're still only 1 tile at most for the shortest path.
My lore imagination says "okay, this is ankle-deep water, maybe shin-deep, but not knee-deep, so a roller bed should be mostly unobstructed, and the water isn't flowing. Beachheads have waves, but not puddles. Ponds are too deep."

Also, I added "deep" water sprites to some places so it would indicate "this is a puddle you cannot do surgery in" because it's "too deep."

Testing Photographs and Procedure

I allowed surgery in another PR.
Same code.

Changelog

🆑
qol: Surgery and construction allowed on select small puddles.
/:cl:

@github-actions github-actions bot added Mapping did you remember to save in tgm format? Quality of Life Make the game harder to play Balance You need to be a professional veteran game maintainer to comprehend what is being done here. labels May 27, 2024
@Osmos123
Copy link

I wonder if you could adjust it in another way by making the plasteel doors cost 10 plasteel instead of 8. That way it will divide evenly with either the 30 or the increase to 40 that you suggested. Purchasing more plasteel in the vendor by 10 will also make sure you will not have any left over plasteel.

@hislittlecuzingames
Copy link
Contributor Author

I wonder if you could adjust it in another way by making the plasteel doors cost 10 plasteel instead of 8. That way it will divide evenly with either the 30 or the increase to 40 that you suggested. Purchasing more plasteel in the vendor by 10 will also make sure you will not have any left over plasteel.

I want to improve the qol of com tech.

@Diegoflores31
Copy link
Contributor

I wonder if you could adjust it in another way by making the plasteel doors cost 10 plasteel instead of 8. That way it will divide evenly with either the 30 or the increase to 40 that you suggested. Purchasing more plasteel in the vendor by 10 will also make sure you will not have any left over plasteel.

10 plasteel per cade is insane . you would be better off making rwalls lmao

code/game/turfs/floors/desert.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
code/game/turfs/floors/desert.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
code/game/turfs/open.dm Outdated Show resolved Hide resolved
@Zonespace27 Zonespace27 marked this pull request as draft May 30, 2024 15:55
@hislittlecuzingames hislittlecuzingames marked this pull request as ready for review May 30, 2024 21:40
@hislittlecuzingames hislittlecuzingames changed the title Misc polish and qol - Surgery, construction, plasteel. Misc polish and qol - Surgery, construction small puddles. Jun 2, 2024
@hislittlecuzingames
Copy link
Contributor Author

@Zonespace27 @Nanu308
Changes have been made appropriately according to feedback.
Looking forward to the next bit of feedback.

Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jun 12, 2024
Copy link
Contributor

@Zonespace27 Zonespace27 left a comment

Choose a reason for hiding this comment

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

code is fine

Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

In general I don't see much point to this PR. Why should very specific locations allow construction and surgery? All this does is introduce inconsistency.

Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't make really any sense that you'd be allowed to construct or do surgery on hot springs.

Copy link
Contributor Author

@hislittlecuzingames hislittlecuzingames Jun 19, 2024

Choose a reason for hiding this comment

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

They don't appear to be hot; do they cause damage? Otherwise, I'm not sure how one would make said distinction unless they read the lore.
If someone can walk in it without getting hurt, then it would make sense that someone could lay on a rollerbed where only the wheels are in the water.

Last night on Desert Dam, I was watching a medic explain that shallow puddles doesn't let people do surgery.

It's just small water puddles.
The consistency is when water is a maximum of one "full" water tile on the shortest distance across the body of water.
I was imagining that puddles were there for visual flare.
Some puddles are in really gameplay inconvenient spots like the area around an FOB, or a safe wall near major combat zones.

I'm not sure if I'm explaining myself coherently.

Copy link
Contributor

@Drulikar Drulikar Jun 19, 2024

Choose a reason for hiding this comment

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

Don't allow surgery on hot springs. Its not some niche lore thing. The entire icy environment is lush from the hot springs. The area is called hot springs. They are intended to be hot springs.

Copy link
Contributor

Choose a reason for hiding this comment

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

Changes are incomplete
image

Copy link
Contributor

Choose a reason for hiding this comment

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

Whats special about the left lake that it allows surgery but the others don't?
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The other ponds are more than 1 tile "deep".

I decided to just allow surgery on all the ponds to avoid confusion.
I assume it'll just be changed that way in a few months anyway.

@Drulikar Drulikar marked this pull request as draft June 19, 2024 00:20
@Drulikar Drulikar removed the Stale beg a maintainer to review your PR label Jun 19, 2024
@hislittlecuzingames hislittlecuzingames changed the title Misc polish and qol - Surgery, construction small puddles. Surgery, construction small puddles. Jun 19, 2024
@hislittlecuzingames hislittlecuzingames changed the title Surgery, construction small puddles. Allow Surgery, construction small puddles. Jun 19, 2024
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jun 19, 2024
@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Jun 20, 2024
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@hislittlecuzingames
Copy link
Contributor Author

In files changed, I see some deleted sections.
I looked all over, but didn't see any "missing tiles" in Strong DMM.

They may all exist in Kutjevo only.

@hislittlecuzingames hislittlecuzingames marked this pull request as ready for review June 20, 2024 16:17
@hislittlecuzingames
Copy link
Contributor Author

I have to redo the merge conflicts and use the hook.
https://github.com/cmss13-devs/cmss13/blob/master/tools/hooks/README.md

@Drulikar Drulikar marked this pull request as draft June 21, 2024 16:33
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Jun 29, 2024
@github-actions github-actions bot closed this Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Mapping did you remember to save in tgm format? Quality of Life Make the game harder to play Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants