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

NanoMed Autoinjectors #20294

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Geevies
Copy link
Contributor

@Geevies Geevies commented Dec 23, 2024

  • Added three inaprovaline and coagzolug autoinjectors to the wall mounted NanoMed.

@Geevies
Copy link
Contributor Author

Geevies commented Dec 23, 2024

!review

@@ -719,6 +719,8 @@
density = FALSE //It is wall-mounted, and thus, not dense. --Superxpdude
vend_id = "meds"
products = list(
/obj/item/reagent_containers/hypospray/autoinjector/inaprovaline = 3,
/obj/item/reagent_containers/hypospray/autoinjector/coagzolug = 3,
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be a new subtype, and put in locations where it's actually needed

or even more ideally, the new subtype could be /expedition and put on horizon shuttles, and have coagzolug and peridaxon (for popped lungs)

cause these nanomeds are used like everywhere, all over horizon, on offships, random away sites, etc, I don't think all of them should have coagzolug

Copy link
Contributor

Choose a reason for hiding this comment

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

actually there's also /obj/machinery/vending/wallmed2 and they're both very inconsistently used and really need a refactor

not requesting a refactor though, but IMO coagzolug shouldn't be added to all nanomeds

Copy link
Contributor

Choose a reason for hiding this comment

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

It doesn't seem too bad to me? If I'm reading the code correctly, coagzolug seems to reduce external bleeding by 20%. The only situation I see it being remarkably useful is when you're a long way from atmosphere and you're bleeding inside your voidsuit, which I don't see any issue with making accommodations for.

I also don't think we need another subtype of the NanoMed. We should be trying to aim for as few subtypes as possible imo, the fewer we have the easier it is to maintain them and the less annoying it is to memorise them while mapping.

@alsoandanswer
Copy link
Contributor

yeah this seems very OP

@FluffyGhoster
Copy link
Contributor

I think it makes sense, but we're literally flooding people with medical things that are already pre-made, so I think if you want to put even more autoinjectors or watever around in the nanomeds, they should be taken from somewhere like the medical pouches in medbay

@NonQueueingMatt NonQueueingMatt added the Changes Required The PR requires changes before it can be approved and/or merged. label Jan 13, 2025
@NonQueueingMatt
Copy link
Contributor

Coagzolug falls under specialized medbay medicine, and I don't want to see an arms race in what gets put in nanomeds and public medkits.

Inaprovaline is fine though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes Required The PR requires changes before it can be approved and/or merged. Review Required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants