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

(finished) Inferior Weapons Patch #14721

Merged
merged 34 commits into from Jan 16, 2016

Conversation

Projects
None yet
10 participants
@evilexecutive
Copy link
Contributor

commented Jan 5, 2016

People have been talking recently about how relatively powerful the medieval weapons are in this game, and some ideas have been thrown around to see how their issues can be fixed. One such idea was to make the medieval weapons far less likely to spawn, which would encourage players to make their own. However, I also had the helpful idea of making "inferior" versions of the Medieval Weapons, that are not outright fakes, but just so happen to be considerably less durable.

This patch seeks to help counterbalance the Medieval Weapons mod by introducing a small number of medieval weapons that are made out of low grade steel. They are indistinguishable; statwise, from the actual weapons, but their inferior material makes them less than battle-ready. Generally my idea is that in the modern day, the vast majority of weapons advertised as "Battle-ready" are often made from things like inferior grades of steel, and have very poor construction choices such as rat-tail tangs.

I'll add in more weapons to use later, and likely update the welder to also repair Stainless Steel.

Done So Far(this list is more for me than anyone else really):
Wakizashi
katana
zweihander
jian
scimitar
longsword
arming sword
battleaxe
cutlass
broadsword

evilexecutive added some commits Jan 5, 2016

Adding Inferior metal materials
This is a new material type intended for use in "Inferior" grade weapons. While not quite fake, weapons made from this material should look and act very similarly to the real deal, but will be [b]Considerably[/b] less durable.
Merge pull request #1 from evilexecutive/patch-5
Adding in Inferior Weapons
["cutlass", 1],
["rapier", 1],
["wakizashi", 1],
["jian_inferior", 30]

This comment has been minimized.

Copy link
@Malkeus

Malkeus Jan 5, 2016

Contributor

Missed a comma here, and the 3 lines below this.

evilexecutive added some commits Jan 5, 2016

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jan 5, 2016

@evilexecutive evilexecutive changed the title Inferior Weapons Patch (WIP)Inferior Weapons Patch Jan 5, 2016

@Coolthulhu

This comment has been minimized.

Copy link
Contributor

commented Jan 5, 2016

Are they supposed to be unrepairable?
This makes sense - if they were repairable, it would be tedious (not hard) to keep them working.

@evilexecutive

This comment has been minimized.

Copy link
Contributor Author

commented Jan 5, 2016

I'm currently mulling over having them be repairable. I've also changed the material name to budget steel, as per a suggestion on the IRC. I'm not quite sure yet, as part of me feels like it would be an exercise in futility to try and get these weapons to retain their edge.

@Malkeus

This comment has been minimized.

Copy link
Contributor

commented Jan 5, 2016

You can barely put an edge on bad steel, let alone maintain one. And it will be gone the first time you cut something with it.

@Coolthulhu

This comment has been minimized.

Copy link
Contributor

commented Jan 5, 2016

Leaving them unrepairable would make the whole thing much more significant.
If they were repairable, you'd only need to solder them after more serious fights, which would defeat the whole purpose.

@Malkeus

This comment has been minimized.

Copy link
Contributor

commented Jan 5, 2016

I like this. Suddenly hording all the things will be an asset and not just a compulsion. I'll break all the swords til I find one that isn't fake.

@Malkeus

This comment has been minimized.

Copy link

commented on data/json/item_groups.json in 841b9b1 Jan 5, 2016

commas

@Malkeus

This comment has been minimized.

Copy link
Contributor

commented Jan 5, 2016

commas are the single most important thing in the entire game. next is brackets and braces.

evilexecutive added some commits Jan 5, 2016

Adding the battleaxe.
This time with a comma
Adding back stainless, and obsidian
Stainless Steel is a variant of steel with less cut and chip restistance, but a higher acid resistance.

Obsidian is a glass material with no durability of any kind at all, except for an extreme amount of acid resistance. Ideally, blades made from it should have an [b]extremely high[/b] cutting ability, at the consequence of breaking after only a single swing.
"name": "Macuahuitl",
"description": "This is an ancient sword that was once used by the aztecs. Its edge menaces with shiny flakes of obsidian. Even the gentlest of touches still breaks your skin.",
"price": 800000,
"material": "obsidian, wood",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

Are you sure this way of material definition works?

"color": "dark_gray",
"name": "cutlass",
"name_plural": "cutlasses",
"description": "This is a broad saber known for its use by sailors and pirates. This sword appears to be made very poorly, but it should still stand up to a few swings.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

double spaces.

"symbol": "/",
"color": "light_gray",
"name": "katana",
"description": "This is a rare sword from Japan. While it's got the right edge and weight, the pommel just snaps off, and the blade seems pretty worn.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

Double spaces required.

"color": "light_gray",
"name": "wakizashi",
"name_plural": "wakizashi",
"description": "This is a comparatively-common Japanese short sword. There's something not quite right about this sword.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

2x spaces.

"symbol": "/",
"color": "light_gray",
"name": "battle axe",
"description": "This is a huge axe designed for warfare. You notice that the axe heads aren't seated properly at all.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

2x spaces.

"color": "light_gray",
"name": "jian",
"name_plural": "jian",
"description": "This is an ancient Chinese doubled-edged straight sword. The sword is quite worn, and bent at an odd angle.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

2x spaces.

"symbol": "/",
"color": "light_gray",
"name": "scimitar",
"description": "This is a curved sword associated with various middle eastern and central asian countries. This one seems oddly dull and worn.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

2x spaces.

"symbol": "/",
"color": "light_gray",
"name": "arming sword",
"description": "This is a classic medieval sword, just the right size to use one-handed. This one doesn't seem to have been made right.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 6, 2016

Member

2x spaces.

@SeanMirrsen

This comment has been minimized.

Copy link
Contributor

commented Jan 6, 2016

Don't know how correct it is to ask, but I have to wonder - why double spaces? I mean, beyond the "apparently that's the way it's done" thing, because I've no idea if some text formatting mechanic depends on it or it's just a thing that you people do.

@illi-kun

This comment has been minimized.

Copy link
Member

commented Jan 6, 2016

@SeanMirrsen please see discussion in #5414. Summary is

Single space after periods is for proportional fonts.
Double space after periods is for fixed-width fonts.
We use fixed-width fonts.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jan 6, 2016

That summary technically does not explain the actual WHY, but as far as I can tell the reasoning is for readability and (secondarily) to maintain consistency.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jan 6, 2016

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jan 6, 2016

Which would fall under readability, yeah. XP

@evilexecutive

This comment has been minimized.

Copy link
Contributor Author

commented Jan 7, 2016

I've removed the content that didn't contribute to the original purpose of this PR, now it purely deals with Inferior Weapons type. I'd keep the other stuff to a seperate PR, if I ever decide to do anything with them.

@evilexecutive

This comment has been minimized.

Copy link
Contributor Author

commented Jan 7, 2016

So, for putting up Replica/Fake for the Fake and Inferior Items, I honestly wouldn't bother with that. The point being that if you enter a building and just give a cursory glance over the contents, you aren't making a close inspection of all the items in it. To that cursory glance, Fakes, Replicas, and genuine swords all look the same to you. It's when you pick it up and examine the item do the differences show up.

@Coolthulhu

This comment has been minimized.

Copy link
Contributor

commented Jan 7, 2016

'V' takes zero time. It will reveal whether the items are fake or real.
The only things same names will cause is annoyance of having to read stats and not just the name, making meta knowledge more important. Gimmicks like that are good maybe for one playthrough, if that.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jan 7, 2016

Hmm. So I'm not sure if it'll really do anything except add a little annoyance. However, unlike some additions, the annoyance has already been present, via the cheap-ass aluminum versions.

So in practice the net effect is making the real deal less common relative to the knockoffs.

@Skidborg

This comment has been minimized.

Copy link
Contributor

commented Jan 7, 2016

"It's already a little annoying." Does not justify making it more so. Having the fake and real items share a name is already a bad design.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jan 7, 2016

True. But like I said, this is by no means the worst idea to date. :V

"symbol": "/",
"color": "light_gray",
"name": "scimitar",
"description": "This is a curved sword associated with various middle eastern and central asian countries. This one seems oddly dull and worn.",

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 8, 2016

Member

Please capitalize the "A" in "asian".

This comment has been minimized.

Copy link
@Rivet-the-Zombie

Rivet-the-Zombie Jan 8, 2016

Member

The M and E in Middle Eastern too.

This comment has been minimized.

Copy link
@chaosvolt

chaosvolt Jan 8, 2016

Contributor

That would be my fault. Doh. @_@

"color": "light_gray",
"name": "zweihänder",
"description": "This is a huge two-handed sword from Germany. You're not quite sure it's supposed to be that bendy.",
"rarity": 30,

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 8, 2016

Member

I've never seen this parameter before, what's supposed to do?

This comment has been minimized.

Copy link
@Coolthulhu

Coolthulhu Jan 8, 2016

Contributor

Probably an old value for old drop system.
Should be removed as it doesn't do anything nowadays.

This comment has been minimized.

Copy link
@illi-kun

illi-kun Jan 8, 2016

Member

I found the same parameter in archery.json, melee.json, and tools.json and I'm going to remove it in separate PR.

@kevingranade kevingranade merged commit 82da8c0 into CleverRaven:master Jan 16, 2016

1 check passed

default
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.