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

Cyborg Fixes #4790

Closed
wants to merge 1 commit into from
Closed

Cyborg Fixes #4790

wants to merge 1 commit into from

Conversation

BurgerLUA
Copy link
Contributor

@BurgerLUA BurgerLUA commented May 23, 2018

Overview

Cyborgs were missing a lot of things they should be able to do, such as carry slime cores or mech parts, for the science module. The medical module hypospray was missing kelotane, for some reason.

Changelog

  • Broken cyborg syringes can now be repaired inside the cyborg charger.
    (If you accidentally broke a syringe as a cyborg, there was no way to repair it unless you had a module reset.)
  • Science Cyborgs now have multitools, wires, and a welder, instead of NanoPaste. Missing Hemostat and Retractor for cyborgifcation were also added. A slime bag and slime scanner were also added. Science Grippers can now carry slime cores, slime potions, and Exosuit parts.
    (Nanopaste is incredibly fucking OP for a cyborg to have, especially when it regenerates. Even then, I think the charge check is broken and you can have unlimited nanopaste. Literally in XenoBiology, I could tank slimes/creatures easily and just give myself nanopaste and all would be fine. I know it feels like that should be in development, but this is honestly a bugfix that is needed. The science cyborg needs the hemostat, retractor, and multitool in order to perform cyborgication as well as some IPC related repairs. The welder tool and wires replace the nanopaste for repairs and assembly.)
  • Added missing Kelotane from borg hypospray. Medical cyborgs have their own inhalers. Medical crippers can now grab blood packs, chem dispensers, and inhaler cartridges.
    (Medical borgs didn't have kelotane for some reason in their hyposprays)
  • Service Cyborgs can now produce Sarezhi Wine and Xuxzi Juice in their shakers.
    (For the Unathi, I guess.)

@Sindorman
Copy link
Contributor

Can you also make janitor bots start with full advanced lights replacer?

@Chada-1
Copy link
Contributor

Chada-1 commented May 24, 2018

I'd also suggest giving the Rescue module inhalers, nothing else, though. They should have them more than the Medical module IMO. However! if you do that, make them stuck with the default cartridge or whatever, and not able to trade it out for different regents. I heard you could do that. Only the Medical Module should be able to swap out cartridges.

@BurgerLUA
Copy link
Contributor Author

Give a mouse a cookie, and they'll ask for a glass of milk.

@LordFowl
Copy link
Contributor

This is a bugfix PR. Any additional developments would require a developmental PR.

src.emag = new /obj/item/weapon/hand_tele(src)
src.modules += new /obj/item/weapon/storage/bag/slimes(src)
src.modules += new /obj/item/device/slime_scanner(src)
src.modules += new /obj/item/weapon/weldingtool/experimental(src)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a lot of modules, and seems an awful bit like powercreep and not even an insidious kind. With the addition of hemostat and retractors, along with slime bags and slime scanners, it seems like the science cyborg is capable of pretty much doing every science job perfectly, which imo is not ideal. There is nothing worse than coming on shift and finding a robot has already done your job for you. Not to mention the design philosophy that cyborgs should be limited in their versatility - specialists and not generalists.

Either way, these changes aren't particularly suitable for a master-targeted PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll remove the slime scanners and the extra surgery equipment. NanoPaste is still a bug tbh.

changes:
- bugfix: "Broken cyborg syringes can now be repaired inside the cyborg charger."
- balance: "Science Cyborgs now have multitools, wires, and a welder, instead of NanoPaste. Missing Hemostat and Retractor for cyborgifcation were also added. A slime bag and slime scanner were also added. Science Grippers can now carry slime cores, slime potions, and Exosuit parts."
- bugfix: "Added missing Kelotane from borg hypospray. Medical cyborgs have their own inhalers. Medical crippers can now grab blood packs, chem dispensers, and inhaler cartridges."
Copy link
Contributor

Choose a reason for hiding this comment

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

grippers instead of crippers

And how is adding an inhaler a bugfix? I don't remember the actual crunch of inhalers particularly well. Are they necessary for cyborgs now?

@scheveningensun
Copy link
Contributor

Give a mouse a cookie, and they'll ask for a glass of milk.

God forbid people exercise their right to make additional suggestions to a QOL pr

@skull132
Copy link
Contributor

Updated to bypass develop but this PR doesn't fit criteria. Only first and potentially third point are fit for merge into master. The rest are unimportant balance changes and go into dev.

@skull132 skull132 added the Changes Required The PR requires changes before it can be approved and/or merged. label May 28, 2018
@BurgerLUA
Copy link
Contributor Author

I'll close this then and put it on dev next cycle.

@BurgerLUA BurgerLUA closed this May 29, 2018
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.

None yet

6 participants