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

Scale banana peel size to banana size #15953

Conversation

gear-space
Copy link
Contributor

[A-Hydroponics][C-Feature]

About the PR

Use a banana's transform (size scaling) matrix to determine the size of the resulting peel after being opened. Minimum size of 80% to prevent micro peels in cases of very unloved rotting bananas. I'm not tied to the floor being 80%. This looked visually appealing to me, and I've been told it's hard to get the size very low.

Current visuals below of a jumbo banana (top), normal banana (middle), and rotting banana (bottom)
image

Why's this needed?

Getting a normal peel after botany worked hard to make a big banana seems counterintuitive. Also big peels are funny.

Changelog

(u)spAAAce
(+)Banana peels now scale to banana size, within reason

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 22, 2023
@gear-space
Copy link
Contributor Author

Please note I'm a hobby coder with minimal Dream Maker experience - if there's a better or more efficient way to do this, please let me know! Thanks for the review :3

Copy link
Member

@frawhst frawhst left a comment

Choose a reason for hiding this comment

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

I enjoy this. Code looks good.

@ZeWaka ZeWaka added C-Feature A new feature or enhancements to existing features A-Game-Objects The point of this PR is to deal with a specific game object A-Hydroponics Deals with plants, hydro, etc. labels Sep 22, 2023
@TobleroneSwordfish TobleroneSwordfish merged commit 872d46a into goonstation:master Sep 23, 2023
25 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 23, 2023
@gear-space gear-space deleted the scale-banana-peel-on-banana-size branch October 24, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Game-Objects The point of this PR is to deal with a specific game object A-Hydroponics Deals with plants, hydro, etc. C-Feature A new feature or enhancements to existing features size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants