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

CRIT Expansion Update #33556

Merged
merged 415 commits into from
May 8, 2020
Merged

CRIT Expansion Update #33556

merged 415 commits into from
May 8, 2020

Conversation

Soupster89
Copy link
Contributor

@Soupster89 Soupster89 commented Aug 26, 2019

Summary

SUMMARY: Mods "Update"

Purpose of change

The start of a line of changes that aims to slowly flesh out the identity of the CRIT content mod. My mission is to refine a mod that evolves into a more polished experience that reflects well on the devs and maintainers that are a part of this wonderful community.

Basic rebalance or additions:

  • MA Styles have been balanced to fit their respective themes.

  • The Rivet Gun no longer produces explosions or shrapnel.

  • Guns have been majorly buffed

  • Techniques offer less eye-stress in regards to the descriptions and should feel more "correct"

  • Minor Mutation additions

  • Various items have been changed

Major Changes:

  • Dark Souls content will become a new mod PR (moved for now)

  • Dead Space content will stay to avoid destroying saves

  • Extended Techniques will stay in the mod (to avoid destroying old saves)

Checklist

Resolve all the issues and get back into standards:

  • Weight Strings

  • Sufficient Spaces

  • Miscellaneous

Re-do files that have been modified to pass conflicts back to rebalanced version:

  • Items

  • Constructions

  • Martial arts

  • Classes/ Scenarios

@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding labels Aug 26, 2019
@Soupster89
Copy link
Contributor Author

Having difficulty seeing if enemies spawn or not, unsure if its because its RNG or because it's being overlooked.

@ghost
Copy link

ghost commented Sep 7, 2019

Stumbled on this looks cool famalam!

@ZhilkinSerg
Copy link
Contributor

"repaired_with": [ "plastic_chunk", "chunk_rubber" ],

should be

"repaired_with": "chunk_rubber",

@Soupster89
Copy link
Contributor Author

Another check in the box!

@Soupster89
Copy link
Contributor Author

Soupster89 commented Apr 4, 2020

Nice! I did it guys!

I have stamped out all the bugs and now I can definitely run a 100% good playtest.

Edit: Nvm, something else went wrong.

@Maleclypse
Copy link
Member

Nice! I did it guys!

I have stamped out all the bugs and now I can definitely run a 100% good playtest.

Edit: Nvm, something else went wrong.

Hate when that happens but it at least it's progress. :)

Copy link
Contributor

@Arcangelus Arcangelus left a comment

Choose a reason for hiding this comment

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

This should address the conflict with data/mods/CRT_EXPANSION/items/crt_clothes.json
You may also want to remove the unchanged mutations GOODHEARING, FANGS, NAILS from data/mods/CRT_EXPANSION/mutations/crt_vamp_mutations.json

data/mods/CRT_EXPANSION/items/crt_clothes.json Outdated Show resolved Hide resolved
data/mods/CRT_EXPANSION/items/crt_clothes.json Outdated Show resolved Hide resolved
Arcangelus and others added 8 commits April 21, 2020 00:56
Copy-pasting mutations can have some weird side effects, depending on the order the definitions ends up loaded. This way will avoid that, save space and allow the trait to be in line with any future vanilla change short of deletion. For a few, it also fixes unintended nerfs:
FASTHEALER didn't include "mending_modifier" : 2
WEAKSCENT didn't include scent_intensity, so it was useless.
PRETTY lacked visibility.
THINSKIN lacked a recent nerf of -1 bash.
ALBINO, SUNBURN lacked a type.

I also removed the mutations GOODHEARING, FANGS, NAILS from here as nothing was done with those.
Changed to use the preferred extend format where applicable. Nothing else changed.
The following unintended nerfs were reverted:
SMELLY lacked scent_intensity, probably negating its effect.
REGEN lacked mending_modifier:16
Update crt_vamp_mutations.json
@Soupster89 Soupster89 changed the title [CR] CRIT Expansion Update CRIT Expansion Update May 3, 2020
@Soupster89
Copy link
Contributor Author

I am finally ready, whoo hoo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON Mods Issues related to mods or modding (P5 - Long-term) Long-term WIP, may stay on the list for a while.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants