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

Moths #34243

Closed
wants to merge 1 commit into from
Closed

Moths #34243

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 10, 2018

New PR opened because I rebased old one.

<pineapples> reopen button greyed out

🆑 More Robust Than You
add: Adds Moths! They're a new species that have (unusable due to gravity and shit) wings, and are prone to autism.
/:cl:

Why:

CREDITS TO @tortellinitony, @FuryMcFlurry, and @TDcoolguy300 for their original work on moths!

@ghost ghost requested a review from ShizCalev as a code owner January 10, 2018 00:55
@tgstation tgstation deleted a comment from IconDiffBot Jan 10, 2018
@mirrorcult
Copy link
Contributor

no

@ghost
Copy link

ghost commented Jan 10, 2018

FUCKING SCREAMING
LIKE A FUCKING LUNATIC

@lordpidey
Copy link
Contributor

@ghost
Copy link
Author

ghost commented Jan 10, 2018

capture

it begins

@optimumtact
Copy link
Member

Add that when a moth player sees a torch they have to compulsively follow it

@@ -241,3 +241,9 @@ GLOBAL_VAR(syndicate_code_response) //Code response for traitors.
. += "."
else
. += ", "

/proc/moth_name(gender)
Copy link
Contributor

Choose a reason for hiding this comment

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

I can't tell the difference between the names in each list so you should probably reconsider this proc.

if(ishuman(user))
var/mob/living/carbon/human/H = user
if(H.dna.species.scream_sound)
playsound(H.loc, H.dna.species.scream_sound, 50, 1, 4, 1.2)
Copy link
Member

Choose a reason for hiding this comment

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

I do not think this is a good idea

Copy link
Contributor

Choose a reason for hiding this comment

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

no cooldown

GOD NO.

Choose a reason for hiding this comment

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

nonsense this is fantastic

Copy link
Contributor

Choose a reason for hiding this comment

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

It's a port from hippie's screamsound mechanics. This is an unlogged change that should probably get added to the changelog.

Copy link
Contributor

Choose a reason for hiding this comment

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

It also calls ..() twice in some situations which may be a bad

@AKlimentyev
Copy link

Leave the scream in. Without it, this pr is bad. with it, it's fantastic.

@Nukie2557
Copy link

Let them headbutt lights and break them on accident.

@ghost
Copy link
Author

ghost commented Jan 10, 2018

No pc access rn you fuck

@AKlimentyev
Copy link

The actual default variant of moth wings isn't set correctly, resulting in wingless moths.

@ghost
Copy link
Author

ghost commented Jan 10, 2018

Balls

@LuamTheElf
Copy link

LuamTheElf commented Jan 10, 2018

I freaking love this.
Somethings to consider is to leave the cooldown proc off, and fix the wingless moth bugs.

@Nukie2557
Copy link

I'm...Worried, to say the least. As what goof said, Slippery slope. Maybe i'm just becoming disabled and have a lead pipe through my skull, but slippery slope, man. We already have citadel for this.

@Daturix
Copy link
Contributor

Daturix commented Jan 10, 2018

Moths are the best thing to be added since atmospherics.

@optimumtact optimumtact added Feature Exposes new bugs in interesting ways Test Merge Candidate You're our unpaid test team labels Jan 10, 2018
@optimumtact optimumtact added this to the Fuckup of the year 2018 milestone Jan 10, 2018
@tgstation tgstation deleted a comment Jan 10, 2018
@BlueNexus
Copy link

BlueNexus commented Jan 10, 2018

What're their unique features? Without features to make them distinct, they're just reskinned humans. Screaming is not a unique feature.

@Wilchenx
Copy link
Contributor

@BlueNexus
image

@BlueNexus
Copy link

BlueNexus commented Jan 10, 2018

So, let's see:

  1. Lose wings at high temperatures
  2. Damaged by pestkiller
  3. Takes more damage from flyswatters

That's barely anything. If you're going to add a silly race, at least make them interesting - A species's unique mechanics shouldn't consist of a mere 21 lines of code. They have wings, right? You could do something with that, rather than just saying "They're unuseable due to gravity.". In exchange, you could make them a highly fragile species. That's just a suggestion - Figure out a way to make them distinct.

@ghost
Copy link
Author

ghost commented Jan 10, 2018

@BlueNexus Tell me, how are lizards unique?

@BlueNexus
Copy link

BlueNexus commented Jan 10, 2018

They aren't - But they should be. It makes very little sense that a species which evolved on an entirely different planet would be exactly identical to humans except in appearance. Regardless, this is a good opportunity to make a unique and interesting species.

@Wilchenx
Copy link
Contributor

@BlueNexus apply the same arguments to lizards

@ghost
Copy link
Author

ghost commented Jan 10, 2018

Moths now walk towards light.

@Wilchenx
Copy link
Contributor

@MoreRobustThanYou bad, will be abused to grief. Make it a flavor text instead.

@FantasticFwoosh
Copy link
Contributor

FantasticFwoosh commented Jan 10, 2018

Lizards are vunerable to cold but have heat resistance, have unique blood, can't possess certain genetics powers and have a tail organ ontop of cosmetic decals for horns/spine fins etc.

Real question, how are mothpeople different to flypeople? If you want people to take this PR seriously.

  • Flypeople also vunerable to swatter and pestkiller, so its just some snowflake wings.

@tgstation tgstation deleted a comment from Spess-Rando Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from ACCount12 Jan 14, 2018
@tgstation tgstation deleted a comment from EmptyLullaby Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from EmptyLullaby Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from tortellinitony Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from EmptyLullaby Jan 14, 2018
@tgstation tgstation deleted a comment from ACCount12 Jan 14, 2018
@tgstation tgstation deleted a comment from EmptyLullaby Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from Spess-Rando Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from tortellinitony Jan 14, 2018
@tgstation tgstation deleted a comment Jan 14, 2018
@tgstation tgstation deleted a comment from praisenarsie Jan 14, 2018
@tgstation tgstation deleted a comment from Nukie2557 Jan 14, 2018
@imtakingabreakdontatme
Copy link
Contributor

Can you guys stop shitting up a PR that was made in good faith and is popular with a majority of the community in some awful attempt to settle petty discord grudges with a third party

@tgstation tgstation deleted a comment from Cheridan Jan 15, 2018
@@ -257,6 +256,8 @@
message_mime = "acts out a scream!"
emote_type = EMOTE_AUDIBLE

//here lies moth *scream. 2018-2018, murdered by oranges.
Copy link
Member

Choose a reason for hiding this comment

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

remove this MRTY you meme

to_chat(M, "<span class='italics'>You hear a buzzing in your ears.</span>")
human.set_species(/datum/species/fly)
else
to_chat(M, "<span class='italics'>You hear fluttering in your ears.</span>")
Copy link
Member

Choose a reason for hiding this comment

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

what justification do you have for this?

If they're going to be added it'll be a server decision to enable them as a race

Copy link
Member

Choose a reason for hiding this comment

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

the whole fly thing is a backported reference and isn't subject to the same rules

liked_food = VEGETABLES | DAIRY
disliked_food = FRUIT | GROSS
toxic_food = MEAT | RAW
//<pineapples> morty you need to listen to the people who have the merge button not randos in sligcord
Copy link
Member

Choose a reason for hiding this comment

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

remove this as well

@Cyberboss
Copy link
Member

Joan did better

@Cyberboss Cyberboss closed this Jan 16, 2018
@ghost ghost deleted the mawths branch January 23, 2018 11:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Config Update Time to bother the headadmins for three months to get your config applied Do Not Merge You must have really upset someone Feature Exposes new bugs in interesting ways Sound Oh god my ears, they bleed, did you normalise the volume? Sprites A bikeshed full of soulless bikes. Test Merge Candidate You're our unpaid test team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet