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

Adds a Synth Species #13122

Merged
merged 3 commits into from Dec 10, 2015
Merged

Adds a Synth Species #13122

merged 3 commits into from Dec 10, 2015

Conversation

RemieRichards
Copy link
Contributor

Changes

  • Adds Synth Species
  • Adds a method for species to react to being force set by an admin.

Synth Species

  • Imitates a species - Say mods, Icons, Mutant parts, etc.
  • When health reached disguise_fail_health their synthflesh falls off partially, revealing their metal skeleton beneath
  • Synthflesh can only be repaired by drinking the reagent, Synthflesh. Drunk synthflesh heals Synths more than synthflesh normal heals for.
  • /synth/military subtype has in built armour (25) and increased punch damage (+10)

Damaged skin (Correctly handles species)
species

Base icon_state adapted from Bay's IPCs, synthflesh automatically created from existing icons via code.

Idea adapted from https://tgstation13.org/phpBB/viewtopic.php?f=9&t=5212&p=134868#p134868
I spoke with GUN_HOG about him using this species datum for the constructed synths in the thread.

This can be used to have people spawn as their characters, and then become some kind of Synth Antag so they're still their character, but technically a synth.

🆑
add: Adds a Synth Species
add: Species can now react to being force set by admins, used for Synth species
/:cl:

…y reach 50 HP their skin partially falls off. Adds a method for species to react to being force set by an admin.
@RemieRichards RemieRichards added the Feature Exposes new bugs in interesting ways label Nov 19, 2015
safe_co2_max = 0
SA_para_min = 0
SA_sleep_min = 0
dangerous_existence = 1
Copy link
Contributor

Choose a reason for hiding this comment

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

They're not actually dangerous to be?
They might be attacked or killed, but they're not actually dangerous like plasmamen or shadowpeople are.

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 forget why I set this var, I did it like an hour ago.
I do assume people wont be so friendly towards a robot with flesh dangling off of it though.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah this is for stuff that's deadly to be under normal circumstances rather than deadly to be because you're a scary antag.
If there's a code reason for it it's okay I guess though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

well dangerous_existence species are converted to humans when they get normal antag roles, so I'd like that here.

@ExcessiveUseOfCobblestone
Copy link
Contributor

well dangerous_existence species are converted to humans when they get normal antag roles, so I'd like that here.

Wait so you can metagame someone by looking at their name?

IE if I'm plasmaman and I'm named Xenon 44, I get traitor tot and become human, people will know I'm a traitor tot because I'm a human with a Pbro's name?

@daman997
Copy link

this looks cool 👍

@Zilenan91
Copy link

I could see there being 1 or 2 of these at round start every round with the "Avoid detection" antag objectives. They allow antagonistic freedom as well as preventing mass murderbone.

@Incoming5643
Copy link
Contributor

Being an antag and having dangerous_existence does NOT turn you into a human. Currently dangerous_existence only makes sure that magic shit doesn't turn people into suitless plasmamen and the like.

@RemieRichards
Copy link
Contributor Author

@Incoming5643 it does turn them Human, it's in @as334's PR. I had thought it already did this partially, but if you claim not, then suuuure.

@ExcessiveUseOfCobblestone it's not all antag roles, it's the weirder ones like Nuke Ops, who obviously can rename themselves anyway.

@Zilenan91 ya, Stealth would very VERY much be the focus of this as a side antag, but for now, admin species. Admins can probably trial how they think it should go too, like how shadowling was an event long before it was a gamemode.

…hem. Moves "Op" stuff to a /synth/military subspecies.
@Supermichael777
Copy link
Contributor

now all we need is the ability for the combat synths to hack cloning computer boards and HUD icons and we can have another shitty team conversion antag!

they should repair like augs Tbh. tox and oxy imunity too.



/datum/species/synth/handle_chemicals(datum/reagent/chem, mob/living/carbon/human/H)
if(chem.id == "synthflesh")
Copy link
Contributor

Choose a reason for hiding this comment

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

aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's how it's all done I'm afraid, check every other handle_chemicals, and everything else about the reagent system, it's all chem.id checks.

Copy link
Member

Choose a reason for hiding this comment

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

should be a blame goof tag

Copy link
Contributor

Choose a reason for hiding this comment

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

make this reagent do some dna affect() new proc or something

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 shit and there's gotta be a better way to redo this, i want to strangle whoever set this handle_chemicals() shit in species code up

@RemieRichards actually no, in SANE CHEMISTRY USES we have a has_reagent() proc or some shit, and quite frankly this should be handled on the reagent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Iamgoofball look at handle_chemicals for all species that have one, nerd.

@Supermichael777
Copy link
Contributor

but is it merge ready?

@Jordie0608 Jordie0608 added the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Nov 29, 2015
@MMMiracles
Copy link
Contributor

dont let the terminator die

@RemieRichards
Copy link
Contributor Author

It's been merge ready for days (it's just conflicting now) the thing Aran's on about is inconsistent to the current species+reagent code and I won't be doing it.

I'll fix conflicts soon.

@RemieRichards RemieRichards removed the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Dec 4, 2015
@RemieRichards
Copy link
Contributor Author

@tgstation/commit-access cmon you guys

@GunHog
Copy link
Contributor

GunHog commented Dec 5, 2015

Yes! For a Remie Richards feature to be done in less than a month is amazing! MERGE!

if(chem.id == "synthflesh")
chem.reaction_mob(H, TOUCH, 2 ,0) //heal a little
handle_disguise(H) //and update flesh disguise
H.reagents.remove_reagent(chem.id, REAGENTS_METABOLISM)
Copy link
Contributor

Choose a reason for hiding this comment

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

aaaAAAAAAAAAAAAAAAAAAAAAAA
AAAAAAAAAAAAAAAAAAAAAAAAAA

standardize this with the proper chemistry procs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this IS proper chemistry procs!

Copy link
Contributor

Choose a reason for hiding this comment

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

can't you just add a species check to synthflesh to heal the disguise

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Handling a species thing anywhere but species

Cmon goof.

Copy link
Contributor

Choose a reason for hiding this comment

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

using direct chem.id string checks with ==

Copy link
Contributor Author

Choose a reason for hiding this comment

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

using checks THE REST OF THIS INSTANCES OF THIS PROC DO

cmon now goof, we both know the entire SYSTEM is a sack of garbage, and fixing this one instance changes -nothing-

Copy link
Contributor

Choose a reason for hiding this comment

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

Little things add up

@Dorsisdwarf
Copy link
Contributor

rrrrrrrrrrrrrrrrrrr

@optimumtact
Copy link
Member

eeeeeeeeeeeeeeeee

@GunHog
Copy link
Contributor

GunHog commented Dec 9, 2015

This is a RemieRichards feature. A minimum of 30 days must pass before she can complete it. You are 10 too early!

(I am joking RR plz no hurt me)

@RemieRichards
Copy link
Contributor Author

@GunHog Before I can complete it? IT'S BEEN COMPLETE FOR DAYS.

@Jazaen
Copy link

Jazaen commented Dec 9, 2015

Where is Raz when you need him?

@Cheridan
Copy link
Contributor

7.6/10 its ok

Cheridan added a commit that referenced this pull request Dec 10, 2015
@Cheridan Cheridan merged commit 5004bcd into tgstation:master Dec 10, 2015
@PKPenguin321
Copy link
Contributor

happening

@Dorsisdwarf
Copy link
Contributor

DUDUDUDUDUH

@Jazaen
Copy link

Jazaen commented Dec 11, 2015

Now, we just need someone to add a procedure of gluying all borg pieces onto a special frame and adding brain/posibrain with some blood for identity, and voila, you can hide on derelict in a base with one way teleporter, abducting crew and replacing them for fun and profit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet