Skip to content

Commit

Permalink
Update item_groups.json
Browse files Browse the repository at this point in the history
#### Summary
```SUMMARY: Content "reweight gemstones in item_groups.json"```
#### Purpose of change

Diamonds and pearls are more commonly worn gemstones in New England.
#### Describe the solution

The item group is subtype is not defined. Therefore it defaults to old, which works as a distribution.

The item_group previously had each item weighted 50. All were equally likely.

I&CleverRaven#39;ve changed diamond and pearl to 100. So now the probability is:

* diamond: 12.5%
* pearl: 12.5%
* any other: 75%
#### Describe alternatives you've considered

I considered making emeralds less common due to their rarity, but I couldn&CleverRaven#39;t determine an appropriate ratio.
#### Testing

None! I did this through github in the browser so no testing was done.
#### Additional context

I want to add turquoise to the table too, but I figured there might be a good reason why this common gemstone was left off.
  • Loading branch information
Jerimee authored Jan 5, 2020
1 parent d34a729 commit 8cd3d7a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions data/json/itemgroups/item_groups.json
Original file line number Diff line number Diff line change
Expand Up @@ -3102,7 +3102,7 @@
"type": "item_group",
"items": [
[ "citrine", 50 ],
[ "diamond", 50 ],
[ "diamond", 100 ],
[ "emerald", 50 ],
[ "peridot", 50 ],
[ "amethyst", 50 ],
Expand All @@ -3111,7 +3111,7 @@
[ "blue_topaz", 50 ],
[ "tourmaline", 50 ],
[ "alexandrite", 50 ],
[ "pearl", 50 ],
[ "pearl", 100 ],
[ "opal", 50 ],
[ "ruby", 50 ],
[ "garnet", 50 ]
Expand Down

0 comments on commit 8cd3d7a

Please sign in to comment.