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

Goblin Species #890

Merged
merged 94 commits into from
Apr 27, 2024
Merged

Conversation

ErhardSteinhauer
Copy link
Contributor

@ErhardSteinhauer ErhardSteinhauer commented Jan 19, 2024

About the PR

  • New species: Goblins.
  • New accent: Goblin Cant - cockney + rogue cant (apparently it's secret rogue language from 1800s).
  • Goblinkind banner.
  • Food Quality System was moved to different PR: Food Quality System (Goblin) #1159

Why / Balance

To make the common saying "maint goblin" a lot more literal.
And because goblins are cool.

Technical details

  • Takes 10% more damage from brute (blunt, slash, piercing).
  • Takes 20% less poison damage.
  • Deal reduced amount of slashing damage in unarmed melee.
  • Smaller and lighter than dwarfs or felinids.
  • Datasets for name generation.
  • Sounds (filled out attributions and license files, outlined changes to the original files as well).
  • Moderate customization options: ears, noses and tusks.
  • Goblins can jump into mail units and toilets

Media

image
image
image
image
goblin image

  • I have added screenshots/videos to this PR showcasing its changes ingame.

Breaking changes

  • There are some changes done to meat.yml in the base SS14.

Changelog
🆑 erhardsteinhauer

  • add: Added new species - Goblins.
  • add: Added new accent - Goblins Cant.
  • add: Added new decoration piece - Goblinkind Banner.

Copy link
Contributor

github-actions bot commented Jan 19, 2024

RSI Diff Bot; head commit c0f7d90 merging into d30add9
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_NF/Mobs/Customization/goblin_ears.rsi

State Old New Status
goblin_ears_default Added
goblin_ears_default_alt Added
goblin_ears_long_01 Added
goblin_ears_long_02 Added

Resources/Textures/_NF/Mobs/Customization/goblin_noses.rsi

State Old New Status
goblin_nose_crooked Added
goblin_nose_default Added
goblin_nose_long Added

Resources/Textures/_NF/Mobs/Customization/goblin_tusks.rsi

State Old New Status
goblin_tusks_big Added
goblin_tusks_none Added
goblin_tusks_small Added

Resources/Textures/_NF/Mobs/Species/Goblin/organs.rsi

State Old New Status
appendix-inflamed Added
appendix Added
brain-inhand-left Added
brain-inhand-right Added
brain Added
ears Added
eyeball-l Added
eyeball-r Added
heart-off Added
heart-on Added
kidney-l Added
kidney-r Added
liver Added
lung-l Added
lung-r Added
muscle Added
nerve Added
stomach Added
tongue Added
vessel Added

Resources/Textures/_NF/Mobs/Species/Goblin/parts.rsi

State Old New Status
eyes Added
full Added
head_f Added
head_m Added
l_arm Added
l_foot Added
l_hand Added
l_leg Added
r_arm Added
r_foot Added
r_hand Added
r_leg Added
torso_f Added
torso_m Added

Resources/Textures/_NF/Structures/Decoration/banner.rsi

State Old New Status
banner_goblin Added

Edit: diff updated after c0f7d90

@ErhardSteinhauer
Copy link
Contributor Author

YAML Linter flags files I didn't even touched in my PR. I assume that this problem was carried from master.

@ErhardSteinhauer
Copy link
Contributor Author

Oh, I fixed it: I restored the lines I deleted by mistake in speech_emote_sounds.yml.

Copy link
Contributor

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

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

Code wise its seem fine.

But what unique to they bring?
A unique feature of them?

Resources/Prototypes/_NF/Entities/Mobs/Species/goblin.yml Outdated Show resolved Hide resolved
Resources/Prototypes/_NF/Entities/Mobs/Species/goblin.yml Outdated Show resolved Hide resolved
@ErhardSteinhauer
Copy link
Contributor Author

Code wise its seem fine.

But what unique to they bring? A unique feature of them?

So right now it's just visual diversity. But there are planned features for them however those require more knowledge on my part or roping in someone with c# skills.

lowered slashing damage to 3, upped fixture radius to 0.3
@FoxxoTrystan
Copy link
Contributor

Code wise its seem fine.
But what unique to they bring? A unique feature of them?

So right now it's just visual diversity. But there are planned features for them however those require more knowledge on my part or roping in someone with c# skills.

Checkraze is fine adding new species but wishes that they have unique features and not a copy of others species.

Vulps was langauge (rework PR is pending)
Vox is oxygen = bad.
ect...

@ErhardSteinhauer
Copy link
Contributor Author

Code wise its seem fine.
But what unique to they bring? A unique feature of them?

So right now it's just visual diversity. But there are planned features for them however those require more knowledge on my part or roping in someone with c# skills.

Checkraze is fine adding new species but wishes that they have unique features and not a copy of others species.

Vulps was langauge (rework PR is pending) Vox is oxygen = bad. ect...

Well, I have two ideas for goblins that I don't know how to implement right now:

  • Make so they can eat only spoiled and rotten food stuffs: don't get poison from spoiled foods but get poison from normal stuff.
  • Give them the ability to scratch tiles (basically infinite gray colored crayon).

@Wolfking6116
Copy link
Contributor

Having an animal stomach is already somewhat unique, as it's already pre-coded to open up some more options for diet. Personally I am already very excited to be a spess goblin

@FoxxoTrystan
Copy link
Contributor

Having an animal stomach is already somewhat unique, as it's already pre-coded to open up some more options for diet. Personally I am already very excited to be a spess goblin

Vulps, Moth, Lizard, Felinds has animal stomach.

@ErhardSteinhauer
Copy link
Contributor Author

Code wise its seem fine.
But what unique to they bring? A unique feature of them?

So right now it's just visual diversity. But there are planned features for them however those require more knowledge on my part or roping in someone with c# skills.

Checkraze is fine adding new species but wishes that they have unique features and not a copy of others species.
Vulps was langauge (rework PR is pending) Vox is oxygen = bad. ect...

Well, I have two ideas for goblins that I don't know how to implement right now:

  • Make so they can eat only spoiled and rotten food stuffs: don't get poison from spoiled foods but get poison from normal stuff.
  • Give them the ability to scratch tiles (basically infinite gray colored crayon).

Mkay, made so that goblins don't get poisoned from eating rotten meat. As the matter of fact, goblins heal a marginal amount of toxin and brute by eating rotten or goblin meat, so if you're a goblin be wary of goblins near you. I count that as a species gimmick. But I will think how I can improve upon it over time.

@ErhardSteinhauer
Copy link
Contributor Author

Code wise its seem fine.
But what unique to they bring? A unique feature of them?

So right now it's just visual diversity. But there are planned features for them however those require more knowledge on my part or roping in someone with c# skills.

Checkraze is fine adding new species but wishes that they have unique features and not a copy of others species.
Vulps was langauge (rework PR is pending) Vox is oxygen = bad. ect...

Well, I have two ideas for goblins that I don't know how to implement right now:

  • Make so they can eat only spoiled and rotten food stuffs: don't get poison from spoiled foods but get poison from normal stuff.
  • Give them the ability to scratch tiles (basically infinite gray colored crayon).

Mkay, made so that goblins don't get poisoned from eating rotten meat. As the matter of fact, goblins heal a marginal amount of toxin and brute by eating rotten or goblin meat, so if you're a goblin be wary of goblins near you. I count that as a species gimmick. But I will think how I can improve upon it over time.

Also cooked food (anything that somehow gets that Flavorol reagent attached to it) is now poisonous for goblins, so don't eat at McCargo (well, unless they are willing to fix you up with some of that rotten meat they stockpile near the toilets).

@GentleButter
Copy link

WOOOOO LETS DO THIS

@github-actions github-actions bot added the Merge Conflict This PR has conflicts that prevent merging label Apr 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict This PR has conflicts that prevent merging label Apr 20, 2024
ErhardSteinhauer and others added 20 commits April 21, 2024 03:21
…rs-14#884)

* Adds howling / growling noises

* revenge of the spelling error!! grahh!!

* trying to fix that error with the schema validator

* Fixing copyright + adds vulp growls to harpy emote

* The case of the missing attributions dashes.

* Fixes up attributions.yml with license.txt info

* Update Resources/Audio/DeltaV/Voice/Vulpkanin/attributions.yml

Co-authored-by: DEATHB4DEFEAT <[email protected]>
Signed-off-by: Adeinitas <[email protected]>

---------

Signed-off-by: Adeinitas <[email protected]>
Co-authored-by: DEATHB4DEFEAT <[email protected]>
@github-actions github-actions bot added Map-Outpost Map - Outpost and removed YML labels Apr 27, 2024
Copy link
Contributor

@dvir001 dvir001 left a comment

Choose a reason for hiding this comment

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

Im going to gabble gabble

@dvir001 dvir001 merged commit 2ed0520 into new-frontiers-14:master Apr 27, 2024
14 checks passed
FrontierATC added a commit that referenced this pull request Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C# FTL Map-Outpost Map - Outpost Sprites Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants