Thanks for all the time and effort put into this update Meneth. It's a fantastic mod, and I wouldn't be able to play CKII without it. Cheers.
Also, it doesn't seem as if the Fighter (Poor, Trained, Skilled, Formidable) traits are available in the character generator. Is this purposeful or mistaken?
This is a change that came in with the [partial] integration of the New Duel Engine's latest update.
Meneth, we could modify the distribution event to simply not assign a combat trait to any character that already has one assigned; it should already do this for robustness reasons (history files with combat traits, any future extensions-- in this mod or not, but compatible-- to initial combat trait distribution for some subpopulation, etc.). Then we could open the combat traits back up to the customizer. They are already pretty expensive in years (IIRC; I usually script a custom character and avoid the RD, but when using it, I've never previously selected them, because I knew that I could get them for free along with a martial education trait), mostly due to the health bonuses, and at the highest tier, vassal opinion and sex appeal opinion. Since a player's on their own after the first generation with only the fairly-random guardianship trait assignment system and my in-progress but still unreleased work to make adult rulers' combat trait advancement possible / fairly deterministic, I support giving players the ability to pay the age cost in the RD for whatever tier of combat trait they want.
One should take a look at what the actual age costs are, though. It's possible that they're overly cheap compared to what they allow you to do in-game, although it is admittedly mostly just flavor when it comes down to it. Custom costs can be specified for the traits in the RD if it's a concern.
Radz- It was purposeful from another author. It was probably just overlooked by Meneth. The reason the traits were closed to the customizer is because the latest duel engine assigns the traits randomly after startup and jordarkelf (New Duel Engine), I guess, didn't want anything but the initial random distribution to be assigned. PB uses a different method than jordarkelf's for initial assignment (that also happens before you unpause), FWIW. The PB approach is not at all incongruous with allowing you to customize these traits; the option is just off right now by default.
EDIT: I sent off some proposed changes to the code to turn customizability on again.
.png){kind=avatar}
