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

Revert "Changes to EZ Autoinjectors (#4167)" #4593

Closed
wants to merge 1 commit into from

Conversation

Git-Nivrak
Copy link
Contributor

@Git-Nivrak Git-Nivrak commented Oct 6, 2023

This reverts commit 49e512e.

About the pull request

Explain why it's good for the game

The original PR had no proper reasoning
Either way waiting 5 seconds to use an injector is not good for the game and does not add anything and being able to use it without delay isn't incredibly broken either.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
balance: Reverts changes to EZ Autoinjectors
/:cl:

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Oct 6, 2023
@Drulikar
Copy link
Contributor

Drulikar commented Oct 6, 2023

The original PR had no proper reasoning

It is in line with the project direction: https://forum.cm-ss13.com/t/maintainer-update-jun2023/2922

First, we’d like to see rounds “slowed down”. Speed rounds are generally not satisfying for anyone. The average length we’d like to see is around 90-120 minutes. Thankfully, we already do a fairly good job on this but keeping it consistent and minimizing speedruns is going to be a priority. Giving time for more roleplay elements to thrive and evolve is important.

Second, any additions to complexity for medical are likely to be approved. Our current system is pretty barebones and the skill/knowledge ceiling gives something to be desired for our more veteran players. This doesn’t explicitly need to be in the form of “realism.” The goal is to give more without making it impossible to understand. We plan on slowly introducing these new concepts and ideas so community knowledge can propagate nicely.

Also its goofy to be capturing a marine and then its injection sfx, injection sfx, injection sfx, injection sfx, injection sfx. Still other ways to easily OD, and a medic could still do the same, but its still a step in the right direction.

How does this increase medical complexity/roleplay? Well it increases the reliance on medics taking care of marines rather than marines just self medicating themselves. Its only 3 shots per though: except they're refillable groundside in many locations.

Being able to use it without delay isn't incredibly broken either.

Then you should be indifferent to the change. This really needs to be a more compelling argument.

@harryob harryob added the Do Not Merge If you merge this PR, I will annihilate you label Oct 6, 2023
@Git-Nivrak
Copy link
Contributor Author

Then you should be indifferent to the change. This really needs to be a more compelling argument.

I don't see the reasoning here, Maybe I wasn't clear - I meant that being able to use it without delay doesn't hurt the game so the delay is redundant.

The project direction lines you quoted are rather ambiguous and can fit in both how the game stands and how it will be after this revert.

As for the goofy part, yeah it might seem weird to see someone injecting quickly but "goofy" is not what the game is balanced around - It doesn't mean this has to be fully reverted, but it in it's current state it's just bad.

@ghost
Copy link

ghost commented Oct 6, 2023

this pr will start a civil war.

@Steelpoint
Copy link
Contributor

Not even a day and people are trying to revert Morrow's changes.

@Superreallycoolguy
Copy link

morrow leaves and the vultures swarm! so sad!

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. Do Not Merge If you merge this PR, I will annihilate you
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants