-
Notifications
You must be signed in to change notification settings - Fork 84
Description
Just formalizing what I've been planning to do for a while now. Sub-issue of #312.
TL;DR
The Race patcher is far too complex for a single patcher. It's more complex than Import Actors, and that's split into four patchers!
Splitting it is necessary if we want to have any chance of moving it to newer games.
Details
My original plan was to split it into three patchers - Import Races, Tweak Races and Race Fixer/NPC Fixer (not sure about that name).
However, my attempt at splitting it failed miserably, because I didn't properly understand how it works.
It turns out the CBash version is already nicely split into five patchers, they've just been hackily glued together:
Import Races- simple preserving importer, handlesBody-F,Body-M,Body-Size-F,Body-Size-M,Hair,R.Attributes-F,R.Attributes-M,R.Description,R.Ears,R.Head,R.Mouth,R.Skills,R.Teeth,Voice-FandVoice-MImport Races: Eyes- this still does too much, it merges eyes via theEyestag but also does all the crazy eye/hair fixingImport Races: Relations- merger/preserver, handlesR.Relations, very similar to Import Relations, maybe we could invent anR.Dereltag tooImport Races: Spells- merger/preserver, handlesR.AddSpellsandR.ChangeSpellsTweak Races- houses the races tweaks (e.g. Bigger Nords and Orcs)
Notably absent is the NPC fixing functionality of the PBash version, CBash just doesn't seem to have that.
Final split design, arrived at after one attempt that failed due to regressions:
Import Races- same as aboveImport Races: Relations- same as aboveImport Races: Spells- same as aboveTweak Races- same as aboveEye Checker- the googly eye filteringRace Checker- the hair/eye sortingNPC Checker- the NPC eye/hair fixing