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

Lube foam! #14773

Merged
merged 5 commits into from Aug 18, 2023
Merged

Lube foam! #14773

merged 5 commits into from Aug 18, 2023

Conversation

M-Earthfire
Copy link
Contributor

@M-Earthfire M-Earthfire commented Jun 29, 2023

[Chemistry][Feature]

About the PR

This PR enables that, if 25% of a chemical mixture inside a foam is lube/harmlube, the foam takes on the slippery properties of that particular type of lube.

Why's this needed?

I had a talk ago with yass, like 3 months ago, who told me that fake cleaner grenades actual use was to make people slip and dissolve their items. Since the mechanics surrounding slipping and foam had changed, it kinda lost this use.

Since fake cleaner grenades are considered fairly weak currently, this PR adds the slippery properties back to it.

Also, this PR makes the behaviour apply to foam in general, since it is considered weak in comparison to smoke powder. One of the main reason behind this is that foam applies chemicals only if people slip in it or if the foam dissolves and only on touch. With slipping happening very rarely with foam in its current state and dissolving happening very delayed, foam left much to be desired in comparison to smoke powder, which applies its chems instantly in the area on creation with ingest and/or touch.

With enabling slipping more often in lube-foam (although with less splashed chemicals because of the dilution of the chems with lube), this PR tries to decrease the gap somewhat.

For application of lube, smoke poweder still leaves superior because of the lesser visibility of it in comparison to foam. For fighting or chases, both methods are inferior in comparison to manual application of lube on the ground.

Changelog

(u)Lord_Earthfire
(+)Having at least 25% lube in a foam mixture, water and fluorosurfactant excluded, applies the lube's slippery properties to the foam

@keywordlabeler keywordlabeler bot added A-Chemistry Deals with the chemistry system in some way C-Feature A new feature or enhancements to existing features labels Jun 29, 2023
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 29, 2023
@flappybatpal
Copy link
Contributor

I don't think mass lube is a good idea at all outside of the antag grenade.

We already have issues with spray bottles and flamethrowers.

@M-Earthfire
Copy link
Contributor Author

M-Earthfire commented Jun 30, 2023

I can understand that thought. The thing is foam already spreads the lube on the groumd, just after it dissipates (this means with a ~4 second delay). So mas lubing with foam is already a thing.

This PR more or less applies the lube effect in the time the foam lingers and thus bridges the gap.

Restricting it to the fake cleaner grenade would mean making a new form of foam chemical. I could do that, but would redesign foam from the ground up, since in its current form, without slips, its mostly useless in delivering chems.

Besides, lube smoke already mass lubes floors directly in a wide area.

@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Jul 15, 2023
@flappybatpal
Copy link
Contributor

I can understand that thought. The thing is foam already spreads the lube on the groumd, just after it dissipates (this means with a ~4 second delay). So mas lubing with foam is already a thing.

This PR more or less applies the lube effect in the time the foam lingers and thus bridges the gap.

Restricting it to the fake cleaner grenade would mean making a new form of foam chemical. I could do that, but would redesign foam from the ground up, since in its current form, without slips, its mostly useless in delivering chems.

Besides, lube smoke already mass lubes floors directly in a wide area.

Sorry, my misunderstanding of what this does!

@github-actions github-actions bot removed the S-Stale An inactive PR that has had no updates in the past two weeks label Jul 18, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Aug 2, 2023
@github-actions github-actions bot removed the S-Stale An inactive PR that has had no updates in the past two weeks label Aug 3, 2023
@deathrobotpunch
Copy link
Contributor

i never knew the fake cleaner grenades were supposed to melt your items

@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Aug 18, 2023
@Tarmunora Tarmunora merged commit b838a48 into goonstation:master Aug 18, 2023
21 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 18, 2023
@Studenterhue Studenterhue added the E-Add-To-Wiki A PR that will require changes to the wiki label Aug 19, 2023
@M-Earthfire M-Earthfire deleted the cleaner_grenade_buffs branch September 19, 2023 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Chemistry Deals with the chemistry system in some way C-Feature A new feature or enhancements to existing features E-Add-To-Wiki A PR that will require changes to the wiki S-Stale An inactive PR that has had no updates in the past two weeks size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants