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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds the Family Heirloom, Nyctophobia, and Monochromacy traits, and rebalances Social Anxiety #36426

Merged
merged 5 commits into from Mar 28, 2018

Conversation

ghost
Copy link

@ghost ghost commented Mar 14, 2018

馃啈 Xhuis
add: Added the Family Heirloom trait. This makes you spawn with a special object on your person; if you don't have it with you, you will gain a mood debuff.
add: Added the Nyctophobia trait. This makes you walk slowly in complete darkness, and gain a mood debuff from the fear!
add: Added the Monochromacy trait, which makes you perceive (almost) the entire world in black-and-white.
balance: Social Anxiety's trigger frequency is now correlated to the number of people near you.
/:cl:

Thanks to PKP for the Nyctophobia and Monochromacy ideas. Thanks to Polyphynx on the forums for the input about Social Anxiety.

Picture of Monochromacy:
image
I'm not sure why lights are still perceived in color, which is why I included the "almost" bit.

  • Family Heirloom (minor neg.): You spawn with an item in your backpack and gain a mood debuff if you don't have it on your person, and a small boost when you do. This can be a d20, deck of cards, crowbar, or Zippo lighter by default. The object itself is named "(last name) family (normal name)" to distinguish it. The object spawned also varies based on job:
    • Clowns get a golden bike horn
    • Mimes get a baguette
    • Lawyers get a gavel
    • Janitors get a mop
    • Security Officers get a space law book
    • Scientists get a slime plushie
    • Assistants get a rusty toolbox that starts empty, does low damage, and fits in backpacks
    • On April Fools' Day, the heirloom can be any item in the game, using the same limitation code as the Christmas gifts (only valid items, so non-abstract and has an icon)
  • Nyctophobia (minor neg.): You can't run in complete darkness, and are forced to walk instead. You also gain a small mood debuff from fear. Combine with Night Vision for the ultimate meta.
  • Monochromacy (neutral): You can't see colors. The entire world (except for light) is black-and-white to you.

I also reordered the Jolly, Depression, and Apathetic traits to be alphabetical in the menu because it pleases my autism. Social Anxiety was rebalanced so that you didn't randomly go mute for no reason, and because it makes more sense to get anxious when talking to a group of people than at random. Thoughts and input are welcome.

@tgstation-server tgstation-server added Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways labels Mar 14, 2018
@ghost
Copy link

ghost commented Mar 14, 2018

add: Added the Family Heirloom trait. This makes you spawn with a special object on your person; if you don't have it with you, you will gain a mood debuff.

AAAAA I LOVE YOUUU

@optimumtact
Copy link
Member

optimumtact commented Mar 14, 2018

it's because the lights are on the wrong layer, which is why they don't spin when you rotate the players HUDs either

edit:wrong plane probably, but it might be for lighting reasons, cna't remember

@@ -11,6 +11,23 @@



/datum/trait/apathetic
Copy link

Choose a reason for hiding this comment

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

undocumented change (not that i mind the change beind made)

Copy link
Author

Choose a reason for hiding this comment

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

I also reordered the Jolly, Depression, and Apathetic traits to be alphabetical in the menu because it pleases my autism.

@xenon-difluoride
Copy link
Contributor

On April Fools' Day, the heirloom can be any item in the game, using the same limitation code as the Christmas gifts (only valid items, so non-abstract and has an icon)

Hell yes.

/datum/trait/family_heirloom
name = "Family Heirloom"
desc = "You are the current owner of an heirloom. passed down for generations. You have to keep it safe!"
value = -1
Copy link

Choose a reason for hiding this comment

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

Minus points for what is essentialy free mood unless you suck?

Copy link
Author

Choose a reason for hiding this comment

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

You have to clutter your inventory to lug around a random object that, in likelihood, won't have any use.

Copy link

Choose a reason for hiding this comment

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

just have it be 0 value to be safe

Copy link
Author

Choose a reason for hiding this comment

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

I view the concept as inherently negative, so I might end up removing the mood boost instead.

Copy link

Choose a reason for hiding this comment

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

it has a passive buff, but I suppose its fair for the 1 mood boost and it means this is an attractive negative trait pick, which I like. So fine with me :D

Copy link
Member

Choose a reason for hiding this comment

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

agreed

Copy link

Choose a reason for hiding this comment

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

@Xhuis also dont remove the mood boost just keep it like this tbf

@silicons
Copy link
Contributor

monochromacy

Time for detecting!

@XDTM
Copy link
Contributor

XDTM commented Mar 14, 2018

Can you make trauma versions of monochromacy and nyctophobia? They're pretty cool

@Tacolizard
Copy link
Contributor

Tacolizard commented Mar 14, 2018

monochromacy should give 1 point, because you can't see hacked APCs, some syndie gear, etc

@ghost
Copy link
Author

ghost commented Mar 14, 2018

Playing monochromatically significantly affects perception, but not gameplay; someone who plays with monochromacy will be able to catch onto the tints of objects and be able to distinguish them from normal ones at a glance, much like monochromatic patients do in real life. I don't think it detracts enough from gameplay to be worth a point, but I'll have to consider it.

@Hatterhat
Copy link
Contributor

the official meta of monochromacy is

play det
activate monochromacy
play the Gunpoint OST

case in point:
https://www.youtube.com/watch?v=AASeTbHFvSQ
https://www.youtube.com/watch?v=6LZ12RGcU1U

@PKPenguin321
Copy link
Contributor

>tfw you post your idea on the forums and some madman actually codes it

feels good, thanks

@optimumtact
Copy link
Member

a detective with the monochromatic should spawn with the original revolver that fires real bullets in honour of their awesomeness

@Tacolizard
Copy link
Contributor

det bullets are better than real bullets because they stun

@optimumtact
Copy link
Member

the original revolver would crit you in two shots taco you newfag

@Tacolizard
Copy link
Contributor

ree

@ghost
Copy link
Author

ghost commented Mar 15, 2018

detectives with monochrome do get a special sound and message cue, so they don't go entirely unacknowledged

@cebutris
Copy link
Contributor

Loss of color doesn't seem like a huge deal at first, but it's a pretty significant drawback. I wouldn't go so far as to say that it's a major setback, but it's certainly not something to be written off.

Try playing a round as security with SecHUDs and glasses color enabled (alt-click the glasses), you'll see what I mean

@81Denton
Copy link
Member

Monochromacy is going to be fun as an atmos tech lmao

Copy link
Contributor

@Jalleo Jalleo left a comment

Choose a reason for hiding this comment

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

Just some comments you can take them aboard or throw them out (Messaged from my mobile sorry for spelling errors)

heirloom_type = /obj/item/storage/toolbox/mechanical/old/heirloom
if(!heirloom_type)
heirloom_type = pick(
/obj/item/toy/cards/deck,
Copy link
Contributor

Choose a reason for hiding this comment

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

Can I suggesy it be basically anything from the arcade machine?

Copy link
Member

Choose a reason for hiding this comment

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

McUrist family pulse rifle




/datum/trait/family_heirloom
Copy link
Contributor

Choose a reason for hiding this comment

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

What we then need next is a possibility for antags to buy a steal heirloom trait which can give them a positive buff.

@imtakingabreakdontatme
Copy link
Contributor

Traits and moods are both config options. What happens to traits based around mood if moods are toggled off? Do they become freebies?

@ghost
Copy link
Author

ghost commented Mar 15, 2018

Mhm, they do. It'd be very easy to add a way to disable them from appearing in the menu if moods are disabled. I can see about doing that after this PR is in, or in the PR itself if wanted.

@epochayur
Copy link
Contributor

that monochromacy is seriously cool, you're doing god's work xhuis

@ghost
Copy link

ghost commented Mar 15, 2018

@KorPhaeron ofcourse nobody would disable mood right?

@Iamgoofball
Copy link
Contributor

Iamgoofball commented Mar 16, 2018

@KorPhaeron The solution is we remove the config options and make them default, and stop being cowards about a vocal minority quitting for hippie just because we made a game design decision

if(mood)
if(heirloom in trait_holder.GetAllContents())
mood.clear_event("family_heirloom_missing")
mood.add_event("family_heirloom", /datum/mood_event/family_heirloom)
Copy link

Choose a reason for hiding this comment

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

Use the send signals instead (changed due to my pr)

@ghost
Copy link
Author

ghost commented Mar 19, 2018

Sorry about not working on this lately, other stuff has come up + haven't had time to work on it. I haven't forgotten about this, so I'll get some work done when I'm able.

@ShizCalev ShizCalev added the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Mar 20, 2018
@ShizCalev
Copy link
Member

conflict

@imtakingabreakdontatme
Copy link
Contributor

@Iamgoofball @Xhuis

Maybe eventually yeah, but until then you need to make sure it works with whichever config settings a server is using

@drawsstuff
Copy link

If you're adding monochrome for the detective, at least do it the way para did and make things such as discarded bullets and blood still colored.

here's a shitty mock up
mock up

@tgstation-server tgstation-server removed the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Mar 20, 2018
@ghost
Copy link
Author

ghost commented Mar 20, 2018

@drawsstuff A neat idea, but my prodding couldn't figure out any immediate ways to do it, so if I do it I'll wait till after this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet